Skip to content

Remove obsolete scatter output hook#15123

Merged
joachifm merged 1 commit intoNixOS:masterfrom
redbaron:remove-scatter-output-hook
May 2, 2016
Merged

Remove obsolete scatter output hook#15123
joachifm merged 1 commit intoNixOS:masterfrom
redbaron:remove-scatter-output-hook

Conversation

@redbaron
Copy link
Contributor

Remove obsolete scatter output hook

There are no users of it in main tree and recent merge
of multiple outputs branch makes it obsolete for private trees
too.

At the time hook was created, recently merged multiple output
branch was relying on passing flags to autotools to split
outputs, which obviously wasn't working for other build systems

Scatter output was taking different approach where files were
moved out from a build tree based on known paths, which is more
or less what current multiple-outputs.sh hook is able to do too.

There are no users of it in main tree and recent merge
of multiple outputs branch makes it obsolete for private trees
too.

At the time hook was created, recently merged multiple output
branch was relying on passing flags to autotools to split
outputs, which obviously wasn't working for other build systems

Scatter output was taking different approach where files were
moved out from a build tree based on known  paths, which is more
or less what current multiple-outputs.sh hook is able to do too.
@joachifm joachifm added the 8.has: clean-up This PR removes packages or removes other cruft label Apr 30, 2016
@joachifm joachifm merged commit f125da3 into NixOS:master May 2, 2016
@redbaron redbaron deleted the remove-scatter-output-hook branch May 2, 2016 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.has: clean-up This PR removes packages or removes other cruft

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants