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
cinnamon.cinnamon-screensaver: init at 4.4.0 #86069
Conversation
installPhase = '' | ||
mkdir -p $out/share | ||
mv build $out/share/iso-flags | ||
''; |
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.
makefile really has no install smh
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.
so is this ok? or what should we do?
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.
Ideally, we would open a PR adding the install target to https://github.com/joielechong/iso-country-flags-svg-collection/blob/master/Makefile
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.
Don't know if that's what the authors intent, as the project is less about installable icon sets, more about generating the resources. It's just the linuxmint team that decided to package it up
postFixup = '' | ||
for f in $out/bin/*; do | ||
wrapProgram "$f" "--prefix" "GI_TYPELIB_PATH" ":" "$out/lib/girepository-1.0" | ||
done | ||
''; |
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 should be unneeded if you have gobject-introspection
in nativeBuildInputs
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 doesn't recognize the typelib in $out
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 really should https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/libraries/gobject-introspection/setup-hook.sh#L21, if not we have a very problematic bug. Do you see giDiscoverSelf
in nix log
?
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.
And they ported to meson on master linuxmint/cinnamon-screensaver@997ae6a 🤣 |
I already tried to package master, but it seems to be one major ahead and doesn't work with the old stuff |
49075ee
to
e4c4062
Compare
4.6.0 seems out, so I'd have to update this |
Unsure how to handle since we'd have a mix of 4.4 on master, 4.4 on my patch branch (stuff that isn't upstreamed) and a bunch of 4.6 stuff I prefer not breaking my current setup 🤣 |
Getting 4.4 shipped, then updating to 4.6? (All of cinnamon, then) |
d82fcc9
to
dd1e6ea
Compare
d73e9db
to
5786d36
Compare
Co-authored-by: Jan Tojnar <jtojnar@gmail.com>
5786d36
to
ce2895d
Compare
@ofborg eval |
Had to fix release.nix to evaluate 😦 |
Motivation for this change
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)