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: provide free derivation by default #38513
Conversation
This reverts commit ca54d50.
Success on x86_64-linux (full log) Attempted: clementine Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: clementine Partial log (click to expand)
|
''; | ||
enableParallelBuilding = true; | ||
meta = with stdenv.lib; { | ||
homepage = http://www.clementine-player.org; | ||
description = "Spotify integration for Clementine"; | ||
# The blob itself is Apache-licensed, although libspotify is unfree. | ||
license = licenses.asl20; | ||
# Therefore the license will be marked as `unfree` because of its dependencies. | ||
license = licenses.unfree; |
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.
Not sure whether that is needed. Because of its dependency on libspotify
the leaf will be unfree and thus this package would fail to build or install because it requires unfree software.
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'll change that for now. The original issue was filed because of the package shipped with a free license, but libspotify
broke the evaluation on systems which disallow unfree packages. However the affected package is now named clementineUnfree
, so I guess that it should be sufficiently obvious :)
The `clementine` package is actually released under Apache license, but requires the unfree `libspotify` package to build. Now `nixpkgs.clementine` points to the free derivation and `nixpkgs.clementineUnfree` has been introduced for the package bundled with spotify support. Fixes NixOS#38315
8f2a469
to
ef5e212
Compare
Success on x86_64-linux (full log) Attempted: clementine Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: clementine Partial log (click to expand)
|
The current checklist for filing new patches suggests to run `nox-review wip` to ensure that the current change (e.g. package bumps) don't break package bumps: https://github.com/NixOS/nixpkgs/blob/e1dee4efcbffc72260025078bf8297a3b732509c/.github/PULL_REQUEST_TEMPLATE.md I experienced this behavior from time to time (see NixOS/nixpkgs#38513 or NixOS/nixpkgs#38361) which cost me some extra time to spot the reasons for additional breackage. Running `nox-review wip` can be quite time consuming, especially when bumping packages that trigger a mass-rebuild or a stdenv rebuild, *however* this patch helps to reduce the risk of additional breackage caused by unwanted package bumps.
I'm not sure if this is the right thing to do. Shouldn't we just add a config option to "celementine", something like "withSpotify"? You can just call it with |
So you propose to disable the unfree build by default? |
A config option is possible, but less convenient/discoverable at the moment without module system. |
A flag is fine as well, as long as the unfree parts are off by default.
We can't have top-level attributes for every combination of parameters. |
previously we used |
Motivation for this change
The
clementine
package is actually released under Apache license, butrequires the unfree
libspotify
package to build.Now
nixpkgs.clementine
points to the free derivation andnixpkgs.clementineUnfree
has been introduced for the package bundledwith spotify support.
Fixes #38315
Note: I had to revert commit ca54d50 as it broke
clementine
previously (see the Hydra logs at https://hydra.nixos.org/build/72403466)Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)