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

haskellPackages.shellFor: fix after recent getHaskellBuildInputs fix. #45348

Merged
merged 1 commit into from Sep 2, 2018

Conversation

shlevy
Copy link
Member

@shlevy shlevy commented Aug 19, 2018

Fixes #45318

@shlevy
Copy link
Member Author

shlevy commented Aug 19, 2018

@sellout Please review
@choener FYI

@shlevy
Copy link
Member Author

shlevy commented Aug 19, 2018

Will cherry-pick to 18.03 after 👍 from Greg.

@shlevy
Copy link
Member Author

shlevy commented Aug 22, 2018

@sellout ping? I don't use shellFor myself, would like to have confirmation from a user.

@choener
Copy link

choener commented Aug 22, 2018

Both the actual patch commit and the patch on top of git rebase master work for me.

Many thanks,
Christian

@cocreature
Copy link
Contributor

Ping :) I can confirm that this fixes things for me (after applying the patch to HEAD) so it would be nice if this could be merged.

@sellout
Copy link
Contributor

sellout commented Sep 2, 2018

Hey, sorry. I am in the wilderness. But I don’t think I use shellFor at all either.

@peti peti merged commit b35b8ba into NixOS:master Sep 2, 2018
@shlevy
Copy link
Member Author

shlevy commented Sep 5, 2018

Oops, got my wires crossed, meant to ping @ElvishJerricco !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants