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.unite-shell: rename to gnomeExtensions.unite #109880
Conversation
@ofborg eval |
@SuperSandro2000 is my commit message wrong, that's why ofborg can't build it? should I rename the commit to |
I am not actually sure what is happening here. @ofborg eval Edit: I guessed it is fixed. |
@rhoriguchi Sorry that I not noticed this earlier but can you add an alias to aliases.nix? I think a normal one without throw should be fine. |
Should be fine now. Please verify that the alias is correct. I'm not sure this is the correct way. Should a set be used or is this fine? |
Good catch. Actually I am not sure but looking at other examples we maybe want it? https://github.com/rhoriguchi/nixpkgs/blob/unite/pkgs/top-level/aliases.nix#L197 |
@jonringer du you think this is correct this way? |
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.
cc gnome people @jtojnar @worldofpeace
Personally, I think the
|
Right, using Moving the extensions to a separate file would be a good improvement as well. And we should probably rename the attribute to But this looks good to me as is. Thanks. |
Building with this commit in master is broken for me, seemingly because of the alias - this is the first bad commit I could find that I couldn't build on.
|
yea, the intention was correct, but the implementation needs to change:
|
nested attrs are not allowed to be aliased in |
fixed in 31e833e
|
well, more accurately, there's an assertion that top-level attrs aren't being shadowed. For nested attrs, it means they will always fail as they all exist on a top-level attr. |
Kind of surprised ofborg didn't fail, as it should have tried to do query rebuilds... unless that's done with |
Motivation for this change
After using the
gnome-shell-extension-prefs
i realized this package should be namedunite
.On GitHub repository the extension is mentioned under the name
Unite
also the header in the demo images is alsoUnite
.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)