Skip to content

Comments

Haskell optional separate bin output#58525

Merged
domenkozar merged 2 commits intoNixOS:masterfrom
domenkozar:haskell-bin-output
Mar 29, 2019
Merged

Haskell optional separate bin output#58525
domenkozar merged 2 commits intoNixOS:masterfrom
domenkozar:haskell-bin-output

Conversation

@domenkozar
Copy link
Member

The first commit adds most minimal separate output support (by not trigger recompilation on generic builder).

The second commit proves it's already useful.

@GrahamcOfBorg GrahamcOfBorg added the 6.topic: haskell General-purpose, statically typed, purely functional programming language label Mar 29, 2019
@domenkozar domenkozar changed the title Haskell bin output Haskell optional separate bin output Mar 29, 2019
@GrahamcOfBorg GrahamcOfBorg added 8.has: clean-up This PR removes packages or removes other cruft 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Mar 29, 2019
@shlevy
Copy link
Member

shlevy commented Mar 29, 2019

Is there a reason we wouldn't want this on by default?

@domenkozar
Copy link
Member Author

domenkozar commented Mar 29, 2019

It's a deeeep rabbit hole to get all packages to build. That's why the PR was stuck for 2 years, let's ship infrastructure first and then do the default switch separately.

@domenkozar
Copy link
Member Author

I'm going to merge this as it's a new toggle defaulting to old behaviour. Feel free to review it regardless and I'll address the comments.

@domenkozar domenkozar merged commit bad5d14 into NixOS:master Mar 29, 2019
@peti
Copy link
Member

peti commented Mar 30, 2019

@domenkozar, it really sucks that you merge this kind of change without giving me (and others) any chance at all to review it first. This is quite anti-social behavior, IMHO, and I am very disappointed because you should know better.

@domenkozar
Copy link
Member Author

@shlevy reviewed it?

@peti
Copy link
Member

peti commented Mar 30, 2019

Very funny.

# Won't find it's header files without help.
sfml-audio = appendConfigureFlag super.sfml-audio "--extra-include-dirs=${pkgs.openal}/include/AL";

cachix = enableSeparateBinOutput super.cachix;
Copy link
Member

Choose a reason for hiding this comment

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

Think this should be in configuration-nix?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm not sure, could be also in common?

@Ericson2314
Copy link
Member

https://github.com/input-output-hk/nix-tools building components separately makes me happy because then there are no bin or lib outputs to separate.

@domenkozar
Copy link
Member Author

@Ericson2314 planning to switch to that soon but since this was a low hanging fruit it was really useful :)

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

Labels

6.topic: haskell General-purpose, statically typed, purely functional programming language 8.has: clean-up This PR removes packages or removes other cruft 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants