Skip Menu |

This queue is for tickets about the CPAN CPAN distribution.

Report information
The Basics
Id: 68835
Status: open
Priority: 0/
Queue: CPAN
People
Owner: Nobody in particular
Requestors: perl [...] galumph.com
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 1.9600
Fixed in: (no value)

History Show all quoted text
  Wed Jun 15 16:34:07 2011 perl [...] galumph.com - Ticket created
CC:
Subject: Install fails if the distribution has a META.json which specifies a configure_requires value and sets dynamic_config to true.
Date: Wed, 15 Jun 2011 20:33:58 +0000 (GMT)
To: bug-CPAN [...] rt.cpan.org
From: perl [...] galumph.com
Distribution has a Build.PL which uses non-core modules. The modules /are/ specified in configure_requires, but the modules don't get installed, so compilation of Build.PL fails. The problem appears to be with CPAN::Distribution::configure_requires(). pick_meta_file() finds the META.json. The condition in configure_requires() calls read_meta(), which correctly parses the JSON, but then discovers that dynamic_config is true and so does not return the meta object. Back in configure_requires(), the else clause kicks in, which then attempts parse_meta_yml() on the META.json file, which obviously fails, and thus configure_requires returns no dependencies. I'm assuming that read_meta() shouldn't be changed to return the object regardless of whether dynamic_config is true, so configure_requires() should be changed to not assume YAML if read_meta() doesn't return an object.
  Mon Jun 20 07:41:03 2011 perl [...] galumph.com - Correspondence added
Subject: Re: [rt.cpan.org #68835] AutoReply: Install fails if the distribution has a META.json which specifies a configure_requires value and sets dynamic_config to true.
Date: Mon, 20 Jun 2011 06:40:51 -0500
To: bug-CPAN [...] rt.cpan.org
From: Elliot Shank <perl [...] galumph.com>
I don't consider the attached patch to be a proper fix, but just a "solves my problem" hack. But it does work, considering that I don't understand most of what is going on.

Message body is not shown because sender requested not to inline it.

  Fri Jun 24 17:18:51 2011 CHORNY [...] cpan.org - Severity Important added
  Fri Jun 24 17:18:52 2011 CHORNY [...] cpan.org - Broken in 1.9600 added
  Fri Jun 24 20:44:01 2011 dagolden [...] cpan.org - Correspondence added
Good catch. That needs to be fixed quickly.
  Fri Jun 24 20:44:02 2011 The RT System itself - Status changed from 'new' to 'open'
  Fri Jun 24 21:17:03 2011 dagolden [...] cpan.org - Correspondence added
Made changes in my repo and will send Andreas a pull request
  Wed Jun 29 07:25:38 2011 ANDK [...] cpan.org - Correspondence added
Should be fixed in ANDK/CPAN-1.97_51.tar.gz
  Wed Jul 06 19:57:38 2011 perl [...] galumph.com - Correspondence added
Subject: Re: [rt.cpan.org #68835] Install fails if the distribution has a META.json which specifies a configure_requires value and sets dynamic_config to true.
Date: Wed, 06 Jul 2011 18:57:23 -0500
To: bug-CPAN [...] rt.cpan.org
From: Elliot Shank <perl [...] galumph.com>
On 6/29/11 6:25 AM, Andreas Koenig via RT wrote: Show quoted text
> Should be fixed in ANDK/CPAN-1.97_51.tar.gz
Fixes my issue.
  Tue Jul 26 11:58:52 2011 email [...] froggs.de - Correspondence added
Am Do 07. Jul 2011, 01:57:38, clonezone schrieb: Show quoted text
> On 6/29/11 6:25 AM, Andreas Koenig via RT wrote:
> > Should be fixed in ANDK/CPAN-1.97_51.tar.gz
> > Fixes my issue.
Had the same issue here. This dev release fixed it. When will this be released as non-dev? It affects many modules using Module::Build 0.38. Cheers, Tobias
  Fri Sep 09 06:34:08 2011 email [...] froggs.de - Correspondence added
Looks like it is still broken in v1.9800