Perl-Toolchain-Gang/Module-Build — comment on commit 886abd6b50511cc3183162817cae0fdfa18f060f
File:  line 0
By dagolden:

My thoughts:

* I don't like commit wars. I'd rather have discussion on a commit that seems flawed until consensus is reached before reverting rather than revert and *then* discuss. I think it's friendlier that way.
* In a configure/make/test/install cycle, I do think that most of the arguments should be given in the configure stage, even if they apply to later actions. E.g. you specify "-Dprefix" to a configuration script, which creates the Makefile, even though the prefix arg isn't used until "make install"
* I'm not sure historically why it wasn't done that way in Module::Build and whether it was thoughtful design or just happenstance. CPAN.pm allows users to specify options at various stages (e.g. suggesting '--uninst=1' for an 'install' argument) so maybe that's why.
* I do think that if we allow 'install' command arguments, it's reasonable to give an example of those as well. Maybe "--uninst=1' is a reasonable one to use install of install base/path.
* When it comes to API's, command line or config script, I think we should have healthy respect for past precedent and a measure-twice/cut-once attitude about changing it. If we're all going to be in Paris, we should discuss the "Build.PL API" I drafted and agree on future direction.

Net: if we're just changing the *example* and not what's possible, then I'm OK with the original commit.