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

ruby.withPackages: preserve setup hooks #72743

Merged
merged 1 commit into from
Nov 3, 2019

Conversation

alyssais
Copy link
Member

@alyssais alyssais commented Nov 3, 2019

This fixes

nix-shell -p 'ruby.withPackages (const [])' ruby.devdoc

which otherwise wouldn't find documentation, unlike

nix-shell -p ruby ruby.devdoc

which would, because ruby has setup hooks to accomodate for this, that
were being masked by the withPackages wrapper.

Verified

This commit was signed with the committer’s verified signature. The key has expired.
alyssais Alyssa Ross
This fixes

    nix-shell -p 'ruby.withPackages (const [])' ruby.devdoc

which otherwise wouldn't find documentation, unlike

    nix-shell -p ruby ruby.devdoc

which would, because ruby has setup hooks to accomodate for this, that
were being masked by the withPackages wrapper.
@alyssais alyssais requested a review from manveru November 3, 2019 21:12
@alyssais alyssais requested a review from zimbatm as a code owner November 3, 2019 21:12
@ofborg ofborg bot added 6.topic: ruby 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux labels Nov 3, 2019
@zimbatm zimbatm merged commit 547b5b7 into NixOS:master Nov 3, 2019
dtzWill pushed a commit to dtzWill/nixpkgs that referenced this pull request Nov 3, 2019

Verified

This commit was signed with the committer’s verified signature. The key has expired.
dtzWill Will Dietz
This fixes

    nix-shell -p 'ruby.withPackages (const [])' ruby.devdoc

which otherwise wouldn't find documentation, unlike

    nix-shell -p ruby ruby.devdoc

which would, because ruby has setup hooks to accomodate for this, that
were being masked by the withPackages wrapper.

(cherry picked from commit 547b5b7)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: ruby 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants