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
shadowfox-updater: init at 1.5.2 #42313
Conversation
@GrahamcOfBorg build shadowfox |
Success on x86_64-linux (full log) Attempted: shadowfox Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: shadowfox Partial log (click to expand)
|
{ fetchFromGitHub, buildGoPackage }: | ||
|
||
buildGoPackage rec { | ||
name = "shadowfox-updater-${version}"; |
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.
The package in all-packages.nix
is called shadowfox
. It should have the same name here to avoid confusion (not shadowfox-updater
).
@xeji Fixed that, also added |
@GrahamcOfBorg build shadowfox |
Success on x86_64-linux (full log) Attempted: shadowfox Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: shadowfox Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: shadowfox Partial log (click to expand)
|
I disagree about this package being categorized under "networking". Mind moving it to "misc" or possibly "package-management"? |
Good point. |
I've been thinking and bringing this up for a while now: The directory-based categories are inherently flawed because of constant ambiguity, lots of packages somewhat belong into multiple directories. I hope to eventually bring a tagging system to nixpkgs which could solve that problem. Anybody with commit rights feel free to move the package, I won't open a PR for that, it's such a minor thing and doesn't matter much anyways imo. |
Motivation for this change
This adds the ShadowFox updater, which is used to install ShadowFox: https://github.com/overdodactyl/ShadowFox/, a full black theme for firefox.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)