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
gnomeExtensions.no-title-bar: 9 -> 11, unbreak #106828
Conversation
9 -> 2020-05-14 Switches to a supported fork, updates patches as necessary
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 minus minor comment
- Commits should probably be squashed
- Builds fine via
nix-review
on x86_64-linux. No further testing done (now sure how, on headless machine).
https://github.com/NixOS/nixpkgs/pull/106828
1 package built:
gnomeExtensions.no-title-bar
39753a6
to
b0cc666
Compare
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
- Commits LGTM
- Builds via
nix-review
: - No further testing done by me
https://github.com/NixOS/nixpkgs/pull/106828
1 package built:
gnomeExtensions.no-title-bar
@ofborg eval |
This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 1 package built:
|
Is there anything I can do to get this merged? Would like to see this in master before 21.05 and there's another release of this extension 😅 |
LGTM |
Motivation for this change
Second attempt at #88679, since upstream tagged a release. I've been running this on my system for a while, it all works.
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)