Skip Menu |

This queue is for tickets about the CPAN-Meta-Check CPAN distribution.

Report information
The Basics
Id: 83461
Status: rejected
Priority: 0/
Queue: CPAN-Meta-Check
People
Owner: Nobody in particular
Requestors: TODDR [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 0.004
Fixed in: (no value)

History Show all quoted text
  Tue Feb 19 20:30:35 2013 TODDR [...] cpan.org - Ticket created
Subject: Missing build_requires modules
I realize there are some circular deps that may make this impossible, but This module is missing the following build_requires deps: CPAN::Meta::Requirements: 0 Parse::CPAN::Meta: 1.4404
  Tue Feb 19 20:49:17 2013 LEONT [...] cpan.org - Correspondence added
Subject: Re: [rt.cpan.org #83461] Missing build_requires modules
Date: Wed, 20 Feb 2013 02:48:42 +0100
To: bug-CPAN-Meta-Check [...] rt.cpan.org
From: Leon Timmermans <leont [...] cpan.org>
On Wed, Feb 20, 2013 at 2:30 AM, Todd Rinaldo via RT <bug-CPAN-Meta-Check@rt.cpan.org> wrote: Show quoted text
> I realize there are some circular deps that may make this impossible, but This module is missing > the following build_requires deps: > > CPAN::Meta::Requirements: 0 > Parse::CPAN::Meta: 1.4404
CPAN::Meta::Requirements is already stated as a runtime-requirement (at a higher version than zero), so adding it to build-time is not necessary. I don't see why a direct dependency on Parse::CPAN::Meta 1.4404 would be required though, could you explain yourself? Leon
  Tue Feb 19 20:49:18 2013 The RT System itself - Status changed from 'new' to 'open'
  Tue Feb 19 20:56:23 2013 TODDR [...] cpan.org - Correspondence added
RT-Send-CC: leont [...] cpan.org
On Tue Feb 19 20:49:17 2013, LEONT wrote: Show quoted text
> CPAN::Meta::Requirements is already stated as a runtime-requirement > (at a higher version than zero), so adding it to build-time is not > necessary. I don't see why a direct dependency on Parse::CPAN::Meta > 1.4404 would be required though, could you explain yourself?
Sure. I'm parsing MYMETA to assert RPM BuildRequires and Requires for these. Because it's a run-time requirement, it doesn't show up and the tests during the RPM build fail. Right now I'm customizing the RPM to work around this by injecting additional deps in addition to what MYMETA specifies. I usually report the missing deps when I find them. If you need more details, I'll need a few days. It's been a while since I fixed this originally. Todd
  Tue Feb 19 21:09:07 2013 LEONT [...] cpan.org - Correspondence added
Subject: Re: [rt.cpan.org #83461] Missing build_requires modules
Date: Wed, 20 Feb 2013 03:08:37 +0100
To: bug-CPAN-Meta-Check [...] rt.cpan.org
From: Leon Timmermans <leont [...] cpan.org>
On Wed, Feb 20, 2013 at 2:56 AM, Todd Rinaldo via RT <bug-CPAN-Meta-Check@rt.cpan.org> wrote: Show quoted text
> Sure. I'm parsing MYMETA to assert RPM BuildRequires and Requires for these. Because it's a > run-time requirement, it doesn't show up and the tests during the RPM build fail.
The Meta 2.0 spec clearly states that runtime dependencies should be present at build time[1], this sounds like a bug in your rpm toolchain Show quoted text
> Right now I'm > customizing the RPM to work around this by injecting additional deps in addition to what > MYMETA specifies. I usually report the missing deps when I find them. If you need more details, > I'll need a few days. It's been a while since I fixed this originally.
I'm not depending on Parse::CPAN::Meta directly, but I do use CPAN::Meta 2.120920 which requires Parse::CPAN::Meta 1.4403. Leon [1]: https://metacpan.org/module/CPAN::Meta::Spec#Phases
  Tue Feb 19 21:11:00 2013 TODDR [...] cpan.org - Correspondence added
On Tue Feb 19 21:09:07 2013, LEONT wrote: Show quoted text
> On Wed, Feb 20, 2013 at 2:56 AM, Todd Rinaldo via RT > <bug-CPAN-Meta-Check@rt.cpan.org> wrote:
> > Sure. I'm parsing MYMETA to assert RPM BuildRequires and Requires
> for these. Because it's a
> > run-time requirement, it doesn't show up and the tests during the
> RPM build fail. > > The Meta 2.0 spec clearly states that runtime dependencies should be > present at build time[1], this sounds like a bug in your rpm toolchain >
> > Right now I'm > > customizing the RPM to work around this by injecting additional deps
> in addition to what
> > MYMETA specifies. I usually report the missing deps when I find
> them. If you need more details,
> > I'll need a few days. It's been a while since I fixed this
> originally. > > I'm not depending on Parse::CPAN::Meta directly, but I do use > CPAN::Meta 2.120920 which requires Parse::CPAN::Meta 1.4403.
If what you're saying is true, then my toolchain should indeed be handling this. I'll investigate this further and get back to you in a week. Maybe I'm reporting the dep issue to the wrong module...
  Tue Mar 19 14:23:38 2013 LEONT [...] cpan.org - Correspondence added
On Tue Feb 19 21:11:00 2013, TODDR wrote: Show quoted text
> > The Meta 2.0 spec clearly states that runtime dependencies should be > > present at build time[1], this sounds like a bug in your rpm
> toolchain
> >
> If what you're saying is true, then my toolchain should indeed be > handling this. I'll investigate > this further and get back to you in a week. Maybe I'm reporting the > dep issue to the wrong > module...
I'm closing this ticket, as AFAIK it's not a bug in my module. Leon
  Tue Mar 19 14:23:39 2013 LEONT [...] cpan.org - Status changed from 'open' to 'rejected'