Skip Menu |

This queue is for tickets about the Module-Build-Tiny CPAN distribution.

Report information
The Basics
Id: 88292
Status: resolved
Priority: 0/
Queue: Module-Build-Tiny

People
Owner: Nobody in particular
Requestors: CHORNY [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in: 0.026
Fixed in: (no value)



Subject: MYMETA has incorrect dynamic_config
dynamic_config is copied to MYMETA from META. This is incorrect. It should be set to 0. -- Alexandr Ciornii, http://chorny.net
Instead of setting dynamic_config to 0 in MYMETA, it should probably die dynamic_config is 1. MBT requires a static dependency list, so dynamic_config should always be 0.
On Wed Sep 04 23:12:58 2013, haarg wrote: Show quoted text
> Instead of setting dynamic_config to 0 in MYMETA, it should probably > die dynamic_config is 1. MBT requires a static dependency list, so > dynamic_config should always be 0.
I agree that's a more sensible way to handle it given the way things function now, but I may add dynamic configuration in the near future so the point could be moot. Leon
On Fri Oct 04 12:12:57 2013, LEONT wrote: Show quoted text
> On Wed Sep 04 23:12:58 2013, haarg wrote:
> > Instead of setting dynamic_config to 0 in MYMETA, it should probably > > die dynamic_config is 1. MBT requires a static dependency list, so > > dynamic_config should always be 0.
> > I agree that's a more sensible way to handle it given the way things > function now, but I may add dynamic configuration in the near future > so the point could be moot. > > Leon
As all author toolchains are outputting dynamic_config=0 already, I'm closing this as "fixed elsewhere". Leon
On Tue Aug 12 08:43:02 2014, LEONT wrote: Show quoted text
> As all author toolchains are outputting dynamic_config=0 already, I'm > closing this as "fixed elsewhere".
This means that for module that was not updated and has dynamic_config=0, it will still fail. -- Alexandr Ciornii, http://chorny.net