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

Fixes the git installation on Darwin #1277

Closed
wants to merge 1 commit into from
Closed

Fixes the git installation on Darwin #1277

wants to merge 1 commit into from

Conversation

zimbatm
Copy link
Member

@zimbatm zimbatm commented Nov 25, 2013

git is missing the xmlto package to install. This patch just changes the allowed platforms of xmlto to "all" since it compiles fine on Darwin.

@peti
Copy link
Member

peti commented Nov 26, 2013

all includes a lot more platforms than just Darwin. Are you sure all of those -- like Cygwin -- actually build?

Also, I don't see how that setting might have any effect on your ability to install git on Darwin. Hydra builds xmlto anyway when it builds git, so adding Darwin to the list of platforms doesn't change anything. What exactly is the problem that you're trying to solve?

xmlto compiles properly on Darwin.

git depends on xmlto to be installed.
@zimbatm
Copy link
Member Author

zimbatm commented Nov 26, 2013

Ok I have changed the platforms to "unix", I suspect it's going to compile find on the bsds if it's working on darwin.

When I try to install git I'm getting the following error:

$ nix-env -i git
installing `git-1.8.4.3-full'
error: user-thrown exception: the package ‘xmlto-0.0.23’ in «unknown-file» is not supported on ‘x86_64-darwin’
(use `--show-trace' to show detailed location information)

I supposed that hydra is not able to pre-build git if that dependency is failing (?).

@peti
Copy link
Member

peti commented Nov 26, 2013

Ah, now I see what you mean. That issue should be fixed by commit 49db8a5.

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

Successfully merging this pull request may close these issues.

2 participants