Bug #56665 for BackPAN-Index: backpan-init broken

Fri Apr 16 11:00:33 2010 daxim [...] cpan.org - Ticket created

Subject:

backpan-init broken

I have the newest version of everything. Show quoted text

> git backpan-init Format::Human::Bytes

Prototype mismatch: sub Git::CPAN::Patch::Import::chmod (@) vs none at /home/daxim/local/lib/perl5/5.10.1/Exporter.pm line 67. at /home/daxim/local/lib/perl5/site_perl/5.10.1/Git/CPAN/Patch/Import.pm line 13 Loading BackPAN index (this may take a while) Dynamic schema detected, will run in 0.04006 mode. Set the 'naming' attribute or the SCHEMA_LOADER_BACKCOMPAT environment variable to disable this warning. Also consider setting 'use_namespaces => 1' if/when upgrading. See perldoc DBIx::Class::Schema::Loader::Manual::UpgradingFromV4 for more details. DBIx::Class::Storage::DBI::__ANON__(): DBI Exception: DBD::SQLite::st execute failed: constraint failed releases.cpanid may not be NULL [for Statement " INSERT INTO releases (path, dist, version, date, size, maturity, cpanid, distvname) VALUES (?, ?, ?, ?, ?, ?, ?, ? ) "] at /home/daxim/local/lib/perl5/site_perl/5.10.1/BackPAN/Index.pm line 172

Sat Apr 17 12:06:22 2010 yanick+cpan [...] babyl.dyndns.org - Taken

Sat Apr 17 12:07:02 2010 yanick+cpan [...] babyl.dyndns.org - Correspondence added

Reproduced on my machine: $ git backpan-init Format::Human::Bytes Loading BackPAN index (this may take a while) Dynamic schema detected, will run in 0.04006 mode. Set the 'naming' attribute or the SCHEMA_LOADER_BACKCOMPAT environment variable to disable this warning. Also consider setting 'use_namespaces => 1' if/when upgrading. See perldoc DBIx::Class::Schema::Loader::Manual::UpgradingFromV4 for more details. DBIx::Class::Storage::DBI::__ANON__(): DBI Exception: DBD::SQLite::st execute failed: constraint failed releases.cpanid may not be NULL [for Statement " INSERT INTO releases (path, dist, version, date, size, maturity, cpanid, distvname) VALUES (?, ?, ?, ?, ?, ?, ?, ? ) "] at /home/yanick/perl5/perlbrew/perls/perl-5.12.0/lib/site_perl/5.12.0/BackPAN/Index.pm line 172

Sat Apr 17 12:07:03 2010 The RT System itself - Status changed from 'new' to 'open'

Sat Apr 17 12:07:35 2010 yanick+cpan [...] babyl.dyndns.org - Severity Important added

Sat Apr 17 12:51:00 2010 The RT System itself - Owner forcibly changed from YANICK to Nobody

Sat Apr 17 12:51:00 2010 yanick+cpan [...] babyl.dyndns.org - Queue changed from Git-CPAN-Patch to BackPAN-Index

Sat Apr 17 12:58:40 2010 yanick+cpan [...] babyl.dyndns.org - Correspondence added

Ah, so I can punt tickets to other queues. Excellent. :-) Anyway, it seems that the latest version of a DBIx depedency of BackPAN::Index is causing it to crash and burn. I'll come up with a tentative patch real soon, stay tuned!

Sat Apr 17 13:14:17 2010 mschwern [...] cpan.org - Correspondence added

I have a fix for this already in the BackPAN::Index repo. http://github.com/acme/parse-backpan-packages/commit/6b3ce4f4484c41dc5f49ff25fb9629e7fc61fea9 I haven't really properly evaluated it so I didn't make a release. I probably should at least push out an alpha.

Sat Apr 17 13:39:58 2010 yanick+cpan [...] babyl.dyndns.org - Correspondence added

On Sat Apr 17 13:14:17 2010, MSCHWERN wrote: Show quoted text

> I have a fix for this already in the BackPAN::Index repo. > http://github.com/acme/parse-backpan- > packages/commit/6b3ce4f4484c41dc5f49ff25fb9629e7fc61fea9 > > I haven't really properly evaluated it so I didn't make a release. I > probably should at least push out an alpha.

Talk about quick response time! :-) I've d/l'ed a copy and started looking at it. It looks like there's still a problem with dists that don't have a cpanid. It means that in BackPAN::Index line 179 (I'll annotate the GitHub copy), the query can turn out to be rejected because cpanid can't be NULL. The part that I don't know is if we want to keep those entries (so have a ' $i->cpanid || ''), or just ignore then (so just wrap the whole thing in an eval{})

Sat Apr 17 13:42:45 2010 mschwern [...] cpan.org - Correspondence added

