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)



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
Download (untitled) / with headers
text/plain 881b
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.
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>
Download (untitled) / with headers
text/plain 174b
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.

Good catch. That needs to be fixed quickly.
Made changes in my repo and will send Andreas a pull request
Should be fixed in ANDK/CPAN-1.97_51.tar.gz
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>
Download (untitled) / with headers
text/plain 112b
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.
Download (untitled) / with headers
text/plain 327b
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
Looks like it is still broken in v1.9800


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.