Skip Menu |

This queue is for tickets about the Module-Build CPAN distribution.

Report information
The Basics
Id: 18186
Status: rejected
Priority: 0/
Queue: Module-Build

People
Owner: Nobody in particular
Requestors: william [...] knowmad.com
Cc:
AdminCc:

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



Subject: Build manifest and filenames with spaces
When using ./Build MANIFEST and ./Build dist, the dist process chokes if manifest adds any filenames with spaces.
Spaces and tabs are simply not allowed in MANIFEST filenames. This isn't a M::B limitation, it's a limitation of the file format and documented in ExtUtils::Manifest. Perhaps we should croak/complain earlier in the process (e.g. during the 'manifest' action rather than in the 'dist' action). -Ken
Actually, I misspoke - we actually call ExtUtils::Manifest::mkmanifest() to create the MANIFEST file, so the bug must reside there. It's creating MANIFEST files that are invalid. I suggest re- opening the bug in that queue (the ExtUtils-MakeMaker queue) and I'll mark it as "rejected" in our queue. -Ken
Subject: Re: [rt.cpan.org #18186] Build manifest and filenames with spaces
Date: Fri, 17 Mar 2006 15:51:49 -0500
To: Ken_Williams via RT <bug-Module-Build [...] rt.cpan.org>
From: William McKee <william [...] knowmad.com>
On Wed, Mar 15, 2006 at 02:45:11PM -0500, Ken_Williams via RT wrote: Show quoted text
> Actually, I misspoke - we actually call ExtUtils::Manifest::mkmanifest() to create the MANIFEST > file, so the bug must reside there. It's creating MANIFEST files that are invalid. I suggest re- > opening the bug in that queue (the ExtUtils-MakeMaker queue) and I'll mark it as "rejected" in > our queue.
Actually there's already a report that's filed on this matter with ExtUtils[1]. It seems that Schwern is waiting for a patch. I may get around to that if it gets to be too painful. Thanks, William [1] http://rt.cpan.org/Public/Bug/Display.html?id=12460 -- Knowmad Services Inc. http://www.knowmad.com