I've kicked out 0.39 which should solve the problem. Really BackPAN::Index should skip with a cached version of the schema and just depended on DBIx::Class. I haven't figured a way to do that yet.

Sat Apr 17 13:42:47 2010 mschwern [...] cpan.org - Status changed from 'open' to 'resolved'

Sat Apr 17 15:07:03 2010 schwern [...] pobox.com - Correspondence added

Subject:	Re: [rt.cpan.org #56665] backpan-init broken
Date:	Sat, 17 Apr 2010 22:06:44 +0300
To:	bug-BackPAN-Index [...] rt.cpan.org
From:	Michael G Schwern <schwern [...] pobox.com>

Yanick Champoux via RT wrote: Show quoted text

> On Sat Apr 17 13:14:17 2010, MSCHWERN wrote:

>> I have a fix for this already in the BackPAN::Index repo. >> http://github.com/acme/parse-backpan- >> packages/commit/6b3ce4f4484c41dc5f49ff25fb9629e7fc61fea9 >> >> I haven't really properly evaluated it so I didn't make a release. I >> probably should at least push out an alpha.

> > Talk about quick response time! :-) > > I've d/l'ed a copy and started looking at it. It looks like there's > still a problem with dists that don't have a cpanid.

Like all the junk in ports and src? Show quoted text

> It means that in > BackPAN::Index line 179 (I'll annotate the GitHub copy), the query can > turn out to be rejected because cpanid can't be NULL. The part that I > don't know is if we want to keep those entries (so have a ' $i->cpanid > || ''), or just ignore then (so just wrap the whole thing in an eval{})

I guess I always run it with the default, releases_only_from_authors, so I don't notice. PBP might not have that as a default. -- s7ank: i want to be one of those guys that types "s/j&jd//.^$ueu*///djsls/sm." and it's a perl script that turns dog crap into gold.

Sat Apr 17 15:07:04 2010 The RT System itself - Status changed from 'resolved' to 'open'

Sat Apr 17 17:32:51 2010 daxim [...] cpan.org - Correspondence added

Installed BackPAN-Index-0.39.tar.gz, but still fails: Show quoted text

> git backpan-init Format::Human::Bytes

Prototype mismatch: sub Git::CPAN::Patch::Import::chmod (@) vs none at /home/daxim/local/lib/perl5/5.10.1/Exporter.pm line 67. at /home/daxim/local/lib/perl5/site_perl/5.10.1/Git/CPAN/Patch/Import.pm line 13 Loading BackPAN index (this may take a while) DBIx::Class::Relationship::HasOne::might_have(): "might_have/has_one" must not be on columns with is_nullable set to true (BackPAN::Index::File/path). This might indicate an incorrect use of those relationship helpers instead of belongs_to. at /tmp/dbicGP7J/BackPAN/Index/File.pm line 59 DBIx::Class::Storage::DBI::__ANON__(): DBI Exception: DBD::SQLite::st execute failed: constraint failed releases.cpanid may not be NULL [for Statement " INSERT INTO releases (path, dist, version, date, size, maturity, cpanid, distvname) VALUES (?, ?, ?, ?, ?, ?, ?, ? ) "] at /home/daxim/local/lib/perl5/site_perl/5.10.1/BackPAN/Index.pm line 172

Thu Jun 24 22:36:27 2010 mschwern [...] cpan.org - Correspondence added

On Sat Apr 17 17:32:51 2010, DAXIM wrote: Show quoted text

> DBIx::Class::Storage::DBI::__ANON__(): DBI Exception: DBD::SQLite::st > execute failed: constraint failed > releases.cpanid may not be NULL [for Statement " > INSERT INTO releases > (path, dist, version, date, size, maturity, cpanid, > distvname) > VALUES (?, ?, ?, ?, ?, ?, ?, ? > ) > "] at > /home/daxim/local/lib/perl5/site_perl/5.10.1/BackPAN/Index.pm line 172

The latest version out of the repository fixes that. DBIx::Class changed out from under us and now requires foreign keys to be NOT NULL.

Mon Dec 31 12:54:19 2012 mschwern [...] cpan.org - Correspondence added

Ping? Has this been resolved? I'm not even sure who owns git-backpan-index any more.

Mon Dec 31 12:54:20 2012 mschwern [...] cpan.org - Status changed from 'open' to 'stalled'

Mon Nov 10 19:23:27 2014 mschwern [...] cpan.org - Correspondence added

No reply in two years, calling it resolved.

Mon Nov 10 19:23:27 2014 The RT System itself - Status changed from 'stalled' to 'open'

Mon Nov 10 19:23:29 2014 mschwern [...] cpan.org - Status changed from 'open' to 'resolved'

Mon Nov 10 19:23:30 2014 mschwern [...] cpan.org - Fixed in 0.39 added