-
-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
clementine: added switches for optional dependences #23730
Conversation
Added switches for ipod, mtp, cd and cloud support. The switch for ipod support, specifically, allows users to build it without libplist, wich is currently marked as insecure.
@xvapx, thanks for your PR! By analyzing the history of the files in this pull request, we identified @jraygauthier, @bjornfor and @joachifm to be potential reviewers. |
Good approach, I'd like to reinstall clementine. Trying it out ... |
EDIT with your patch + Could you set the default for |
No problem, i didn't know if it was better to maintain the defaults to current values, i'll change it. EDIT: sorry, i didn't even know about |
I tested it again with |
That's because You probably wanted |
Lol thanks, i completely misunderstood it. i'm going to read about it and try it, thanks! EDIT: done, it works perfectly, checked against master. |
'fix/clementine-add-options'
Added switches for ipod, mtp, cd and cloud support.
The switch for ipod support, specifically, allows users to build it without libplist, wich is marked as insecure.
Motivation for this change
issue #23696
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)I have actually Tested using sandboxing and using
nix-shell -p clementine --run "clementine-review wip"
, but these methods seem to ignorenixpkgs.config.clementine.ipod = false;
in my configuration.nix, so i get the same result i got before my changes, an error stating that libplist-1.12 is marked as insecure.