stsc / DateTime-Format-Natural

Parse informal natural language date/time strings
5 stars 5 forks source link

distribution includes both a Makefile.PL and Build.PL [rt.cpan.org #106723] #9

Closed stsc closed 1 year ago

stsc commented 3 years ago

Migrated from rt.cpan.org#106723 (status was 'new')

Requestors:

From ether@cpan.org on 2015-08-28 21:37:10 :

Providing both files is generally not a good idea -- e.g. see http://neilb.org/2015/05/18/two-build-files-considered-harmful.html

Module::Build::Compat especially is a giant bag of razorblades and best avoided. The easiest thing to do is to drop the use of the 'compat' option in Module::Build, but since you're providing a Makefile.PL with no evident loss of functionality, even better would be to drop the use of Module::Build entirely and just use ExtUtils::MakeMaker in Makefile.PL. 
stsc commented 1 year ago

Decided to drop the compat Makefile.PL as advised, but to keep the use of Module::Build.

Closing. Feel free to open a new issue as the need arises.