Bug #81355 for Test-DistManifest: filenames with spaces are poorly handled

Thu Nov 22 19:29:07 2012 ether [...] cpan.org - Ticket created

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

Thu Nov 22 21:54:40 2012 JAWNSY [...] cpan.org - Correspondence added

From:

JAWNSY [...] cpan.org

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

Thu Nov 22 21:54:41 2012 The RT System itself - Status changed from 'new' to 'open'

Fri Nov 23 14:27:30 2012 ether [...] cpan.org - Correspondence added

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>

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.

Wed Apr 03 08:06:52 2019 abraxxa [...] cpan.org - Correspondence added

I stumbled over this issue as well with font license filenames that have whitespace. Any chance this gets resolved?