Conversation
6266451 to
78f455f
Compare
78f455f to
eb4545f
Compare
|
| @@ -112,6 +278,9 @@ stdenv.mkDerivation { | |||
| homepage = "https://dragonflydb.io/"; | |||
| license = licenses.bsl11; | |||
| platforms = platforms.linux; | |||
| maintainers = with maintainers; [ yureien ]; | |||
| maintainers = with maintainers; [ | |||
| typedrat | |||
There was a problem hiding this comment.
We're standardizing on using Nix for our development environment at work, and well... That means we need all of our runtime dependencies to be up to date in nixpkgs.
Thankfully, I think that most updates in the future will be more or less mechanical tweaks to the patch files. (Famous last words?)
mdaniels5757
left a comment
There was a problem hiding this comment.
One other thing: the commit message for your last commit should be "dragonflydb: 0.1.0 -> 1.34.2". Same for the PR title.
ee43a7b to
11afb61
Compare
11afb61 to
35e95bd
Compare
This comment was marked as outdated.
This comment was marked as outdated.
|
I don't have access to a Mac and their docs simultaneously claim that they're Linux only but then they have a bunch of Mac stuff in their CMake configs? It may end up being for the best to just follow upstream and say it's Linux-only. |
|
Sounds good to me. |
35e95bd to
d433f80
Compare
|
I have done so. |
|
mdaniels5757
left a comment
There was a problem hiding this comment.
Approved automatically following the successful run of nixpkgs-review.
|
We were asked about this change because of the NixOS release freeze. I would say, we can still update it, as having a more up-to-date package that receives security updates. Tho, have you tested that the NixOS module still works and does the automatic migration of the data directory work? I see no comment that this was tested in this PR. Could you please also add a release note entry. |
|
Also setting a security flag on this. A quick look shows multiple issues (CVE-2025-26268, CVE-2025-26269, CVE-2025-52935…) and I'm guessing my 15 seconds search on a smartphone is incomplete |
Those are all resolved, as is every other CVE I can find against dragonflydb. |
Can confirm that the NixOS module still works at a fundamental level, but I haven't exercised every option. The options-to-flags mapping seems current from a quick glance over the docs. Haven't tested data dir migration. |
|
This is my first time touching the release notes, so if you want me to squash it or change the line I added, please tell me. |
72b0527 to
03c505e
Compare
|
Rebuilded locally and played a bit with it. Everything seems to be OK. Can you rebase to resolve the conflict on the release notes so we can merge this? |
03c505e to
5f938a9
Compare
Done. |
LeSuisse
left a comment
There was a problem hiding this comment.
Let's merge
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 452792
Commit: 5f938a94bcfa4151f24652359745504f4786a449
x86_64-linux
⏩ 1 package blacklisted:
- tests.nixos-functions.nixos-test
✅ 4 packages built:
- croncpp
- dragonflydb
- nixpkgs-manual
- uni-algo
It's been several years since this was touched. Honestly, I can't entirely blame anyone considering what a pain it was to update 😅.
Closes: #243300
Closes: #361454
Closes: #405217
Closes: #405608
Closes: #452741
maintainer: @Yureien
Things done
passthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.Add a 👍 reaction to pull requests you find important.