-
-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
quick-webapps: init at 0-unstable-2024-08-05 #322226
base: master
Are you sure you want to change the base?
Conversation
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, but I guess I have to link to:
#327063
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.
The upstream name changed:
https://github.com/elevenhsoft/WebApps
The developer said that is is Quick WebApps: |
Yes, but you are using the name to derive the repo. |
Sorry I don't follow? |
The |
It built fine for me? I think I'm following but the logic of how it is currently tracks. |
Oh I see now that is new. |
2f3ebf2
to
17107a6
Compare
postInstall = '' | ||
wrapProgram "$out/bin/${pname}" \ | ||
--suffix XDG_DATA_DIRS : "${cosmic-icons}/share" \ | ||
--prefix LD_LIBRARY_PATH : ${ |
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.
Can we avoid LD_LIBRARY_PATH by using patchelf?
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.
I'm not quite sure about how to go about that but I'm not against if someone does it as long as it works.
Can it be backported to 24.05 to be integrated into https://github.com/lilyinstarlight/nixos-cosmic flake that indeed work with stable? |
17107a6
to
478f276
Compare
Description of changes
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.