On Wed, Oct 17, 2012 at 05:20:00PM +0300, Niko Tyni wrote: > tag 688842 patch > thanks > > On Wed, Oct 03, 2012 at 11:41:30AM +0300, Niko Tyni wrote: > > On Wed, Sep 26, 2012 at 09:12:00AM +0300, Niko Tyni wrote: > > > > When the site dirs don't exist, the CPAN shell interactive configuration > > > states > > > Warning: You do not have write permission for Perl library directories. > > > and proceeds to suggest local::lib for installation, which is not the > > > optimal suggestion for root users. > > > My current thinking is that a minimally invasive fix would be a loop > > that tests the first existing directory up the tree for writability. > > This way we can leave the mkdir part (along with umask settings, > > see debian/patches/debian/writable_site_dirs.diff) for later. > > Looking at the issue, I realized that we don't want to test the > writability of the core dirs (install*lib) on Debian at all. We're > configuring CPAN to pass INSTALLDIRS=site precisely to keep it from > touching the core directories. > > This way root and non-root users who have write access to /usr/local > will get the right default (system-wide install), and the rest will be > prompted about using local::lib or sudo. > > Here's my first take on a patch. It seems to work, but eyeballs > and testing would be welcome.
Nice work - the patch looks fine to me and I've tested it appears to fix the problem discussed on this report (I actually tested ntyni/for-wheezy at 154f2c7768c838e1b278a73eabec545865df0508). -- Dominic Hargreaves | http://www.larted.org.uk/~dom/ PGP key 5178E2A5 from the.earth.li (keyserver,web,email) -- To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
