Skip to content
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

k3b misses cdrkit dependency #6002

Closed
solatis opened this issue Jan 27, 2015 · 6 comments
Closed

k3b misses cdrkit dependency #6002

solatis opened this issue Jan 27, 2015 · 6 comments
Assignees
Labels
6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS

Comments

@solatis
Copy link

solatis commented Jan 27, 2015

In the current unstable channel, k3b complains about a missing "cdrecord" executable. This executable can be found in the "cdrkit" package. After manually installing this package, k3b works fine.

This dependency should be added to the k3b package.

@ttuegel
Copy link
Member

ttuegel commented Jan 28, 2015

This is a known issue, and cannot be fixed with the current architecture of Nix. The cdrecord executable must be setuid-root, so it cannot reside in the store. Adding the dependency to the K3B package will not help.

As a workaround, we should add a NixOS option, such as programs.k3b.enable which installs K3B and cdrkit into the system environment.

@ttuegel ttuegel added the 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS label Jan 28, 2015
@ttuegel ttuegel self-assigned this Jan 28, 2015
@edolstra
Copy link
Member

IIRC, cdrecord doesn't have to be setuid root. That's only necessary to get realtime scheduling. But that's probably not so important anymore.

@ttuegel
Copy link
Member

ttuegel commented Jan 28, 2015

In that case, I'll wrap k3b to suffix the path with cdrecord. If the user has the setuid version installed, that will be used instead.

@solatis
Copy link
Author

solatis commented Jan 30, 2015

I might be missing something, but wouldn't just adding cdrkit as a dependency solve the problem? If you look at this:

https://github.com/NixOS/nixpkgs/blob/master/pkgs/tools/cd-dvd/cdrkit/default.nix#L19

it makes wodin alias to cdrecord, which seems to work just fine in this case.

@ttuegel
Copy link
Member

ttuegel commented Jan 30, 2015

@solatis That will make cdrecord available at build-time, but not at runtime.

@ttuegel
Copy link
Member

ttuegel commented Feb 1, 2015

Fixed in 95829c8. I used propagatedUserEnvPkgs to propagate a dependency on cdrkit. Users who prefer the original cdrtools can override the cdrtools argument. This should not interfere with setuid root tools, which appear first on the path.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS
Projects
None yet
Development

No branches or pull requests

3 participants