Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Perl-Critic CPAN distribution.

Report information
The Basics
Id: 43908
Status: open
Priority: 0/
Queue: Perl-Critic

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

Bug Information
Severity: Unimportant
Broken in: 1.097_002
Fixed in: (no value)



Subject: Add MANIFEST to the repository
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: 472
Download (untitled) / with headers
text/plain 472b
The MANIFEST is missing from the repository. I'm sure this is deliberate so I didn't just go ahead and commit it. Normally this would be a religious argument as to whether a generated file should be committed, but it's a static generated file (ie. its the same for everyone) and there's tests that don't work without it (00_modules, 00_config and 05_utils). So from a purely practical view it would be very nice if it were there so things work straight from a checkout.
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-2264-1236365178-1545.43908-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 377
Download (untitled) / with headers
text/plain 377b
Case in point, I did a "make manifest" and now 00_modules fails. t/00_modules..................................2213/2081 # Looks like you planned 2081 tests but ran 2214. t/00_modules.................................. Dubious, test returned 255 (wstat 65280, 0xff00) All 2081 subtests passed Did the MANIFEST get generated wrong? Did it pull in too many files? Who knows?
MIME-Version: 1.0
X-Spam-Status: No, hits=-0.0 required=8.0 tests=SPF_PASS
In-Reply-To: <rt-3.6.HEAD-2264-1236365178-1545.43908-5-0 [...] rt.cpan.org>
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.6.HEAD-2264-1236365178-1545.43908-5-0 [...] rt.cpan.org>
Message-ID: <49B1C47E.4040308 [...] galumph.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
X-RT-Original-Encoding: utf-8
Received: from la.mx.develooper.com (x1.develooper.com [63.251.223.170]) by diesel.bestpractical.com (Postfix) with SMTP id E430B19B817D for <bug-Perl-Critic [...] rt.cpan.org>; Fri, 6 Mar 2009 19:49:12 -0500 (EST)
Received: (qmail 3713 invoked by uid 103); 7 Mar 2009 00:49:12 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 7 Mar 2009 00:49:12 -0000
Received: from pendrell.textdrive.com (HELO pendrell.textdrive.com) (207.7.108.149) by 16.mx.develooper.com (qpsmtpd/0.43rc1) with ESMTP; Fri, 06 Mar 2009 16:49:09 -0800
Received: from quaquaversal.local (unknown [75.3.128.71]) by pendrell.textdrive.com (Postfix) with ESMTP id D8D6EBB3CE for <bug-Perl-Critic [...] rt.cpan.org>; Sat, 7 Mar 2009 00:49:03 +0000 (GMT)
Delivered-To: cpan-bug+Perl-Critic [...] diesel.bestpractical.com
Subject: Re: [rt.cpan.org #43908] Add MANIFEST to the repository
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1.19) Gecko/20081209 Thunderbird/2.0.0.19 Mnenhy/0.7.5.0
Return-Path: <perl [...] galumph.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: bug-Perl-Critic [...] rt.cpan.org
Date: Fri, 06 Mar 2009 18:49:02 -0600
X-Spam-Level: *
To: bug-Perl-Critic [...] rt.cpan.org
Content-Transfer-Encoding: 7bit
From: Elliot Shank <perl [...] galumph.com>
RT-Message-ID: <rt-3.6.HEAD-2264-1236386957-1933.43908-0-0 [...] rt.cpan.org>
Content-Length: 233
Download (untitled) / with headers
text/plain 233b
Michael G Schwern via RT wrote: Show quoted text
> Did the MANIFEST get generated wrong? Did it pull in too many files? > Who knows?
We support EU::MM for install, but not for development. If you use M::B, the MANIFEST will be there when needed.
MIME-Version: 1.0
X-Spam-Status: No, hits=-0.0 required=8.0 tests=SPF_PASS
In-Reply-To: <rt-3.6.HEAD-2264-1236365007-1136.43908-4-0 [...] rt.cpan.org>
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.6.HEAD-2264-1236365007-1136.43908-4-0 [...] rt.cpan.org>
Message-ID: <49B1C4C0.6000108 [...] galumph.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
X-RT-Original-Encoding: utf-8
Received: from la.mx.develooper.com (x1.develooper.com [63.251.223.170]) by diesel.bestpractical.com (Postfix) with SMTP id 8B06919B817D for <bug-Perl-Critic [...] rt.cpan.org>; Fri, 6 Mar 2009 19:50:15 -0500 (EST)
Received: (qmail 3945 invoked by uid 103); 7 Mar 2009 00:50:15 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 7 Mar 2009 00:50:15 -0000
Received: from pendrell.textdrive.com (HELO pendrell.textdrive.com) (207.7.108.149) by 16.mx.develooper.com (qpsmtpd/0.43rc1) with ESMTP; Fri, 06 Mar 2009 16:50:12 -0800
Received: from quaquaversal.local (adsl-75-3-128-71.dsl.chcgil.sbcglobal.net [75.3.128.71]) by pendrell.textdrive.com (Postfix) with ESMTP id 89358BB3C6 for <bug-Perl-Critic [...] rt.cpan.org>; Sat, 7 Mar 2009 00:50:09 +0000 (GMT)
Delivered-To: cpan-bug+Perl-Critic [...] diesel.bestpractical.com
Subject: Re: [rt.cpan.org #43908] Add MANIFEST to the repository
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1.19) Gecko/20081209 Thunderbird/2.0.0.19 Mnenhy/0.7.5.0
Return-Path: <perl [...] galumph.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: bug-Perl-Critic [...] rt.cpan.org
Date: Fri, 06 Mar 2009 18:50:08 -0600
X-Spam-Level: *
To: bug-Perl-Critic [...] rt.cpan.org
Content-Transfer-Encoding: 7bit
From: Elliot Shank <perl [...] galumph.com>
RT-Message-ID: <rt-3.6.HEAD-2264-1236387018-557.43908-0-0 [...] rt.cpan.org>
Content-Length: 254
Download (untitled) / with headers
text/plain 254b
Michael G Schwern via RT wrote: Show quoted text
> The MANIFEST is missing from the repository. I'm sure this is > deliberate so I didn't just go ahead and commit it.
Yup. It's deliberate. I don't see the point of putting stuff into source control that is generated.
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-2264-1236420950-1962.43908-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 1501
Download (untitled) / with headers
text/plain 1.4k
I reran with Build.PL. 00_modules still failing. Still don't know why. t/00_modules..................................2212/2081 # Looks like you planned 2081 tests but ran 2214. t/00_modules.................................. Dubious, test returned 255 (wstat 65280, 0xff00) All 2081 subtests passed The reason you don't commit generated files is because typically they change per user's environment. Things like Makefiles and config files and the like. The MANIFEST remains exactly the same from computer to computer. It's supposed to be static. That it's generated is just a convenience. Regenerating the MANIFEST as part of the build process defeats the point of the MANIFEST. The MANIFEST exists to guarantee to the user that they got the complete source distribution. If the distribution regenerates the MANIFEST based on what they downloaded and unpacked this guarantee is lost. If any files are added to the distribution directory they may clog the MANIFEST. All of which leads to a non-static MANIFEST file and I'm left with a test that fails and I don't know why. This is not worth upholding the purity of not checking in any generated files. It's just one more thing that can go wrong. I also noticed MANIFEST is deleted as part of the clean target. This is a terrible idea as it wipes out part of the source distribution. The clean target should keep the distribution intact. After a user runs build clean they will be unable to check the integrity of their distribution.
MIME-Version: 1.0
X-Spam-Status: No, hits=-0.0 required=8.0 tests=SPF_PASS
In-Reply-To: <rt-3.6.HEAD-2264-1236420950-1962.43908-5-0 [...] rt.cpan.org>
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.6.HEAD-2264-1236420950-1962.43908-5-0 [...] rt.cpan.org>
Message-ID: <49B27FD3.8070502 [...] galumph.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
X-RT-Original-Encoding: utf-8
Received: from la.mx.develooper.com (x1.develooper.com [63.251.223.170]) by diesel.bestpractical.com (Postfix) with SMTP id D7A6A19B8177 for <bug-Perl-Critic [...] rt.cpan.org>; Sat, 7 Mar 2009 09:09:00 -0500 (EST)
Received: (qmail 8453 invoked by uid 103); 7 Mar 2009 14:08:59 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 7 Mar 2009 14:08:59 -0000
Received: from pendrell.textdrive.com (HELO pendrell.textdrive.com) (207.7.108.149) by 16.mx.develooper.com (qpsmtpd/0.43rc1) with ESMTP; Sat, 07 Mar 2009 06:08:57 -0800
Received: from quaquaversal.local (adsl-75-3-128-71.dsl.chcgil.sbcglobal.net [75.3.128.71]) by pendrell.textdrive.com (Postfix) with ESMTP id 92C2BBB753 for <bug-Perl-Critic [...] rt.cpan.org>; Sat, 7 Mar 2009 14:08:20 +0000 (GMT)
Delivered-To: cpan-bug+Perl-Critic [...] diesel.bestpractical.com
Subject: Re: [rt.cpan.org #43908] Add MANIFEST to the repository
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1.19) Gecko/20081209 Thunderbird/2.0.0.19 Mnenhy/0.7.5.0
Return-Path: <perl [...] galumph.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: bug-Perl-Critic [...] rt.cpan.org
Date: Sat, 07 Mar 2009 08:08:19 -0600
X-Spam-Level: *
To: bug-Perl-Critic [...] rt.cpan.org
Content-Transfer-Encoding: 7bit
From: Elliot Shank <perl [...] galumph.com>
RT-Message-ID: <rt-3.6.HEAD-2264-1236434944-1970.43908-0-0 [...] rt.cpan.org>
Content-Length: 997
Download (untitled) / with headers
text/plain 997b
Michael G Schwern via RT wrote: Show quoted text
> Regenerating the MANIFEST as part of the build process defeats the > point of the MANIFEST. The MANIFEST exists to guarantee to the user > that they got the complete source distribution. If the distribution > regenerates the MANIFEST based on what they downloaded and unpacked > this guarantee is lost.
The MANIFEST is included in the distribution tarball. It is already present at configuration time on users' machines and therefore any warnings about missing files will correctly appear. Show quoted text
> If any files are added to the distribution directory they may clog > the MANIFEST.
Not if MANIFEST.SKIP is correct. Show quoted text
> I also noticed MANIFEST is deleted as part of the clean target. This > is a terrible idea as it wipes out part of the source distribution.
The problem is that the automatic MANIFEST generation doesn't delete files that have been removed. I got sick and tired of having to delete the damn file by hand every time I wanted the file regenerated.
MIME-Version: 1.0
X-Spam-Status: No, hits=-0.0 required=8.0 tests=SPF_PASS
In-Reply-To: <rt-3.6.HEAD-2264-1236420950-1962.43908-5-0 [...] rt.cpan.org>
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.6.HEAD-2264-1236420950-1962.43908-5-0 [...] rt.cpan.org>
Message-ID: <49B281D8.3070806 [...] galumph.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
X-RT-Original-Encoding: utf-8
Received: from la.mx.develooper.com (x1.develooper.com [63.251.223.170]) by diesel.bestpractical.com (Postfix) with SMTP id 591F219B8177 for <bug-Perl-Critic [...] rt.cpan.org>; Sat, 7 Mar 2009 09:17:20 -0500 (EST)
Received: (qmail 10285 invoked by uid 103); 7 Mar 2009 14:17:19 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 7 Mar 2009 14:17:19 -0000
Received: from pendrell.textdrive.com (HELO pendrell.textdrive.com) (207.7.108.149) by 16.mx.develooper.com (qpsmtpd/0.43rc1) with ESMTP; Sat, 07 Mar 2009 06:17:16 -0800
Received: from quaquaversal.local (adsl-75-3-128-71.dsl.chcgil.sbcglobal.net [75.3.128.71]) by pendrell.textdrive.com (Postfix) with ESMTP id C2AF0BB346 for <bug-Perl-Critic [...] rt.cpan.org>; Sat, 7 Mar 2009 14:16:57 +0000 (GMT)
Delivered-To: cpan-bug+Perl-Critic [...] diesel.bestpractical.com
Subject: Re: [rt.cpan.org #43908] Add MANIFEST to the repository
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1.19) Gecko/20081209 Thunderbird/2.0.0.19 Mnenhy/0.7.5.0
Return-Path: <perl [...] galumph.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: bug-Perl-Critic [...] rt.cpan.org
Date: Sat, 07 Mar 2009 08:16:56 -0600
X-Spam-Level: *
To: bug-Perl-Critic [...] rt.cpan.org
Content-Transfer-Encoding: 7bit
From: Elliot Shank <perl [...] galumph.com>
RT-Message-ID: <rt-3.6.HEAD-2264-1236435444-507.43908-0-0 [...] rt.cpan.org>
Content-Length: 541
Download (untitled) / with headers
text/plain 541b
Michael G Schwern via RT wrote: Show quoted text
> Queue: Perl-Critic > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=43908 > > > I reran with Build.PL. 00_modules still failing. Still don't know why. > > t/00_modules..................................2212/2081 # Looks like you > planned 2081 tests but ran 2214. > t/00_modules.................................. Dubious, test returned > 255 (wstat 65280, 0xff00) > All 2081 subtests passed
Whoops. My bad on not updating the test count for is_enabled() testing on Policies. Fixed.
MIME-Version: 1.0
X-Spam-Status: No, hits=-0.0 required=8.0 tests=SPF_PASS
In-Reply-To: <rt-3.6.HEAD-2264-1236434944-1970.43908-6-0 [...] rt.cpan.org>
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.6.HEAD-2264-1236420950-1962.43908-5-0 [...] rt.cpan.org> <49B27FD3.8070502 [...] galumph.com> <rt-3.6.HEAD-2264-1236434944-1970.43908-6-0 [...] rt.cpan.org>
Message-ID: <49B2E0D9.9080502 [...] pobox.com>
Content-Type: text/plain; charset=UTF-8
X-RT-Original-Encoding: utf-8
Received: from la.mx.develooper.com (x1.develooper.com [63.251.223.170]) by diesel.bestpractical.com (Postfix) with SMTP id E039E19B809A for <bug-Perl-Critic [...] rt.cpan.org>; Sat, 7 Mar 2009 16:08:41 -0500 (EST)
Received: (qmail 14798 invoked by uid 103); 7 Mar 2009 21:08:41 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 7 Mar 2009 21:08:41 -0000
Received: from a-sasl-fastnet.sasl.smtp.pobox.com (HELO sasl.smtp.pobox.com) (207.106.133.19) by 16.mx.develooper.com (qpsmtpd/0.43rc1) with ESMTP; Sat, 07 Mar 2009 13:08:02 -0800
Received: from localhost.localdomain (unknown [127.0.0.1]) by a-sasl-fastnet.sasl.smtp.pobox.com (Postfix) with ESMTP id C80D89F128 for <bug-Perl-Critic [...] rt.cpan.org>; Sat, 7 Mar 2009 16:02:25 -0500 (EST)
Received: from [10.23.42.2] (unknown [69.64.236.3]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by a-sasl-fastnet.sasl.smtp.pobox.com (Postfix) with ESMTPSA id E7B329F127 for <bug-Perl-Critic [...] rt.cpan.org>; Sat, 7 Mar 2009 16:02:24 -0500 (EST)
Delivered-To: cpan-bug+Perl-Critic [...] diesel.bestpractical.com
Subject: Re: [rt.cpan.org #43908] Add MANIFEST to the repository
User-Agent: Thunderbird 2.0.0.19 (Macintosh/20081209)
Return-Path: <schwern [...] pobox.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: bug-Perl-Critic [...] rt.cpan.org
Date: Sat, 07 Mar 2009 13:02:17 -0800
X-Spam-Level: *
To: bug-Perl-Critic [...] rt.cpan.org
X-Enigmail-Version: 0.95.7
Content-Transfer-Encoding: 7bit
X-Pobox-Relay-ID: 42C6E1C4-0B5B-11DE-8344-CFA5EBB1AA3C-02258300!a-sasl-fastnet.pobox.com
From: Michael G Schwern <schwern [...] pobox.com>
RT-Message-ID: <rt-3.6.HEAD-2264-1236460131-59.43908-0-0 [...] rt.cpan.org>
Content-Length: 2424
Download (untitled) / with headers
text/plain 2.3k
Elliot Shank via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=43908 > > > Michael G Schwern via RT wrote:
>> Regenerating the MANIFEST as part of the build process defeats the >> point of the MANIFEST. The MANIFEST exists to guarantee to the user >> that they got the complete source distribution. If the distribution >> regenerates the MANIFEST based on what they downloaded and unpacked >> this guarantee is lost.
> > The MANIFEST is included in the distribution tarball. It is already present at configuration time on users' machines and therefore any warnings about missing files will correctly appear.
It will be untrusted after the first build and cannot be used to recheck the distribution (Build distcheck). Also since the tests use it it might cause tests to fail. Show quoted text
>> If any files are added to the distribution directory they may clog >> the MANIFEST.
> > Not if MANIFEST.SKIP is correct.
It can't predict all the possible junk that might wind up in there. For example, you're missing anything having to do with git. If I check out using git-svn, which I usually do, its going to wind up a mess. Which reminds me, it should be using #!include_default so it can take advantage of updates to the default skips... but it can't because its going to be regenerated on user's machines which might not have a new enough ExtUtils::Manifest to recognize #!include_default or their default MANIFEST.SKIP might be out of date. Plugging holes in the MANIFEST.SKIP is an expensive game to play to keep from checking in one file. Show quoted text
>> I also noticed MANIFEST is deleted as part of the clean target. This >> is a terrible idea as it wipes out part of the source distribution.
> > The problem is that the automatic MANIFEST generation doesn't delete files that have been removed. I got sick and tired of having to delete the damn file by hand every time I wanted the file regenerated.
I figured it was something like that. Simply customize the manifest target, and allllll this goes away. sub ACTION_manifest { my $self = shift; unlink 'MANIFEST' or die "Can't unlink MANIFEST: $!"; return $self->SUPER::ACTION_manifest(@_); } Poof! -- 87. If the thought of something makes me giggle for longer than 15 seconds, I am to assume that I am not allowed to do it. -- The 213 Things Skippy Is No Longer Allowed To Do In The U.S. Army http://skippyslist.com/list/
MIME-Version: 1.0
X-Spam-Status: No, hits=-0.0 required=8.0 tests=SPF_PASS
In-Reply-To: <rt-3.6.HEAD-2264-1236460131-59.43908-5-0 [...] rt.cpan.org>
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.6.HEAD-2264-1236420950-1962.43908-5-0 [...] rt.cpan.org> <49B27FD3.8070502 [...] galumph.com> <rt-3.6.HEAD-2264-1236434944-1970.43908-6-0 [...] rt.cpan.org> <49B2E0D9.9080502 [...] pobox.com> <rt-3.6.HEAD-2264-1236460131-59.43908-5-0 [...] rt.cpan.org>
Message-ID: <49B3C6F9.5020905 [...] galumph.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
X-RT-Original-Encoding: utf-8
Received: from la.mx.develooper.com (x1.develooper.com [63.251.223.170]) by diesel.bestpractical.com (Postfix) with SMTP id B7D5819B81E8 for <bug-Perl-Critic [...] rt.cpan.org>; Sun, 8 Mar 2009 09:24:40 -0400 (EDT)
Received: (qmail 23913 invoked by uid 103); 8 Mar 2009 13:24:39 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 8 Mar 2009 13:24:39 -0000
Received: from pendrell.textdrive.com (HELO pendrell.textdrive.com) (207.7.108.149) by 16.mx.develooper.com (qpsmtpd/0.43rc1) with ESMTP; Sun, 08 Mar 2009 06:24:36 -0700
Received: from quaquaversal.local (unknown [75.3.128.71]) by pendrell.textdrive.com (Postfix) with ESMTP id 31C9ABB2D8 for <bug-Perl-Critic [...] rt.cpan.org>; Sun, 8 Mar 2009 13:24:11 +0000 (GMT)
Delivered-To: cpan-bug+Perl-Critic [...] diesel.bestpractical.com
Subject: Re: [rt.cpan.org #43908] Add MANIFEST to the repository
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1.19) Gecko/20081209 Thunderbird/2.0.0.19 Mnenhy/0.7.5.0
Return-Path: <perl [...] galumph.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: bug-Perl-Critic [...] rt.cpan.org
Date: Sun, 08 Mar 2009 08:24:09 -0500
X-Spam-Level: *
To: bug-Perl-Critic [...] rt.cpan.org
Content-Transfer-Encoding: 7bit
From: Elliot Shank <perl [...] galumph.com>
RT-Message-ID: <rt-3.6.HEAD-2264-1236518684-1697.43908-0-0 [...] rt.cpan.org>
Content-Length: 262
Download (untitled) / with headers
text/plain 262b
Michael G Schwern via RT wrote: Show quoted text
> It can't predict all the possible junk that might wind up in there. > For example, you're missing anything having to do with git. If I > check out using git-svn, which I usually do, its going to wind up a > mess.
Git is evil.
MIME-Version: 1.0
X-Spam-Status: No, hits=-0.0 required=8.0 tests=SPF_PASS
In-Reply-To: <rt-3.6.HEAD-2264-1236460131-59.43908-5-0 [...] rt.cpan.org>
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.6.HEAD-2264-1236420950-1962.43908-5-0 [...] rt.cpan.org> <49B27FD3.8070502 [...] galumph.com> <rt-3.6.HEAD-2264-1236434944-1970.43908-6-0 [...] rt.cpan.org> <49B2E0D9.9080502 [...] pobox.com> <rt-3.6.HEAD-2264-1236460131-59.43908-5-0 [...] rt.cpan.org>
Message-ID: <49B3C8FC.4090808 [...] galumph.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
X-RT-Original-Encoding: utf-8
Received: from la.mx.develooper.com (x1.develooper.com [63.251.223.170]) by diesel.bestpractical.com (Postfix) with SMTP id 3E10C19B81E8 for <bug-Perl-Critic [...] rt.cpan.org>; Sun, 8 Mar 2009 09:33:08 -0400 (EDT)
Received: (qmail 27012 invoked by uid 103); 8 Mar 2009 13:33:07 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 8 Mar 2009 13:33:07 -0000
Received: from pendrell.textdrive.com (HELO pendrell.textdrive.com) (207.7.108.149) by 16.mx.develooper.com (qpsmtpd/0.43rc1) with ESMTP; Sun, 08 Mar 2009 06:33:04 -0700
Received: from quaquaversal.local (adsl-75-3-128-71.dsl.chcgil.sbcglobal.net [75.3.128.71]) by pendrell.textdrive.com (Postfix) with ESMTP id 897A1BB3CC for <bug-Perl-Critic [...] rt.cpan.org>; Sun, 8 Mar 2009 13:32:45 +0000 (GMT)
Delivered-To: cpan-bug+Perl-Critic [...] diesel.bestpractical.com
Subject: Re: [rt.cpan.org #43908] Add MANIFEST to the repository
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1.19) Gecko/20081209 Thunderbird/2.0.0.19 Mnenhy/0.7.5.0
Return-Path: <perl [...] galumph.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: bug-Perl-Critic [...] rt.cpan.org
Date: Sun, 08 Mar 2009 08:32:44 -0500
X-Spam-Level: *
To: bug-Perl-Critic [...] rt.cpan.org
Content-Transfer-Encoding: 7bit
From: Elliot Shank <perl [...] galumph.com>
RT-Message-ID: <rt-3.6.HEAD-2264-1236519191-1143.43908-0-0 [...] rt.cpan.org>
Content-Length: 236
Download (untitled) / with headers
text/plain 236b
Michael G Schwern via RT wrote: Show quoted text
> It will be untrusted after the first build and cannot be used to recheck the > distribution (Build distcheck).
If it gets regenerated, nothing is going to be missing, so there's no need for distcheck.
MIME-Version: 1.0
X-Spam-Status: No, hits=-0.0 required=8.0 tests=SPF_PASS
In-Reply-To: <rt-3.6.HEAD-2264-1236460131-59.43908-5-0 [...] rt.cpan.org>
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.6.HEAD-2264-1236420950-1962.43908-5-0 [...] rt.cpan.org> <49B27FD3.8070502 [...] galumph.com> <rt-3.6.HEAD-2264-1236434944-1970.43908-6-0 [...] rt.cpan.org> <49B2E0D9.9080502 [...] pobox.com> <rt-3.6.HEAD-2264-1236460131-59.43908-5-0 [...] rt.cpan.org>
Message-ID: <49B3CF56.4030306 [...] galumph.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
X-RT-Original-Encoding: utf-8
Received: from la.mx.develooper.com (x1.develooper.com [63.251.223.170]) by diesel.bestpractical.com (Postfix) with SMTP id BB9DB19B81E8 for <bug-Perl-Critic [...] rt.cpan.org>; Sun, 8 Mar 2009 10:00:17 -0400 (EDT)
Received: (qmail 3576 invoked by uid 103); 8 Mar 2009 14:00:16 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 8 Mar 2009 14:00:16 -0000
Received: from pendrell.textdrive.com (HELO pendrell.textdrive.com) (207.7.108.149) by 16.mx.develooper.com (qpsmtpd/0.43rc1) with ESMTP; Sun, 08 Mar 2009 07:00:13 -0700
Received: from quaquaversal.local (adsl-75-3-128-71.dsl.chcgil.sbcglobal.net [75.3.128.71]) by pendrell.textdrive.com (Postfix) with ESMTP id 7C634BB434 for <bug-Perl-Critic [...] rt.cpan.org>; Sun, 8 Mar 2009 13:59:52 +0000 (GMT)
Delivered-To: cpan-bug+Perl-Critic [...] diesel.bestpractical.com
Subject: Re: [rt.cpan.org #43908] Add MANIFEST to the repository
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1.19) Gecko/20081209 Thunderbird/2.0.0.19 Mnenhy/0.7.5.0
Return-Path: <perl [...] galumph.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: bug-Perl-Critic [...] rt.cpan.org
Date: Sun, 08 Mar 2009 08:59:50 -0500
X-Spam-Level: *
To: bug-Perl-Critic [...] rt.cpan.org
Content-Transfer-Encoding: 7bit
From: Elliot Shank <perl [...] galumph.com>
RT-Message-ID: <rt-3.6.HEAD-2264-1236520822-1723.43908-0-0 [...] rt.cpan.org>
Content-Length: 634
Download (untitled) / with headers
text/plain 634b
Michael G Schwern via RT wrote: Show quoted text
> Plugging holes in the MANIFEST.SKIP is an expensive game to play to > keep from checking in one file.
Not really. Show quoted text
>>> I also noticed MANIFEST is deleted as part of the clean target. >>> This is a terrible idea as it wipes out part of the source >>> distribution.
>> >> The problem is that the automatic MANIFEST generation doesn't >> delete files that have been removed. I got sick and tired of >> having to delete the damn file by hand every time I wanted the file >> regenerated.
> > I figured it was something like that.
Also, if something is being regenerated, you shouldn't keep it around.
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-10812-1330138467-341.43908-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 1293
Download (untitled) / with headers
text/plain 1.2k
I'm reopening this because I'm again working on Perl-Critic from the repository and, once again, the tests are failing because of the missing MANIFEST. Once I generate the MANIFEST tests are still failing and I can't tell if it's because the MANIFEST was generated incorrectly or if something else is wrong. Show quoted text
> Also, if something is being regenerated, you shouldn't keep it around.
That's garbled version control dogma. The actual rule is if something is being generated **differently on different machines** you shouldn't check it in. For example MakeMaker's Makefile. There's nothing wrong with checking in a generated file which is supposed to be the same for everybody (ex. MANIFEST and META.*). In fact it's beneficial. * Like any other file in the distribution, it is useful to track its contents at different points in the history for historical bug investigations. * It avoids mistakes resulting from differing build environments. Most importantly, it removes a barrier to new developers. Ease of use should trump version control dogma. As much as possible, the code should work straight out of the repository. I would request somebody other than Elliot in the Perl::Critic project respond please. His selective single line responses are dismissive and disrespectful.
From jeff [...] imaginative-software.com Fri Feb 24 23: 40:54 2012
MIME-Version: 1.0 (Apple Message framework v1084)
X-Spam-Status: No, score=-3.844 tagged_above=-99.9 required=10 tests=[AWL=-1.945, BAYES_00=-1.9, HTML_MESSAGE=0.001] autolearn=ham
In-Reply-To: <rt-3.8.HEAD-10812-1330138467-1959.43908-5-0 [...] rt.cpan.org>
X-Mailer: Apple Mail (2.1084)
X-Spam-Flag: NO
Received-SPF: pass (google.com: domain of jeff [...] imaginative-software.com designates 10.68.74.138 as permitted sender) client-ip=10.68.74.138;
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.8.HEAD-10812-1330138467-1959.43908-5-0 [...] rt.cpan.org>
X-Virus-Checked: Checked by ClamAV on 16.mx.develooper.com
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Content-Type: multipart/alternative; boundary=Apple-Mail-1--349674632
Message-ID: <6035D60E-0BA1-4D25-AD97-D90CADE30C18 [...] imaginative-software.com>
X-Spam-Score: -3.844
Authentication-Results: mr.google.com; spf=pass (google.com: domain of jeff [...] imaginative-software.com designates 10.68.74.138 as permitted sender) smtp.mail=jeff [...] imaginative-software.com
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id C99AF240026 for <cpan-bug+Perl-Critic [...] hipster.bestpractical.com>; Fri, 24 Feb 2012 23:40:54 -0500 (EST)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id hw+AUu9NiQdd for <cpan-bug+Perl-Critic [...] hipster.bestpractical.com>; Fri, 24 Feb 2012 23:40:53 -0500 (EST)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 2CA6424001E for <bug-Perl-Critic [...] rt.cpan.org>; Fri, 24 Feb 2012 23:40:53 -0500 (EST)
Received: (qmail 8379 invoked by uid 103); 25 Feb 2012 04:40:51 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 25 Feb 2012 04:40:51 -0000
Received: from mail-pz0-f50.google.com (HELO mail-pz0-f50.google.com) (209.85.210.50) by 16.mx.develooper.com (qpsmtpd/0.80/v0.80-19-gf52d165) with ESMTP; Fri, 24 Feb 2012 20:40:48 -0800
Received: by dald2 with SMTP id d2so3011472dal.9 for <bug-Perl-Critic [...] rt.cpan.org>; Fri, 24 Feb 2012 20:40:46 -0800 (PST)
Received: from mr.google.com ([10.68.74.138]) by 10.68.74.138 with SMTP id t10mr15185080pbv.126.1330144846266 (num_hops = 1); Fri, 24 Feb 2012 20:40:46 -0800 (PST)
Received: by 10.68.74.138 with SMTP id t10mr12518797pbv.126.1330144846219; Fri, 24 Feb 2012 20:40:46 -0800 (PST)
Received: from [192.168.1.3] (c-24-6-41-85.hsd1.ca.comcast.net. [24.6.41.85]) by mx.google.com with ESMTPS id g9sm6199402pba.6.2012.02.24.20.40.44 (version=TLSv1/SSLv3 cipher=OTHER); Fri, 24 Feb 2012 20:40:45 -0800 (PST)
Delivered-To: cpan-bug+Perl-Critic [...] hipster.bestpractical.com
Subject: Re: [rt.cpan.org #43908] Add MANIFEST to the repository
Return-Path: <jeff [...] imaginative-software.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: cpan-bug+Perl-Critic [...] hipster.bestpractical.com
X-RT-Mail-Extension: perl-critic
Date: Fri, 24 Feb 2012 20:40:43 -0800
X-Spam-Level:
To: bug-Perl-Critic [...] rt.cpan.org
From: Jeffrey Thalhammer <jeff [...] imaginative-software.com>
X-GM-Message-State: ALoCoQli88KvyTbKt/rflLuuApKnhLfCFBvNkoueFq8mzIdRjAUMIlZGkm3RbOx+n1Bi9V3eN7IQ
RT-Message-ID: <rt-3.8.HEAD-10813-1330144855-565.43908-0-0 [...] rt.cpan.org>
Content-Length: 0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
X-RT-Original-Encoding: utf-8
Content-Length: 797
Download (untitled) / with headers
text/plain 797b
On Feb 24, 2012, at 6:54 PM, Michael G Schwern via RT wrote: Show quoted text
> I would request somebody other than Elliot in the Perl::Critic project respond please.
I can see both sides of the argument. But I have been bitten by the lack of MANIFEST a few times recently (and I do use M::B rather than EU::MM, so I'm not sure why). So I'm ok with stashing the MANIFEST in the repository. Since Schwern is the only person actively working on Perl::Critic right now, I don't mind accommodating his preferences. When/if Schwern moves on and Elliot comes back in, then we can switch it back. Let's just do the transition in one commit so it is easy to revert. For argument's sake though, how is this different from using Dist::Zilla? With DZ, you don't have a MANIFEST at all until you make a build. -Jeff
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
X-RT-Original-Encoding: utf-8
Content-Length: 1682
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-10813-1330144855-565.43908-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.8.HEAD-10812-1330138467-1959.43908-5-0 [...] rt.cpan.org> <6035D60E-0BA1-4D25-AD97-D90CADE30C18 [...] imaginative-software.com> <rt-3.8.HEAD-10813-1330144855-565.43908-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-10813-1330150911-69.43908-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 505
Download (untitled) / with headers
text/plain 505b
On Fri Feb 24 23:40:55 2012, jeff@imaginative-software.com wrote: Show quoted text
> > On Feb 24, 2012, at 6:54 PM, Michael G Schwern via RT wrote: >
> > I would request somebody other than Elliot in the Perl::Critic
> project respond please.
OK. Occasionally I fall over the lack of a MANIFEST, usually when adding modules. Or when I update and someone else has added a module. When that happens, I grumble, do a realclean, then run Build.PL, and ./Build manifest. So far, that has always straightened things out.
From schwern [...] pobox.com Sat Feb 25 12: 17:11 2012
MIME-Version: 1.0
X-Spam-Status: No, score=-1.221 tagged_above=-99.9 required=10 tests=[AWL=0.000, BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, SPF_NEUTRAL=0.779] autolearn=no
In-Reply-To: <rt-3.8.HEAD-10813-1330144856-1285.43908-6-0 [...] rt.cpan.org>
X-Spam-Flag: NO
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.8.HEAD-10812-1330138467-1959.43908-5-0 [...] rt.cpan.org> <6035D60E-0BA1-4D25-AD97-D90CADE30C18 [...] imaginative-software.com> <rt-3.8.HEAD-10813-1330144856-1285.43908-6-0 [...] rt.cpan.org>
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Message-ID: <4F49178D.3090302 [...] pobox.com>
Content-Type: text/plain; charset=UTF-8
X-RT-Original-Encoding: utf-8
X-Spam-Score: -1.221
Authentication-Results: hipster.bestpractical.com (amavisd-new); dkim=pass header.i= [...] pobox.com
Authentication-Results: hipster.bestpractical.com (amavisd-new); domainkeys=pass header.from=schwern [...] pobox.com
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 46270240095 for <cpan-bug+Perl-Critic [...] hipster.bestpractical.com>; Sat, 25 Feb 2012 12:17:11 -0500 (EST)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id H0JFfzgf57j3 for <cpan-bug+Perl-Critic [...] hipster.bestpractical.com>; Sat, 25 Feb 2012 12:17:10 -0500 (EST)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id EB2C3240093 for <bug-Perl-Critic [...] rt.cpan.org>; Sat, 25 Feb 2012 12:17:08 -0500 (EST)
Received: (qmail 32676 invoked by uid 103); 25 Feb 2012 17:17:08 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 25 Feb 2012 17:17:08 -0000
Received: from b-pb-sasl-quonix.pobox.com (HELO smtp.pobox.com) (208.72.237.35) by 16.mx.develooper.com (qpsmtpd/0.80/v0.80-19-gf52d165) with ESMTP; Sat, 25 Feb 2012 09:17:05 -0800
Received: from smtp.pobox.com (unknown [127.0.0.1]) by b-sasl-quonix.pobox.com (Postfix) with ESMTP id 0BBEA70A0 for <bug-Perl-Critic [...] rt.cpan.org>; Sat, 25 Feb 2012 12:17:02 -0500 (EST)
Received: from b-pb-sasl-quonix.pobox.com (unknown [127.0.0.1]) by b-sasl-quonix.pobox.com (Postfix) with ESMTP id 01358709F for <bug-Perl-Critic [...] rt.cpan.org>; Sat, 25 Feb 2012 12:17:02 -0500 (EST)
Received: from [10.23.42.10] (unknown [76.115.1.76]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by b-sasl-quonix.pobox.com (Postfix) with ESMTPSA id 5CC19709D for <bug-Perl-Critic [...] rt.cpan.org>; Sat, 25 Feb 2012 12:17:00 -0500 (EST)
Delivered-To: cpan-bug+Perl-Critic [...] hipster.bestpractical.com
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:10.0.2) Gecko/20120216 Thunderbird/10.0.2
Subject: Re: [rt.cpan.org #43908] Add MANIFEST to the repository
Domainkey-Signature: a=rsa-sha1; c=nofws; d=pobox.com; h=message-id:date :from:mime-version:to:subject:references:in-reply-to :content-type:content-transfer-encoding; q=dns; s=sasl; b=rBqudQ Gm8g6foaY3C634aCmNQh+E5GT4HkJr2/1jO5a2KwNW/cbbMxzVom2emBvaRZEfor zVaoWoEwKuqYFORxCwqYjj97ykXPHMtWgCksp2Lxh+Noui/G2OHd+zGPtvY9dB4L kNB0wUkmbZ0d+6vEqMNcTYpJyjyeDbMg0heZQ=
Return-Path: <schwern [...] pobox.com>
Dkim-Signature: v=1; a=rsa-sha1; c=relaxed; d=pobox.com; h=message-id :date:from:mime-version:to:subject:references:in-reply-to :content-type:content-transfer-encoding; s=sasl; bh=/zQ74NnU6D0K N+RNxJzgP31S4is=; b=l8vp6BxJJp1JAes15DcZ9JOpDz4a2OFTEORPO3eo91m6 p58nlahZi6dmO1DSSAss3Jk/pycuCXJvXJgryRLwS+/5btIQ2qvtL3qZP64LG+bz UnxsMetH/Gs9gcItNqjjK0KlVnv2t0hJFXv7WKJSL49FAgmfwkegt8BdyKDBisg=
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: cpan-bug+Perl-Critic [...] hipster.bestpractical.com
X-RT-Mail-Extension: perl-critic
Date: Sat, 25 Feb 2012 09:17:01 -0800
X-Spam-Level:
To: bug-Perl-Critic [...] rt.cpan.org
X-Enigmail-Version: 1.3.5
Content-Transfer-Encoding: 7bit
X-Pobox-Relay-ID: 87421BB8-5FD4-11E1-81A4-9DB42E706CDE-02258300!b-pb-sasl-quonix.pobox.com
From: Michael G Schwern <schwern [...] pobox.com>
RT-Message-ID: <rt-3.8.HEAD-10810-1330190232-122.43908-0-0 [...] rt.cpan.org>
Content-Length: 2410
Download (untitled) / with headers
text/plain 2.3k
On 2012.2.24 8:40 PM, Jeffrey Thalhammer via RT wrote: Show quoted text
> I can see both sides of the argument. But I have been bitten by the lack > of MANIFEST a few times recently (and I do use M::B rather than EU::MM, > so I'm not sure why). So I'm ok with stashing the MANIFEST in the repository. > > Since Schwern is the only person actively working on Perl::Critic right > now, I don't mind accommodating his preferences. When/if Schwern moves > on and Elliot comes back in, then we can switch it back.
While I'm happy about the conclusion, I'm not so happy about the "do whatever the current developer wants" rationale. The principle driving this should be about ease of use for new and existing developers. There's three counts of missing MANIFEST frustration (you, me and Tom) with little benefit for not checking it in. Under the cockroach law that means there's probably dozens more who have been stymied and never said anything. Tailoring the dev workflow to the particular quirks of whatever single developer happens to be working on it at the moment is a great way to ensure that you only ever have a single developer working on the project. Show quoted text
> Let's just do the transition in one commit so it is easy to revert.
I figure all that needs to be done is add the MANIFEST and then delete it to revert. Is there something else? Show quoted text
> For argument's sake though, how is this different from using Dist::Zilla? > With DZ, you don't have a MANIFEST at all until you make a build.
I don't use Dist::Zilla much, so I can't really comment on that. Though I do gripe about similar issues with Dist::Zilla distributions: the normal CPAN module workflow does not work fresh out of the repository and requires special setup. What I can say for Perl::Critic is neither `Build` nor `Build test` build the MANIFEST which leaves the tests in an unrunable state. A build which leaves the tests broken would clearly be considered a failure were this argument not muddled by VCS dogma. Maybe I don't have an "author" flag set, but that's yet more complexity devs have to deal with just to avoid checking in a file. This could be minimally resolved by having the build step generate the MANIFEST if it's missing, but this would lose the benefit of having a historical log of MANIFEST changes and retain the disadvantage that the MANIFEST might be generated incorrectly by each developer. -- Hating the web since 1994.
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-10810-1330190232-122.43908-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.8.HEAD-10812-1330138467-1959.43908-5-0 [...] rt.cpan.org> <6035D60E-0BA1-4D25-AD97-D90CADE30C18 [...] imaginative-software.com> <rt-3.8.HEAD-10813-1330144856-1285.43908-6-0 [...] rt.cpan.org> <4F49178D.3090302 [...] pobox.com> <rt-3.8.HEAD-10810-1330190232-122.43908-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-10809-1330219576-728.43908-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 2782
Download (untitled) / with headers
text/plain 2.7k
On Sat Feb 25 12:17:12 2012, schwern@pobox.com wrote: Show quoted text
> > The principle driving this should be about ease of use for new and > existing developers. There's three counts of missing MANIFEST > frustration (you, me and Tom) with little benefit for not checking > it in. Under the cockroach law that means there's probably dozens > more who have been stymied and never said anything.
Well, I would have called it an occasional minor inconvenience, rather than frustration. I have never been, or felt, blocked. Show quoted text
> > Tailoring the dev workflow to the particular quirks of whatever > single developer happens to be working on it at the moment is a > great way to ensure that you only ever have a single developer > working on the project.
Exactly right. My take on it is that it is Jeff's sandbox. He has allowed me to play in it. If I do not like his games and his toys, the civilized thing for me to do is to thank him kindly, and go home. Well, OK, maybe try to convince him my game or my toy is better -- but remembering that it is, after all, his sandbox, and trying to accord him the same consideration that I would like if he were playing in mine. Show quoted text
> > This could be minimally resolved by having the build step generate the > MANIFEST if it's missing, but this would lose the benefit of having a > historical log of MANIFEST changes and retain the disadvantage that > the MANIFEST might be generated incorrectly by each developer.
I don't know that I have ever felt that the MANIFEST was generated incorrectly. There have been times when something I did rendered it incorrect, but a reset (spelled "./Build realclean") generally seems to fix things up. As for MANIFEST history, it seems to me that if it's derived, the history of the MANIFEST is less interesting than the project it's derived from -- at least under SVN. Under CVS, it's a different story, because commits are per-file. Under SVN, if the history says I added or deleted a file, it will say that whether or not MANIFEST is tracked. Development requires Module::Build. I find that its documentation seems to be written for everyone but me, but withal it seems to be pretty protean. I can think of at least two alternatives to hard-coding the MANIFEST: * Override the default target (whatever that is) to make it depend on the 'makemanifest' target (whatever it's called); * Generate the MANIFEST using code in Build.PL. As for Dist::Zilla, I don't use it either. Every time I look at it, I come away with the impression of a Procrustean, all-your-distribution-are-belong-to-us behemoth that can't wait to scribble all over my source. But I know that Jeff is looking into it, and if he works out the details and gets it going to his satisfaction, I will at least try letting it scribble all over his files.
From jeff [...] imaginative-software.com Sun Feb 26 01: 16:22 2012
MIME-Version: 1.0 (Apple Message framework v1084)
X-Spam-Status: No, score=-3.358 tagged_above=-99.9 required=10 tests=[AWL=-1.459, BAYES_00=-1.9, HTML_MESSAGE=0.001] autolearn=ham
In-Reply-To: <rt-3.8.HEAD-10810-1330190232-238.43908-5-0 [...] rt.cpan.org>
X-Mailer: Apple Mail (2.1084)
X-Spam-Flag: NO
Received-SPF: pass (google.com: domain of jeff [...] imaginative-software.com designates 10.68.197.71 as permitted sender) client-ip=10.68.197.71;
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.8.HEAD-10812-1330138467-1959.43908-5-0 [...] rt.cpan.org> <6035D60E-0BA1-4D25-AD97-D90CADE30C18 [...] imaginative-software.com> <rt-3.8.HEAD-10813-1330144856-1285.43908-6-0 [...] rt.cpan.org> <4F49178D.3090302 [...] pobox.com> <rt-3.8.HEAD-10810-1330190232-238.43908-5-0 [...] rt.cpan.org>
X-Virus-Checked: Checked by ClamAV on 16.mx.develooper.com
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Content-Type: multipart/alternative; boundary=Apple-Mail-26--257547329
Message-ID: <F106F418-636A-4EBC-AB0F-252049A7324F [...] imaginative-software.com>
X-Spam-Score: -3.358
Authentication-Results: mr.google.com; spf=pass (google.com: domain of jeff [...] imaginative-software.com designates 10.68.197.71 as permitted sender) smtp.mail=jeff [...] imaginative-software.com
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id BA8F42400AB for <cpan-bug+Perl-Critic [...] hipster.bestpractical.com>; Sun, 26 Feb 2012 01:16:21 -0500 (EST)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id s5hpn4E+LPEv for <cpan-bug+Perl-Critic [...] hipster.bestpractical.com>; Sun, 26 Feb 2012 01:16:20 -0500 (EST)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 3D7CE240095 for <bug-Perl-Critic [...] rt.cpan.org>; Sun, 26 Feb 2012 01:16:20 -0500 (EST)
Received: (qmail 19479 invoked by uid 103); 26 Feb 2012 06:16:19 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 26 Feb 2012 06:16:19 -0000
Received: from mail-pw0-f50.google.com (HELO mail-pw0-f50.google.com) (209.85.160.50) by 16.mx.develooper.com (qpsmtpd/0.80/v0.80-19-gf52d165) with ESMTP; Sat, 25 Feb 2012 22:16:16 -0800
Received: by pbcwy7 with SMTP id wy7so3854677pbc.9 for <bug-Perl-Critic [...] rt.cpan.org>; Sat, 25 Feb 2012 22:16:13 -0800 (PST)
Received: from mr.google.com ([10.68.197.71]) by 10.68.197.71 with SMTP id is7mr24343348pbc.33.1330236973608 (num_hops = 1); Sat, 25 Feb 2012 22:16:13 -0800 (PST)
Received: by 10.68.197.71 with SMTP id is7mr20528285pbc.33.1330236973515; Sat, 25 Feb 2012 22:16:13 -0800 (PST)
Received: from [192.168.1.3] (c-24-6-41-85.hsd1.ca.comcast.net. [24.6.41.85]) by mx.google.com with ESMTPS id y7sm9075599pbp.25.2012.02.25.22.16.11 (version=TLSv1/SSLv3 cipher=OTHER); Sat, 25 Feb 2012 22:16:12 -0800 (PST)
Delivered-To: cpan-bug+Perl-Critic [...] hipster.bestpractical.com
Subject: Re: [rt.cpan.org #43908] Add MANIFEST to the repository
Return-Path: <jeff [...] imaginative-software.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: cpan-bug+Perl-Critic [...] hipster.bestpractical.com
X-RT-Mail-Extension: perl-critic
Date: Sat, 25 Feb 2012 22:16:10 -0800
X-Spam-Level:
To: bug-Perl-Critic [...] rt.cpan.org
From: Jeffrey Thalhammer <jeff [...] imaginative-software.com>
X-GM-Message-State: ALoCoQkuNNL/wDLWxiUHcbOCwAPuhUjSq/Sw9aaqHRliNZLyCCFWF4d/lPcb0MibikaE2MoEN09r
RT-Message-ID: <rt-3.8.HEAD-10813-1330236982-1102.43908-0-0 [...] rt.cpan.org>
Content-Length: 0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
X-RT-Original-Encoding: utf-8
Content-Length: 526
Download (untitled) / with headers
text/plain 526b
On Feb 25, 2012, at 9:17 AM, Michael G Schwern via RT wrote: Show quoted text
> While I'm happy about the conclusion, I'm not so happy about the "do whatever > the current developer wants" rationale.
I should have said that differently. The custom handling of the MANIFEST file contradicts the workflow implied by M::B and EU::MM. Our customization is not without merit, nor are M::B and EU::MM the final word on development workflows. But on the principle of least astonishment, I declare the MANIFEST shall be in the repository. -Jeff
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
X-RT-Original-Encoding: utf-8
Content-Length: 1368
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-10812-1330242686-136.43908-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 161
Download (untitled) / with headers
text/plain 161b
All dists now stash a their MANIFEST in the repos. This issue is now resolved. -- Jeffrey Thalhammer Imaginative Software Systems www.imaginative-software.com
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-10812-1330242686-136.43908-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <rt-3.8.HEAD-10812-1330242686-136.43908-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-10814-1330300094-1667.43908-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 312
Download (untitled) / with headers
text/plain 312b
Le 2012-02-26 08:51:26, THALJEF a écrit : Show quoted text
> All dists now stash a their MANIFEST in the repos. This issue is now > resolved.
The committed MANIFEST is incomplete. It lacks generated files: - META.yml - META.json - README - lib/Perl/Critic/PolicySummary.pod -- Olivier Mengué - http://perlresume.org/DOLMEN
From jeff [...] imaginative-software.com Mon Feb 27 00: 53:50 2012
MIME-Version: 1.0 (Apple Message framework v1084)
X-Spam-Status: No, score=-3.245 tagged_above=-99.9 required=10 tests=[AWL=-1.346, BAYES_00=-1.9, HTML_MESSAGE=0.001] autolearn=ham
In-Reply-To: <rt-3.8.HEAD-10814-1330300094-249.43908-5-0 [...] rt.cpan.org>
X-Mailer: Apple Mail (2.1084)
X-Spam-Flag: NO
Received-SPF: pass (google.com: domain of jeff [...] imaginative-software.com designates 10.68.201.196 as permitted sender) client-ip=10.68.201.196;
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.8.HEAD-10812-1330242686-136.43908-5-0 [...] rt.cpan.org> <rt-3.8.HEAD-10814-1330300094-249.43908-5-0 [...] rt.cpan.org>
X-Virus-Checked: Checked by ClamAV on 16.mx.develooper.com
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Content-Type: multipart/alternative; boundary=Apple-Mail-27--172513032
Message-ID: <158CFF4C-95C1-48CD-8A19-A5CC4C091FBB [...] imaginative-software.com>
X-Spam-Score: -3.245
Authentication-Results: mr.google.com; spf=pass (google.com: domain of jeff [...] imaginative-software.com designates 10.68.201.196 as permitted sender) smtp.mail=jeff [...] imaginative-software.com
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 383C62400E7 for <cpan-bug+Perl-Critic [...] hipster.bestpractical.com>; Mon, 27 Feb 2012 00:53:50 -0500 (EST)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id jeRwpdAkzv17 for <cpan-bug+Perl-Critic [...] hipster.bestpractical.com>; Mon, 27 Feb 2012 00:53:48 -0500 (EST)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 64E8D2400B0 for <bug-Perl-Critic [...] rt.cpan.org>; Mon, 27 Feb 2012 00:53:48 -0500 (EST)
Received: (qmail 430 invoked by uid 103); 27 Feb 2012 05:53:47 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 27 Feb 2012 05:53:47 -0000
Received: from mail-pw0-f50.google.com (HELO mail-pw0-f50.google.com) (209.85.160.50) by 16.mx.develooper.com (qpsmtpd/0.80/v0.80-19-gf52d165) with ESMTP; Sun, 26 Feb 2012 21:53:44 -0800
Received: by pbcwz7 with SMTP id wz7so59937pbc.9 for <bug-Perl-Critic [...] rt.cpan.org>; Sun, 26 Feb 2012 21:53:41 -0800 (PST)
Received: from mr.google.com ([10.68.201.196]) by 10.68.201.196 with SMTP id kc4mr27327043pbc.141.1330322021746 (num_hops = 1); Sun, 26 Feb 2012 21:53:41 -0800 (PST)
Received: by 10.68.201.196 with SMTP id kc4mr23441892pbc.141.1330322021695; Sun, 26 Feb 2012 21:53:41 -0800 (PST)
Received: from [192.168.1.3] (c-24-6-41-85.hsd1.ca.comcast.net. [24.6.41.85]) by mx.google.com with ESMTPS id s10sm10678444pbq.13.2012.02.26.21.53.39 (version=TLSv1/SSLv3 cipher=OTHER); Sun, 26 Feb 2012 21:53:40 -0800 (PST)
Delivered-To: cpan-bug+Perl-Critic [...] hipster.bestpractical.com
Subject: Re: [rt.cpan.org #43908] Add MANIFEST to the repository
Return-Path: <jeff [...] imaginative-software.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: cpan-bug+Perl-Critic [...] hipster.bestpractical.com
X-RT-Mail-Extension: perl-critic
Date: Sun, 26 Feb 2012 21:53:25 -0800
X-Spam-Level:
To: bug-Perl-Critic [...] rt.cpan.org
From: Jeffrey Thalhammer <jeff [...] imaginative-software.com>
X-GM-Message-State: ALoCoQnH8L3oLcE2Xsd+RNxqjWsRl7+5lpu1m5novx1teilrO5CRzsvtd8OA9v2uT1x8a+PX5Wg2
RT-Message-ID: <rt-3.8.HEAD-10809-1330322031-140.43908-0-0 [...] rt.cpan.org>
Content-Length: 0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
X-RT-Original-Encoding: utf-8
Content-Length: 1139
Download (untitled) / with headers
text/plain 1.1k
On Feb 26, 2012, at 3:48 PM, Olivier Mengué via RT wrote: Show quoted text
> The committed MANIFEST is incomplete. It lacks generated files:
Ugh. This is probably why we generated the MANIFEST in the first place :-/ So I've got to put all these files in the repo too, I guess. Now I'm starting to feel like the EU::MM and M::B workflow really is just broken. With M::B anyway, it seems to view authors and users as the same. But when an author makes a build, he/she may want certain files generated, and then removed upon cleanup. But for the user, those files should already be present (e.g. MANIFEST) and not removed on cleanup. Trouble is, the targets all do the same thing, no matter who you are. I think Elliot was trying to address all this, but the limitations of M::B may have gotten in the way. This is one reason I like Dist::Zilla. It establishes a distinct authoring phase *before* building where you can automate stuff, and you don't have to expose your users to how all that happened. Maybe I'll just bite the bullet and move the project to D::Z now, rather than trying to rig up M::B in a way that makes everyone happy. -Jeff
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
X-RT-Original-Encoding: utf-8
Content-Length: 2077
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-10809-1330322031-140.43908-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.8.HEAD-10812-1330242686-136.43908-5-0 [...] rt.cpan.org> <rt-3.8.HEAD-10814-1330300094-249.43908-5-0 [...] rt.cpan.org> <158CFF4C-95C1-48CD-8A19-A5CC4C091FBB [...] imaginative-software.com> <rt-3.8.HEAD-10809-1330322031-140.43908-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-10813-1330336173-921.43908-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
RT-Send-CC: schwern [...] pobox.com, perl [...] galumph.com, jeff [...] imaginative-software.com
Content-Length: 2104
Le 2012-02-27 06:53:51, jeff@imaginative-software.com a écrit : Show quoted text
> > On Feb 26, 2012, at 3:48 PM, Olivier Mengué via RT wrote: >
> > The committed MANIFEST is incomplete. It lacks generated files:
> > Ugh. This is probably why we generated the MANIFEST in the first > place :-/ > > So I've got to put all these files in the repo too, I guess. Now I'm > starting to feel like the EU::MM and M::B workflow really is just > broken. > > With M::B anyway, it seems to view authors and users as the same.
That's a great feature for open source projects. Unfortunately SubVersion does not treat authors (commiters) and users (patch writers) contrary to Git (where non-committers can easily write a patch as multiple committed step and submit that). Show quoted text
> But > when an author makes a build, he/she may want certain files > generated, and then removed upon cleanup. But for the user, those > files should already be present (e.g. MANIFEST) and not removed on > cleanup. Trouble is, the targets all do the same thing, no matter > who you are.
The M::B solution would be to override the 'test' action to run 'manifest' when missing and override 'manifest' to require 'policysummary'. Show quoted text
> This is one reason I like Dist::Zilla. It establishes a distinct > authoring phase *before* building where you can automate stuff, and > you don't have to expose your users to how all that happened. > Maybe I'll just bite the bullet and move the project to D::Z now, > rather than trying to rig up M::B in a way that makes everyone > happy.
The tricky part with D::Z is this: embedding in the distribution itself some D::Z plugins that are part of the build process (generation of the PolicySummary). The trick is to use '=' as a prefix to load plugins using absolute names. For example, use [=inc::PolicySummary] in dist.ini to load inc/PolicySummary.pm. Also, one problem that D::Z will create is that it is really slow. And re-generating "PolicySummary.pod" and MANIFEST for every "dzil test" will make that much worse. -- Olivier Mengué - http://perlresume.org/DOLMEN
From jeff [...] imaginative-software.com Mon Feb 27 14: 21:27 2012
MIME-Version: 1.0 (Apple Message framework v1084)
X-Spam-Status: No, score=-1.899 tagged_above=-99.9 required=10 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001] autolearn=ham
In-Reply-To: <rt-3.8.HEAD-10813-1330336174-207.43908-5-0 [...] rt.cpan.org>
X-Mailer: Apple Mail (2.1084)
X-Spam-Flag: NO
Received-SPF: pass (google.com: domain of jeff [...] imaginative-software.com designates 10.68.132.102 as permitted sender) client-ip=10.68.132.102;
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.8.HEAD-10812-1330242686-136.43908-5-0 [...] rt.cpan.org> <rt-3.8.HEAD-10814-1330300094-249.43908-5-0 [...] rt.cpan.org> <158CFF4C-95C1-48CD-8A19-A5CC4C091FBB [...] imaginative-software.com> <rt-3.8.HEAD-10809-1330322031-140.43908-5-0 [...] rt.cpan.org> <rt-3.8.HEAD-10813-1330336174-207.43908-5-0 [...] rt.cpan.org>
X-Virus-Checked: Checked by ClamAV on 16.mx.develooper.com
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Content-Type: multipart/alternative; boundary=Apple-Mail-32--124040925
Message-ID: <92860187-4658-43BC-B771-95120ED4C99A [...] imaginative-software.com>
X-Spam-Score: -1.899
Authentication-Results: mr.google.com; spf=pass (google.com: domain of jeff [...] imaginative-software.com designates 10.68.132.102 as permitted sender) smtp.mail=jeff [...] imaginative-software.com
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 770012400FF for <cpan-bug+Perl-Critic [...] hipster.bestpractical.com>; Mon, 27 Feb 2012 14:21:27 -0500 (EST)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 1qkhkYYGT-Ks for <cpan-bug+Perl-Critic [...] hipster.bestpractical.com>; Mon, 27 Feb 2012 14:21:26 -0500 (EST)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id A16EE24001F for <bug-Perl-Critic [...] rt.cpan.org>; Mon, 27 Feb 2012 14:21:25 -0500 (EST)
Received: (qmail 15415 invoked by uid 103); 27 Feb 2012 19:21:24 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 27 Feb 2012 19:21:24 -0000
Received: from mail-pw0-f50.google.com (HELO mail-pw0-f50.google.com) (209.85.160.50) by 16.mx.develooper.com (qpsmtpd/0.80/v0.80-19-gf52d165) with ESMTP; Mon, 27 Feb 2012 11:21:21 -0800
Received: by pbcwz7 with SMTP id wz7so731433pbc.9 for <bug-Perl-Critic [...] rt.cpan.org>; Mon, 27 Feb 2012 11:21:19 -0800 (PST)
Received: from mr.google.com ([10.68.132.102]) by 10.68.132.102 with SMTP id ot6mr15084817pbb.157.1330370479300 (num_hops = 1); Mon, 27 Feb 2012 11:21:19 -0800 (PST)
Received: by 10.68.132.102 with SMTP id ot6mr12705712pbb.157.1330370479240; Mon, 27 Feb 2012 11:21:19 -0800 (PST)
Received: from callahan.dhcp.gene.com (fgt-rwc-ext-open.gene.com. [72.34.128.250]) by mx.google.com with ESMTPS id y1sm13404049pbg.73.2012.02.27.11.21.18 (version=TLSv1/SSLv3 cipher=OTHER); Mon, 27 Feb 2012 11:21:18 -0800 (PST)
Delivered-To: cpan-bug+Perl-Critic [...] hipster.bestpractical.com
Subject: Re: [rt.cpan.org #43908] Add MANIFEST to the repository
Return-Path: <jeff [...] imaginative-software.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: cpan-bug+Perl-Critic [...] hipster.bestpractical.com
X-RT-Mail-Extension: perl-critic
Date: Mon, 27 Feb 2012 11:21:17 -0800
X-Spam-Level:
To: bug-Perl-Critic [...] rt.cpan.org
From: Jeffrey Thalhammer <jeff [...] imaginative-software.com>
X-GM-Message-State: ALoCoQndeavmBoRiW8l7zyfFHlP7Jrim/Z5UMCrQ5bp0fgEEAq9LwDOSgCkMTp04n5Ps3T6MvUNL
RT-Message-ID: <rt-3.8.HEAD-10815-1330370488-1358.43908-0-0 [...] rt.cpan.org>
Content-Length: 0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
X-RT-Original-Encoding: utf-8
Content-Length: 1828
Download (untitled) / with headers
text/plain 1.7k
On Feb 27, 2012, at 1:49 AM, Olivier Mengué via RT wrote: Show quoted text
> The M::B solution would be to override the 'test' action to run > 'manifest' when missing and override 'manifest' to require 'policysummary'.
I've been thinking about this all morning. I think it all boils down to whether you feel you should be able to do the default build cycle directly on the code as it comes from the repository (e.g. "perl Build.PL; ./Build test"). For simple projects you can get away with using the default build cycle, but it is only by accident. More complex projects (like Perl::Critic) need certain things to happen before you can actually make a build. As I see it, this is what the "dist" action is for. The "dist" action is where you generate all the bits that have to be present for packaging. In other words, this is when you make something that can be built. So this is when the MANIFEST, META, PolicySummary, etc. should be created. By the time you (or a user) runs the "test" action, those files must already be present. The key implication here is that authors can't really expect to use the "test" and "clean" actions on the code in the repository. Rather, you need to use "disttest" and "distclean" if you really want to run the build cycle properly. The more I think about it, the more convinced I am that this is exactly how the M::B workflow is supposed to go. It's just that I (and perhaps others) have fallen into the habit of using the "test" and "clean" actions because it saves a bit of time. But that time is a necessary step in the package/build/test cycle for Perl::Critic. For those of you who are more seasoned developers, this all may be obvious to you. But I am just now coming to this realization. Ironically, I think this leads me back to *not* stashing the MANIFEST in the repository. -Jeff
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
X-RT-Original-Encoding: utf-8
Content-Length: 2841
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-10815-1330370488-1358.43908-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.8.HEAD-10812-1330242686-136.43908-5-0 [...] rt.cpan.org> <rt-3.8.HEAD-10814-1330300094-249.43908-5-0 [...] rt.cpan.org> <158CFF4C-95C1-48CD-8A19-A5CC4C091FBB [...] imaginative-software.com> <rt-3.8.HEAD-10809-1330322031-140.43908-5-0 [...] rt.cpan.org> <rt-3.8.HEAD-10813-1330336174-207.43908-5-0 [...] rt.cpan.org> <92860187-4658-43BC-B771-95120ED4C99A [...] imaginative-software.com> <rt-3.8.HEAD-10815-1330370488-1358.43908-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-10813-1330400792-1383.43908-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 855
Download (untitled) / with headers
text/plain 855b
On Mon Feb 27 14:21:28 2012, jeff@imaginative-software.com wrote: Show quoted text
> For those of you who are more seasoned developers, this all may be > obvious to you. But I am just now coming to this realization. > Ironically, I think this leads me back to *not* stashing the > MANIFEST in the repository.
Go for it. Or revert for it. Either way is good with me. However it was originally decided not to have the MANIFEST in the repository, the current situation seems to be that putting it there is not as simple as it originally appeared. FWIW, I always use 'realclean', but mostly because when I'm cleaning at all I'm starting from scratch. I do 'test', except that I always 'disttest' as part of actually making a distribution. I suspect the recent spate of releases of Net::SSLeay to be an example of what happens when you _don't_ 'disttest'. Tom
From jeff [...] imaginative-software.com Tue Feb 28 01: 29:23 2012
MIME-Version: 1.0 (Apple Message framework v1084)
X-Spam-Status: No, score=-3.149 tagged_above=-99.9 required=10 tests=[AWL=-1.250, BAYES_00=-1.9, HTML_MESSAGE=0.001] autolearn=ham
In-Reply-To: <rt-3.8.HEAD-10813-1330336174-207.43908-5-0 [...] rt.cpan.org>
X-Mailer: Apple Mail (2.1084)
X-Spam-Flag: NO
Received-SPF: pass (google.com: domain of jeff [...] imaginative-software.com designates 10.68.232.230 as permitted sender) client-ip=10.68.232.230;
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.8.HEAD-10812-1330242686-136.43908-5-0 [...] rt.cpan.org> <rt-3.8.HEAD-10814-1330300094-249.43908-5-0 [...] rt.cpan.org> <158CFF4C-95C1-48CD-8A19-A5CC4C091FBB [...] imaginative-software.com> <rt-3.8.HEAD-10809-1330322031-140.43908-5-0 [...] rt.cpan.org> <rt-3.8.HEAD-10813-1330336174-207.43908-5-0 [...] rt.cpan.org>
X-Virus-Checked: Checked by ClamAV on 16.mx.develooper.com
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Content-Type: multipart/alternative; boundary=Apple-Mail-34--84001912
Message-ID: <E2975B81-CC7C-4018-B0E2-2A6AC5AE5CDD [...] imaginative-software.com>
X-Spam-Score: -3.149
Authentication-Results: mr.google.com; spf=pass (google.com: domain of jeff [...] imaginative-software.com designates 10.68.232.230 as permitted sender) smtp.mail=jeff [...] imaginative-software.com
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id DB5CF240108 for <cpan-bug+Perl-Critic [...] hipster.bestpractical.com>; Tue, 28 Feb 2012 01:29:23 -0500 (EST)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id hErtNYZ8vay4 for <cpan-bug+Perl-Critic [...] hipster.bestpractical.com>; Tue, 28 Feb 2012 01:29:21 -0500 (EST)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id D4AAD2400B8 for <bug-Perl-Critic [...] rt.cpan.org>; Tue, 28 Feb 2012 01:29:20 -0500 (EST)
Received: (qmail 12228 invoked by uid 103); 28 Feb 2012 06:29:20 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 28 Feb 2012 06:29:20 -0000
Received: from mail-pz0-f50.google.com (HELO mail-pz0-f50.google.com) (209.85.210.50) by 16.mx.develooper.com (qpsmtpd/0.80/v0.80-19-gf52d165) with ESMTP; Mon, 27 Feb 2012 22:29:14 -0800
Received: by dald2 with SMTP id d2so348066dal.9 for <bug-Perl-Critic [...] rt.cpan.org>; Mon, 27 Feb 2012 22:29:09 -0800 (PST)
Received: from mr.google.com ([10.68.232.230]) by 10.68.232.230 with SMTP id tr6mr49386296pbc.165.1330410549985 (num_hops = 1); Mon, 27 Feb 2012 22:29:09 -0800 (PST)
Received: by 10.68.232.230 with SMTP id tr6mr41705014pbc.165.1330410549926; Mon, 27 Feb 2012 22:29:09 -0800 (PST)
Received: from [192.168.1.3] (c-24-6-41-85.hsd1.ca.comcast.net. [24.6.41.85]) by mx.google.com with ESMTPS id k6sm14832950pbl.27.2012.02.27.22.29.07 (version=TLSv1/SSLv3 cipher=OTHER); Mon, 27 Feb 2012 22:29:08 -0800 (PST)
Delivered-To: cpan-bug+Perl-Critic [...] hipster.bestpractical.com
Subject: Re: [rt.cpan.org #43908] Add MANIFEST to the repository
Return-Path: <jeff [...] imaginative-software.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: cpan-bug+Perl-Critic [...] hipster.bestpractical.com
X-RT-Mail-Extension: perl-critic
Date: Mon, 27 Feb 2012 22:28:36 -0800
X-Spam-Level:
To: bug-Perl-Critic [...] rt.cpan.org
From: Jeffrey Thalhammer <jeff [...] imaginative-software.com>
X-GM-Message-State: ALoCoQmd3Jeryi0DYD3a73wfoITLQ65ss7rc8ee07sjUZhzfhLLkpZfx9WWLY+KZDQSdn7HH1zSX
RT-Message-ID: <rt-3.8.HEAD-10813-1330410564-16.43908-0-0 [...] rt.cpan.org>
Content-Length: 0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
X-RT-Original-Encoding: utf-8
Content-Length: 2914
Download (untitled) / with headers
text/plain 2.8k
On Feb 27, 2012, at 1:49 AM, Olivier Mengué via RT wrote: Show quoted text
> The tricky part with D::Z is this: embedding in the distribution itself > some D::Z plugins that are part of the build process (generation of the > PolicySummary). The trick is to use '=' as a prefix to load plugins > using absolute names. For example, use [=inc::PolicySummary] in dist.ini > to load inc/PolicySummary.pm.
I haven't adopted the habit of putting D::Z plugins in my distribution. And I doubt I ever will. This whole discussion about the MANIFEST leads me to think that the development model for M::B and EU::MM is somewhat out of date. Both of them treat the distribution as something that can be re-packaged and re-released. In the past that was usually true, and it helped the open source community build itself. But these days, modern development uses a lot of tools to generate code. So the code that is in the distribution could be a long way from what was actually authored. Unless I ship the distribution with all the authoring tools that I used, then users can't re-package the code without getting those same tools for themselves. I don't think this is evil -- it's just progress. The development process is more sophisticated and the tool chain is longer than it used to be. But I don't think I'd want to ship my dist with D::Z plugins for the same reasons that I don't ship it with make or vim. These days, patching is done against the VCS, not the distribution[1]. So I feel that aspiring to make your distribution re-packagable to end users is a noble goal, but it is increasingly impractical. In that light, I don't feel so guilty about contorting the default M::B and EU::MM workflow to suit our needs. As long as it builds "the right way" for end users, then all is well. Show quoted text
> Also, one problem that D::Z will create is that it is really slow. And > re-generating "PolicySummary.pod" and MANIFEST for every "dzil test" > will make that much worse.
The performance is unfortunate, but not so bad that I want to abandon good engineering principles. So after much though, I've concluded that I do not agree with Schwern about these so-called "static generated files". I generally do believe the in "version control dogma" that you should never commit something that you can generate. I think the real debate here is more about *who* should be generating these files and *when* they should be doing it. My career as a software engineer is relatively short, and my career in open source is even shorter. So I welcome enlightenment from those of you with more experience and insight. But at this moment, all the above best describes my point of view. [1] This is one reason I found the RequireVcsKeywords policy helpful -- so that end users know where to get the actual (human-authored) source code if they want to make a patch. But that is another story. Thanks for letting me bend your ear. -Jeff
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
X-RT-Original-Encoding: utf-8
Content-Length: 4605
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-10813-1330410564-16.43908-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <RT-Ticket-43908 [...] rt.cpan.org> <rt-3.8.HEAD-10812-1330242686-136.43908-5-0 [...] rt.cpan.org> <rt-3.8.HEAD-10814-1330300094-249.43908-5-0 [...] rt.cpan.org> <158CFF4C-95C1-48CD-8A19-A5CC4C091FBB [...] imaginative-software.com> <rt-3.8.HEAD-10809-1330322031-140.43908-5-0 [...] rt.cpan.org> <rt-3.8.HEAD-10813-1330336174-207.43908-5-0 [...] rt.cpan.org> <E2975B81-CC7C-4018-B0E2-2A6AC5AE5CDD [...] imaginative-software.com> <rt-3.8.HEAD-10813-1330410564-16.43908-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-10809-1330424602-1653.43908-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
RT-Send-CC: jeff [...] imaginative-software.com
Content-Length: 4301
Download (untitled) / with headers
text/plain 4.2k
Le 2012-02-28 07:29:24, jeff@imaginative-software.com a écrit : Show quoted text
> I haven't adopted the habit of putting D::Z plugins in my > distribution. And I doubt I ever will.
This doesn't have to be a habit. But it is the right and only way to make distribution content generation as part of a D::Z build process. Bundling custom D::Z plugins in inc/ is the D::Z counterpart of M::B subclassing. Show quoted text
> This whole discussion about the MANIFEST leads me to think that the > development model for M::B and EU::MM is somewhat out of date. > Both of them treat the distribution as something that can be re- > packaged and re-released. In the past that was usually true, and > it helped the open source community build itself. > > But these days, modern development uses a lot of tools to generate > code. So the code that is in the distribution could be a long way > from what was actually authored. Unless I ship the distribution > with all the authoring tools that I used, then users can't re- > package the code without getting those same tools for themselves.
The CPAN archive is the only thing I trust because I know it will be kept forever. But I usually do not trust an author source code repository due to its relative lifetime. Just see how hundreds of Marcel Grünauer repositories disappeared one day from GitHub because he decided to quit. Or see how an author may change of VCS because he prefers it (at one point Dave Rolsky used Mercurial which almost nobody else is using in the Perl community). So I think that any CPAN distribution should be rebuildable from its own content plus the BackPAN. This is why I was describing how to do that with D::Z in a way that still makes the distribution usable and rebuildable without direct access to the source code repository. The current M::B-based process is fine too. I have no problem with it (except that extending M::B for authoring purpose makes the maintenance of M::B harder). I'm using that myself for my Number::Phone::FR distribution (which also has code generation) until the day I will take time to migrate it to D::Z. Show quoted text
> I don't think this is evil -- it's just progress. The development > process is more sophisticated and the tool chain is longer than it > used to be. But I don't think I'd want to ship my dist with D::Z > plugins for the same reasons that I don't ship it with make or vim.
The current trend is to separate the build tools from the install tool. Our install tools are now big beasts because they are also authoring tools. I have good hope for Module::Build::Tiny. I plan to work on that at the QA Hackathon. Show quoted text
> These days, patching is done against the VCS, not the distribution[1]. > So I feel that aspiring to make your distribution re-packagable to > end users is a noble goal, but it is increasingly impractical. In > that light, I don't feel so guilty about contorting the default > M::B and EU::MM workflow to suit our needs. As long as it builds > "the right way" for end users, then all is well.
You seem to separate the world in two categories: - committers - "end users": users of the CPAN distribution I think that you omit one important category: the occasionnal patch submitter like myself. (S)He's not a committer on the project. (S)He may not be familiar with the VCS you are using. Show quoted text
> So after much though, I've concluded > that I do not agree with Schwern about these so-called "static > generated files". I generally do believe the in "version control > dogma" that you should never commit something that you can > generate.
I'm also of this point of view. And especially because the generated file depends on the version of the tool that generates it. For example, META.json is not created by M::B if you have no JSON module, and so will not be in MANIFEST in that environment. Think also to differences of default end-of-line (PolicySummaryGenerator use platform EOL instead of using a fixed Unix style, so the build toolchain is not portable on Win32). So generated files in a VCS are painful when different developers have different tool versions that you can't control. Of course committing those files can also help to be aware of those differences, but often with pain. -- Olivier Mengué - http://perlresume.org/DOLMEN


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.