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
cde-gtk-theme: init at 1.3 #89163
cde-gtk-theme: init at 1.3 #89163
Conversation
I'm getting a 410 gone from the URL:
Seems like this is a temporary URL? |
@sauyon Fixed, thanks |
The details section of the theme description seem to suggest that it's licensed under GPLv3, but that's not found in the repo you added as a source. Are you the author of the theme? It looks like you own the opencode repo, at least. Otherwise, this LGTM. I'll test this on Arch in a bit. |
@sauyon No, I'm not the author, just uploaded it to opencode. I've contacted the author with kind ask to upload it by himself and add license. |
@sauyon Theme author uploaded it to github and added license 👍 I've updated the package accordingly. |
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.
It fits better at the ### DATA
section instead of the current ### BUILD SUPPORT
section of all-packages.nix
.
@romildo Fixed, thank you! |
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.
LGTM
Tested in lxappearance on nixos. Thanks for your contribution! Result of 1 package built:- cde-gtk-theme |
Motivation for this change
Gtk2 / Gtk3 theme mimicking CDE / Motif
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)