allow multiple outputs in ncrystal-feedstock #1344
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In order to split the C++/library parts of NCrystal from the Python layer, the ncrystal-feedstock needs to be modified to use multiple outputs. So before there was just an "ncrystal" package, but in the future there will be at least "ncrystal-core", "ncrystal-python", and "ncrystal" packages, where the last will be a metapackage depending on the rest and ensuring synchronisation of versions. In the future the ncrystal-core itself might be split further (into libs, headers, data files), but that is not going to happen in the short term.
I am the maintainer of the ncrystal-feedstock, so I count myself as having been pinged :-)