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
cc-wrapper: propagate man and info to propagated-build-inputs #44516
Conversation
Thanks so much!! |
Does this actually work? I don't think either I think you should be just able to do:
|
Does this actually work? I don't think either `environment.systemPackages` or `nix-env -i` will follow propagated build inputs.
I'm not sure as I can't reproduce the original issue.
#43547 (comment) says
`nix-support/propagated-build-inputs`, which is strictly stronger
and I made this assuming that's correct. But if it's not, then I feel like `propagated-user-env-packages` makes more sense there semantically (judging by the name, not by the actual semantics whatever it is).
Thinking some more about this, however, I think that a better way is to simply sidestep all of this and symlink `.man` and `.info` outputs to the originals like so
```nix
ln -s ${bintools.man} $man
```
instead of doing it like now
```nix
printWords ${bintools.man or ""} >> $man/nix-support/propagated-build-inputs
```
(or `propagated-user-env-packages` like it was before). I'm now rebuilding `stdenv` with that change to see what happens (found 1 bug in `stdenv` already).
Btw, if we are discussing this, there's also `nativeTools` option which does similar things in `bintools-wrapper`, that goes to `propagated-user-env-packages` (and I think that's correct, semantically speaking).
|
Thinking some more about this, however
... which is exactly your suggestion above too. I gotta get a coffee, it seems. :)
|
See #44558. |
I'm sorry. I might has improperly tested that. (Worse, I have some dejavu as if I made the same mistake before.) In a post |
@Ericson2314 Nah, don't mind, breaking things is half the fun after all. #44558 needs some testing love on Darwin, though. |
See discussion in NixOS#44516.
I agree, I also thought
Why? propagatedBuildInputs are only intended for building stuff that depends on the package. In multiple-outputs packages they aren't even visible at runtime, because the |
You said it yourself; it isn't used on things people want to install anyways. OTOH, if you manually install a |
Manually installing |
Should fix #43547.