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

Update default.nix #6056

Closed
wants to merge 1 commit into from
Closed

Update default.nix #6056

wants to merge 1 commit into from

Conversation

psibi
Copy link
Member

@psibi psibi commented Jan 29, 2015

Change it to lowercase to according to the convention presented here: http://nixos.org/nixpkgs/manual/#chap-conventions

Thanks to goibhniu from #nixos for pointing me this.

Change it to lowercase to according to the convention presented here: http://nixos.org/nixpkgs/manual/#chap-conventions

Thanks to goibhniu from #nixos for pointing me this.
@vcunat vcunat closed this in 8e5cf58 Jan 29, 2015
@vcunat
Copy link
Member

vcunat commented Jan 29, 2015

Thanks!

@utdemir
Copy link
Member

utdemir commented Jan 29, 2015

This patch makes imagemagick url invalid. Currently I can't build imagemagick unless if I revert it.

vcunat added a commit that referenced this pull request Jan 29, 2015
@vcunat
Copy link
Member

vcunat commented Jan 29, 2015

Ah, I'm sorry, thanks! I didn't read enough context, thinking this could break nothing. Fixed in b35e0a0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
12. first-time contribution This PR is the author's first one; please be gentle!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants