Skip Menu |

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

Report information
The Basics
Id: 4124
Status: resolved
Priority: 0/
Queue: Module-Build
People
Owner: Nobody in particular
Requestors: domm [...] zsi.at
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 0.20_02
Fixed in: (no value)

History Show all quoted text
  Thu Oct 16 05:59:45 2003 Guest - Ticket created
Subject: Build dist creates tarballs with bad permissions
While playing around with CPANTS data I found something interesting: There are 108 dists on CPAN using Build.PL 20 of those have bad permissions problmes. I.e. after extracting the tarball, there are files with permissions other than rw-r--r-- I quickly made a test dist, did ./Build dist unpacked the generated tarball, and indeed: all files are with permissions r--r--r-- (which makes them annoying to delete) I grabed Module-Build-0.21, tested again: same problem. Module_Build-0.21 itself suffers from the bug, too. Maybe this is caused by my OS / setup, but I tested it on a Debian and a Redhat box, both with the same problems... So this seems like a bug to me. I did't look at the sources (lack of time), so no patch - and I'm still not 100% sure wheater this is a real bug. I attached a list of dists that seem to suffer from this problem...
  Thu Oct 16 08:42:12 2003 kwilliams [...] cpan.org - Correspondence added
I believe the permissions are all set by ExtUtils::Manifest::manicopy(), and I thought it was actually proper to make the distribution files read-only. Can you elaborate on why it's a problem? -Ken
  Thu Oct 16 08:42:13 2003 kwilliams [...] cpan.org - Status changed from 'new' to 'open'
  Thu Oct 16 22:23:38 2003 kwilliams [...] cpan.org - Correspondence added
Aha, yeah, you're right. It looks like if you use the 'ln' method of copying for manicopy(), then you get different permissions than if you use the 'cp' method. This is because the cp() function in ExtUtils::Manifest explicitly removes write-permission. Grr. I wonder if we should just avoid using manicopy() here, or maybe try to co-opt its cp() method so we don't do the permissions changes.
  Sun Oct 19 13:15:50 2003 kwilliams [...] cpan.org - Correspondence added
It turned out to be fairly easy to avoid using manicopy(), so this bug is fixed in CVS. Now we just use our own copy_if_modified() method. -Ken
  Sun Oct 19 13:15:51 2003 kwilliams [...] cpan.org - Status changed from 'open' to 'resolved'
  Thu Oct 30 18:39:25 2003 mschwern [...] cpan.org - Correspondence added
[KWILLIAMS - Sun Oct 19 13:15:50 2003]: Show quoted text
> It turned out to be fairly easy to avoid using manicopy(), so this bug > is fixed in CVS. > Now we just use our own copy_if_modified() method.
I've just fixed this in ExtUtils::Manifest in the snapshot it'll be in 6.18. Oh, and THANKS FOR FORWARDING THIS BUG ON! ;P
  Thu Oct 30 18:39:26 2003 The RT System itself - Status changed from 'resolved' to 'open'
  Tue Nov 04 10:02:14 2003 kwilliams [...] cpan.org - Status changed from 'open' to 'resolved'