New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
trilium: 0.40.7 -> 0.42.1 #86192
trilium: 0.40.7 -> 0.42.1 #86192
Conversation
@GrahamcOfBorg build trilium-desktop |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the the deprecation and update should be done in two separate commits. trilium: deprecate for trilium-desktop
and trilium-desktop: 0.40.7 -> 0.41.6
maybe?
EDIT: As noted below, since both packages are updated in tandem, trilium: 0.40.7 -> 0.41.6
is the correct commit message.
0023ad3
to
86cf1f9
Compare
I split the commits. As for the commit message: trilium is currently updated as one (server and desktop) as they are built from the same source release. I'd therefore opt to keep the generic |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Diff LGTM, builds fine, runs fine.
[6 built, 4 copied (0.7 MiB), 0.1 MiB DL]
https://github.com/NixOS/nixpkgs/pull/86192
2 packages built:
trilium-desktop trilium-server
@GrahamcOfBorg build trilium-desktop |
Result of 2 packages built:- trilium-desktop - trilium-server |
Motivation for this change
We currently have two package names for the trilium desktop app,
trilium
andtrilium-desktop
, which feels like a sensible distinction as the server packagetrilium-server
also exists.With this change, we deprecate the original
trilium
package in favor of the more explicit naming scheme.Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)