Skip to content

Conversation

@domenkozar
Copy link
Contributor

No description provided.

@domenkozar domenkozar requested a review from jmitchell January 31, 2018 19:01
@jmitchell
Copy link
Contributor

The CI job timed out. Can you ensure the CI script passes locally before merging?

@jmitchell jmitchell mentioned this pull request Jan 31, 2018
-- given nixAttr, build it and add $out/bin to $PATH
ensureExecutable :: String -> IO ()
ensureExecutable nixAttr = do
(exitcode2, stdout, err2) <- readProcessWithExitCode "nix-build" ["-A", nixAttr, "<nixpkgs>"] mempty
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this have --no-build-output?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@domenkozar
Copy link
Contributor Author

@jmitchell turns out CI failed due to stack2nix.nix being outdated.

@domenkozar
Copy link
Contributor Author

It works now - but very close to travis-ci time limit.

@domenkozar domenkozar merged commit 4f54c60 into master Feb 1, 2018
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.

4 participants