Bug #98243 for Module-Install-RTx: x_requires_rt_plugins only lists last required RT plugin

Fri Aug 22 04:27:34 2014 cloos [...] netcologne.de - Ticket created

Subject:	x_requires_rt_plugins only lists last required RT plugin
Date:	Fri, 22 Aug 2014 10:27:22 +0200
To:	bug-Module-Install-RTx [...] rt.cpan.org
From:	Christian Loos <cloos [...] netcologne.de>

Hi, x_requires_rt_plugins in META.yml only lists the last mentioned required RT extension. See: https://github.com/bestpractical/rt-extension-ldapimport-multiemail/blob/master/META.yml By the way, there are also some pod errors: https://github.com/bestpractical/module-install-rtx/blob/master/README#L107 Chris

Sun Aug 24 20:29:50 2014 falcone [...] bestpractical.com - Correspondence added

Subject:	Re: [rt.cpan.org #98243] x_requires_rt_plugins only lists last required RT plugin
Date:	Sun, 24 Aug 2014 19:16:41 -0400
To:	"cloos [...] netcologne.de via RT" <bug-Module-Install-RTx [...] rt.cpan.org>
From:	Kevin Falcone <falcone [...] bestpractical.com>

On Fri, Aug 22, 2014 at 04:27:35AM -0400, cloos@netcologne.de via RT wrote: Show quoted text

> x_requires_rt_plugins in META.yml only lists the last mentioned required > RT extension. > See: > https://github.com/bestpractical/rt-extension-ldapimport-multiemail/blob/master/META.yml

Looks like we weren't reaching deep enough into the data structure. I've committed a fix and we'll see if the original author of the feature sees anything wrong with it and cut a new release. Show quoted text

> By the way, there are also some pod errors: > https://github.com/bestpractical/module-install-rtx/blob/master/README#L107

This was an easy fix (and my fault) pushing a fix. Thanks -kevin

Sun Aug 24 20:29:51 2014 The RT System itself - Status changed from 'new' to 'open'

Tue Sep 02 19:43:57 2014 alexmv [...] bestpractical.com - Correspondence added

Subject:	Re: [rt.cpan.org #98243] x_requires_rt_plugins only lists last required RT plugin
Date:	Tue, 02 Sep 2014 19:43:54 -0400
To:	bug-Module-Install-RTx [...] rt.cpan.org
From:	Alex Vandiver <alexmv [...] bestpractical.com>

On 08/24/2014 08:29 PM, falcone@bestpractical.com via RT wrote: Show quoted text

> On Fri, Aug 22, 2014 at 04:27:35AM -0400, cloos@netcologne.de via RT wrote:

>> x_requires_rt_plugins in META.yml only lists the last mentioned required >> RT extension. >> See: >> https://github.com/bestpractical/rt-extension-ldapimport-multiemail/blob/master/META.yml

> > Looks like we weren't reaching deep enough into the data structure. > I've committed a fix and we'll see if the original author of the > feature sees anything wrong with it and cut a new release.

I've pushed 0.36 with this change. - Alex

Mon Feb 16 22:19:54 2015 cpan [...] chmrr.net - Status changed from 'open' to 'resolved'