Skip Menu |

This queue is for tickets about the Dist-Zilla-PluginBundle-MARCEL CPAN distribution.

Report information
The Basics
Id: 114748
Status: resolved
Priority: 0/
Queue: Dist-Zilla-PluginBundle-MARCEL
People
Owner: dolmen [...] cpan.org
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Critical
Broken in: 1.161470
Fixed in: 1.161760

History Show all quoted text
  Thu May 26 16:55:57 2016 SREZIC [...] cpan.org - Ticket created
Subject: Recursive dependency detected
CPAN.pm cannot install the distribution due to a dependency cycle: Recursive dependency detected: Dist::Zilla::Plugin::InlineFilesMARCEL (have: N/A; want: 1.161470) => DOLMEN/Dist-Zilla-PluginBundle-MARCEL-1.161470.tar.gz => Dist::Zilla::Plugin::InlineFilesMARCEL (have: N/A; requires: 0). Cannot resolve. DOLMEN/Dist-Zilla-PluginBundle-MARCEL-1.161470.tar.gz [depend] -- NOT OK
  Fri May 27 16:34:00 2016 dolmen [...] cpan.org - Correspondence added
Le 2016-05-26 22:55:57, SREZIC a écrit :
Show quoted text
> CPAN.pm cannot install the distribution due to a dependency cycle:

Thanks for the report!

I'll use the [RemovePrereqs::Provided] dzil plugin.
https://metacpan.org/pod/Dist::Zilla::Plugin::RemovePrereqs::Provided

-- 
Olivier Mengué - http://perlresume.org/DOLMEN
  Fri May 27 16:34:00 2016 The RT System itself - Status changed from 'new' to 'open'
  Tue Jun 21 12:26:13 2016 dolmen [...] cpan.org - Severity Critical added
  Tue Jun 21 16:18:21 2016 ether [...] cpan.org - Correspondence added
On 2016-05-27 13:34:00, DOLMEN wrote: Show quoted text
> Le 2016-05-26 22:55:57, SREZIC a écrit :
> > CPAN.pm cannot install the distribution due to a dependency cycle:
> > Thanks for the report! > > I'll use the [RemovePrereqs::Provided] dzil plugin. > https://metacpan.org/pod/Dist::Zilla::Plugin::RemovePrereqs::Provided > > -- > Olivier Mengué - http://perlresume.org/DOLMEN
This is bizarre. What is populating the prerequisite list? [AutoPrereqs] doesn't add prereqs for files in the distribution. You should add [CheckSelfDependency] to your bundle to catch these errors in the future :) (PS. please don't strip dist.ini from the uploaded distribution on CPAN - it makes it harder to diagnose issues, and it's also impossible to use grep.cpan.me to find users of particular plugins or configurations.)
  Thu Jun 23 04:38:08 2016 dolmen [...] cpan.org - Correspondence added
Le 2016-06-21 22:18:21, ETHER a écrit :
Show quoted text
> This is bizarre. What is populating the prerequisite list?
> [AutoPrereqs] doesn't add prereqs for files in the distribution.

Dist::Zilla::Role::BundleDeps is the one that injects the circular dependency.

Show quoted text
> You should add [CheckSelfDependency] to your bundle to catch these
> errors in the future :)

If I add [RemovePrereqs::Provided] that should be enough.
If I add both I will have to ensure the order is correct. Just more nightmares to come.

Show quoted text
> (PS. please don't strip dist.ini from the uploaded distribution on
> CPAN - it makes it harder to diagnose issues, and it's also impossible
> to use grep.cpan.me to find users of particular plugins or
> configurations.)

This is from MARCEL's original code that I was just maintaining, and only recently modernized.
So far I just tried to preserve existing behaviour, even if I disagree. Like you, I think that removing dist.ini from the distribution is not a good thing.

-- 
Olivier Mengué - http://perlresume.org/DOLMEN
  Fri Jun 24 03:50:10 2016 dolmen [...] cpan.org - Status changed from 'open' to 'resolved'
  Fri Jun 24 03:50:15 2016 dolmen [...] cpan.org - Taken
  Fri Jun 24 03:50:15 2016 dolmen [...] cpan.org - Fixed in 1.161470 added
  Fri Jun 24 03:50:47 2016 dolmen [...] cpan.org - Fixed in 1.161470 deleted
  Fri Jun 24 08:42:53 2016 dolmen [...] cpan.org - Fixed in 1.161760 added