On 5 July 2017 at 15:49, Donald Stufft
I’ve had a niggling feeling about this hook from the beginning, but I couldn’t quite put my finger on it until Nathaniel’s email made me realize it. I feel like this hook is really *only* useful for flit, and for other projects it is largely either going to be completely redundant or be an attractive nuisance that ends up only causing issues. It’s a pretty narrow use case where this hook is both able to do something AND doesn’t have the exact same requirements as build_sdist.
When I felt this was a more generic hook, I was OK with it, but I don’t think it’s a good idea now that I’ve thought on it more and it feels entirely ungeneric.
I don't think Thomas's plans for it are unusual, as it's normal for a build system to only be aware of the input files that are actually referenced by the build recipe, and also normal for published source archives to include additional files that *aren't* used by the build process for the binary artifacts. If you'd prefer some external validation for the concept, I see the "prepare_input_for_build_wheel" hook as fairly analagous to the "%prep" phase in the process of building an RPM: https://fedoraproject.org/wiki/How_to_create_an_RPM_package#.25prep_section The current difference is that we expect backends to be able to cope with frontends *not* calling that implicitly when building from an unpacked sdist. Cheers, Nick. -- Nick Coghlan | ncoghlan@gmail.com | Brisbane, Australia