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
Gnome shell extension pano #189091
Gnome shell extension pano #189091
Conversation
How much work would it be to get this to work as an automatically packaged extension (i.e. through our patch mechanism in |
@piegamesde Not sure yet, I will give it a shot. |
In the meantime, for anybody interested, this overlay works i.a. on nixos-22.05: https://github.com/michojel/NixOS/blob/master/overlays/gnome-shell-extension-pano.nix |
28039b6
to
f1da58b
Compare
a47a3ce
to
56448f4
Compare
I'm now using pano with the following overlay: final: prev: with prev;
{
pano = callPackage "${
builtins.fetchTarball "https://github.com/michojel/nixpkgs/archive/gnome-shell-extension-pano.tar.gz"
}/pkgs/desktops/gnome/extensions/pano" {};
} I added |
Good catch! It looks to be a part of Yaru theme:
That seems like an unwanted dependency that should be reported upstream. |
@jtojnar would you agree to set |
I would just ignore it since it is a minor cosmetic detail. Or wait for upstream to vendor the icon as per GNOME guidelines. |
Bumped to v9. Removed |
Signed-off-by: Michal Minář <mm@michojel.cz>
based on: oae/gnome-shell-pano@d727754 Signed-off-by: Michal Minář <mm@michojel.cz>
Signed-off-by: Michal Minář <mm@michojel.cz>
b823145
to
8f2fc56
Compare
Signed-off-by: Michal Minář <mm@michojel.cz>
8f2fc56
to
9d70792
Compare
as it is a standard path Signed-off-by: Michal Minář <mm@michojel.cz>
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
|
let | ||
dataDirPaths = lib.concatStringsSep ":" [ | ||
"${gnome.gnome-shell}/share/gnome-shell" | ||
"${gnome.mutter}/lib/mutter-10" |
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.
This is now mutter-11
. You can use gnome.mutter.libdir
instead.
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.
Also please follow the contributing guide on how commit messages should be structured.
}); | ||
in | ||
stdenv.mkDerivation rec { | ||
inherit pname 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.
Why not use those here directly, when we have rec
attrset?
]; | ||
|
||
buildInputs = [ | ||
atk |
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.
Do not see gtk3 and atk needed.
name = "${pname}-modules-${version}"; | ||
packageJSON = "${src}/package.json"; | ||
yarnLock = "${src}/yarn.lock"; | ||
yarnNix = ./yarn.nix; |
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 is not clear how to update this. Maybe add an passthru.updateScript
. Did not test it but something like the following should work:
passthru = {
lockFileNix = runCommand
"pano-yarn.nix"
{
nativeBuildInputs = [
yarn2nix-moretea
];
}
''
unpackPhase
cd "''${sourceRoot:-.}"
yarn2nix > "$out"
'';
updateScript =
let
updateSource = gitUpdater { };
updateLockFile = _experimental-update-script-combinators.copyAttrOutputToFile "gnomeExtensions.pano.lockFileNix" ./yarn.nix;
in
_experimental-update-script-combinators.sequence [
updateSource
updateLockFile
];
};
@jtojnar would a small patch in extensionOverrides.nix be preferred to packaging this extension separately? With the following overlay the extension works just fine for me. final: prev: {
gnomeExtensions = prev.gnomeExtensions // {
pano = prev.gnomeExtensions.pano.overrideAttrs (oldAttrs: {
patches = [
(final.substituteAll {
src = ./pano.patch;
gda_path = "${final.libgda}/lib/girepository-1.0";
gsound_path = "${final.gsound}/lib/girepository-1.0";
})
];
});
};
} diff --git a/extension.js b/extension.js
index 26561f2..01209e7 100644
--- a/extension.js
+++ b/extension.js
@@ -1,3 +1,5 @@
+imports.gi.GIRepository.Repository.prepend_search_path('@gda_path@')
+imports.gi.GIRepository.Repository.prepend_search_path('@gsound_path@')
try {
|
@piousdeer yet it would be preferred: #189091 (comment) A PR with your patch would be appreciated. |
Description of changes
Resolves #188736
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes