Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the PPIx-EditorTools CPAN distribution.

Report information
The Basics
Id: 48277
Status: resolved
Priority: 0/
Queue: PPIx-EditorTools

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

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



Subject: Algorithm::Diff should be listed as a dependency.
Padre Team, Thanks for all of your work on Padre. It is now my editor of choice for Perl. Anyway, I thought you might want to know that Algorithm::Diff should be listed as a dependency. Thanks, Christopher Bottoms
Subject: Re: [Padre-dev] Fwd: [rt.cpan.org #48277] Algorithm::Diff should be listed as a dependency.
Date: Fri, 31 Jul 2009 14:11:27 -0400
To: Padre development discussion list <padre-dev [...] perlide.org>, bug-PPIx-EditorTools [...] rt.cpan.org
From: Mark Grimes <mgrimes [...] cpan.org>
Hmmm.... I'm not sure what's causing the failure. Algorithm::Diff is already an eventual prereq of Test::Differences which is listed via build_requires in the Makefile.PL: PPIx::EditorTools build_requires: - Test::Differences requires: - Text::Diff requires: - Algorithm::Diff This is hierarchy exists in all the version of all of these modules that I could find on CPAN. As far as I can remember (which is confirmed by a quick check with Devel::Hide) there is no need for Algorithm::Diff except during the tests, so it shouldn't need to be changed to a "requires". I would be willing to add Algorithm::Diff as a direct requirement, but it seems unnecessary unless there is something I am missing. I did see in the irc log that someone else ran into this problem but there are no FAIL reports from cpantesters. Is there some env variable or Module::Install config flag that might cause it not to install "build" requirements? Seems unlikely, but I'm a Module::Build guy--M::I and MakeMaker have always seemed a bit quirky to me. -Mark On Tue, Jul 28, 2009 at 11:31 PM, Gabor Szabo <szabgab@gmail.com> wrote: Show quoted text
> Can some look into this (again) please ? > > Gabor > > > ---------- Forwarded message ---------- > From: Christopher Bottoms via RT <bug-PPIx-EditorTools@rt.cpan.org> > Date: Tue, Jul 28, 2009 at 9:30 PM > Subject: [rt.cpan.org #48277] Algorithm::Diff should be listed as a > dependency. > To: > > > Tue Jul 28 14:30:33 2009: Request 48277 was acted upon. > Transaction: Ticket created by MOLECULES > Queue: PPIx-EditorTools > Subject: Algorithm::Diff should be listed as a dependency. > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: MOLECULES@cpan.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=48277 > > > > Padre Team, > Thanks for all of your work on Padre. It is now my editor of choice for > Perl. > > Anyway, I thought you might want to know that Algorithm::Diff should be > listed as a dependency. > > Thanks, > Christopher Bottoms > _______________________________________________ > Padre-dev mailing list > Padre-dev@perlide.org > http://mail.perlide.org/mailman/listinfo/padre-dev >
Since Algorithm::Diff is a prereq of Test::Differences which is in build_requires, I don't believe this is necessary. Closing.