Skip Menu |
 

This queue is for tickets about the Test-DistManifest CPAN distribution.

Report information
The Basics
Id: 81355
Status: open
Priority: 0/
Queue: Test-DistManifest

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

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



Subject: filenames with spaces are poorly handled
My MANIFEST contains (snipped): 'doc/America Corp Developer Guide ABC API Reference.doc' 'doc/America Corp Developer Guide XYZ API Reference.doc' Test::DistManifest (via the dzil plugin) doesn't like this: t/release-dist-manifest.t ......... Duplicate file or mask 'doc/America at /home/ad/karene/perl5/perlbrew/perls/perl-5.16.0/lib/site_perl/5.16.0/Test/DistManifest.pm line 74. t/release-dist-manifest.t ......... 1/4 # Distribution files are missing in MANIFEST: # doc/America Corp Developer Guide ABC API Reference.doc # doc/America Corp Developer Guide XYZ API Reference.doc # Failed test 'All files are listed in MANIFEST or skipped' # at t/release-dist-manifest.t line 16. # MANIFEST lists the following missing files: # 'doc/America # Failed test 'All files listed in MANIFEST exist on disk' # at t/release-dist-manifest.t line 16. # Looks like you failed 2 tests of 4. t/release-dist-manifest.t ......... Dubious, test returned 2 (wstat 512, 0x200) Failed 2/4 subtests
From: JAWNSY [...] cpan.org
Download (untitled) / with headers
text/plain 502b
Hi Karen, I don't know if the MANIFEST file is designed to handle filenames with spaces. I've seen people use comments, e.g. "README # blah blah" where the trailing part should be treated as a "comment" and ignored. I don't remember if that's why I wrote it that way - from your test it looks like it's just looking at the first part (before whitespace) of each line. I don't know if there's actually a spec for MANIFEST files, I think I just guessed. So I might've done it wrong. Cheers, Jonathan
Subject: Re: [rt.cpan.org #81355] filenames with spaces are poorly handled
Date: Fri, 23 Nov 2012 11:24:12 -0800
To: Jonathan Yu via RT <bug-Test-DistManifest [...] rt.cpan.org>
From: Karen Etheridge <ether [...] cpan.org>
Download (untitled) / with headers
text/plain 427b
On Thu, Nov 22, 2012 at 09:54:41PM -0500, Jonathan Yu via RT wrote: Show quoted text
> I don't know if the MANIFEST file is designed to handle filenames with > spaces.
Show quoted text
> I don't know if there's actually a spec for MANIFEST files, I think I > just guessed. So I might've done it wrong.
I asked around about this, and apparently whatever ExtUtils::Manifest does is "standard". There is also Module::Manifest for poking into an existing file.
Download (untitled) / with headers
text/plain 115b
I stumbled over this issue as well with font license filenames that have whitespace. Any chance this gets resolved?


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.