Bug #80114 for Dist-Zilla: config.ini.old considered an "existing configuration" by dzil setup

Wed Oct 10 15:00:26 2012 KENTNL [...] cpan.org - Ticket created

Subject:

config.ini.old considered an "existing configuration" by dzil setup

move config.ini to config.ini.old

Expectation: config.ini.old should be ignored.

Reality:

Show quoted text

>dzil setup -v

per-user configuration files already exist in C:\Users\KentF\.dzil: C:\Users\KentF\.dzil\config.ini.old at C:/strawberry/perl/site/lib/Dist/Zilla/App/Command/setup.pm line 34.

Apparently, this is interpreting 'config.ini.old' as { basename => 'config' , extension => 'ini.old' } when by convention, periods are not part of the file extension, and by convention, I believe it should be interpreted as { 'basename' => 'config.ini', extension => 'old' }

I can't recall where this is handled, perhaps Config::MVP , but the problem is only apparent to 'dzil setup' , because other commands are probably going 'oh right, I have no handler for the extension 'ini.old' , so we'll just silently pretend theres no config file'

Tue Jan 01 18:11:31 2013 rjbs [...] cpan.org - Correspondence added

Not much of a mystery here if you look at the code from which the error came: my $config_root = Dist::Zilla::Util->_global_config_root; if ( -d $config_root and my @files = grep { -f and $_->basename =~ /\Aconfig\./ } $config_root->children ) { $chrome->logger->log_fatal([ "per-user configuration files already exist in %s: %s", "$config_root", join(q{, }, @files), ]); Fixed by: [master f3a8cf9] require that the extension is dotless -- rjbs

Tue Jan 01 18:11:32 2013 The RT System itself - Status changed from 'new' to 'open'

Tue Jan 01 18:11:32 2013 rjbs [...] cpan.org - Status changed from 'open' to 'resolved'

Bug #80114 for Dist-Zilla: config.ini.old considered an "existing configuration" by dzil setup

Preferred bug tracker