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.
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Type: text/plain
Charset: utf8
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 217
Download (untitled) / with headers
text/plain 217b
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
MIME-Version: 1.0
X-Spam-Status: No, hits=0.0 required=8.0 tests=DK_SIGNED,HTML_MESSAGE,SPF_PASS
In-Reply-To: <d8a74af10907282331t6bb0b262re4a269f7951a2be7 [...] mail.gmail.com>
References: <RT-Ticket-48277 [...] rt.cpan.org> <rt-3.6.HEAD-14916-1248805834-792.48277-4-0 [...] rt.cpan.org> <d8a74af10907282331t6bb0b262re4a269f7951a2be7 [...] mail.gmail.com>
X-Virus-Checked: Checked by ClamAV on 16.mx.develooper.com
Message-ID: <7c99e1970907311111q758b3083r18650d73cd836a82 [...] mail.gmail.com>
Content-Type: multipart/alternative; boundary=000e0cd2e7544ad3ee0470045980
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by diesel.bestpractical.com (Postfix) with SMTP id 648F019B827E for <bug-PPIx-EditorTools [...] rt.cpan.org>; Fri, 31 Jul 2009 14:11:36 -0400 (EDT)
Received: (qmail 12286 invoked by uid 103); 31 Jul 2009 18:11:35 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 31 Jul 2009 18:11:35 -0000
Received: from mail-yw0-f182.google.com (HELO mail-yw0-f182.google.com) (209.85.211.182) by 16.mx.develooper.com (qpsmtpd/0.80) with ESMTP; Fri, 31 Jul 2009 11:11:30 -0700
Received: by ywh12 with SMTP id 12so1998475ywh.12 for <bug-PPIx-EditorTools [...] rt.cpan.org>; Fri, 31 Jul 2009 11:11:27 -0700 (PDT)
Received: by 10.150.215.16 with SMTP id n16mr3821781ybg.279.1249063887098; Fri, 31 Jul 2009 11:11:27 -0700 (PDT)
Delivered-To: cpan-bug+PPIx-EditorTools [...] diesel.bestpractical.com
Subject: Re: [Padre-dev] Fwd: [rt.cpan.org #48277] Algorithm::Diff should be listed as a dependency.
Domainkey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:content-type; b=v9sIJGnXppL5hE6Ptkc3fPkZMa2o7jwQ62+Sy4g9BXptnqtLbmi+cRBXmzJ3ZgBZi3 5EJLdt+XiRu1B83YIOjzIXelvoCEz0bUgN5y/RdNyqazs+uFmuVT6EwK5xCjTgl9RsuP XVpXfI6nXiSY2SOs2x1uvf4/IG4d+ahMaA5ik=
Return-Path: <mvgrimes [...] gmail.com>
Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:sender:received:in-reply-to :references:date:x-google-sender-auth:message-id:subject:from:to :content-type; bh=Iv4+1cMz9MBnGxIf487dmufOSIFyjD8PTxGSvr/xpQU=; b=Pyv6wSOwE7HTnMkqYbovQwJHBITxsPwwXVlGW3H1U5K6Z5wXGTsYQru9IjO/cfr/na 86YXNiNsYlfByx6ApLZx6KqSZbqJYPOflEYCB38Tp9zYvWi74FzaqG8n2uD/8cN2M9ka mPSnqec5Km2w5wM+DANvlQNG07sVhBHyuNfjQ=
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: bug-PPIx-EditorTools [...] rt.cpan.org
X-Google-Sender-Auth: 7b9a297a89ead346
Date: Fri, 31 Jul 2009 14:11:27 -0400
Sender: mvgrimes [...] gmail.com
X-Spam-Level: *
To: Padre development discussion list <padre-dev [...] perlide.org>, bug-PPIx-EditorTools [...] rt.cpan.org
From: Mark Grimes <mgrimes [...] cpan.org>
RT-Message-ID: <rt-3.6.HEAD-14916-1249063915-796.48277-0-0 [...] rt.cpan.org>
Content-Length: 0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
X-RT-Original-Encoding: utf-8
Content-Length: 2216
Download (untitled) / with headers
text/plain 2.1k
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 >
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
X-RT-Original-Encoding: utf-8
Content-Length: 3391
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Charset: utf8
Content-Type: text/plain
Message-ID: <rt-3.6.HEAD-9877-1260464222-1094.48277-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 126
Download (untitled) / with headers
text/plain 126b
Since Algorithm::Diff is a prereq of Test::Differences which is in build_requires, I don't believe this is necessary. Closing.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.