Skip Menu |

This queue is for tickets about the CPANPLUS CPAN distribution.

Report information
The Basics
Id: 29904
Status: resolved
Priority: 0/
Queue: CPANPLUS
People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 0.82
Fixed in: (no value)

History Show all quoted text
  Wed Oct 10 15:07:18 2007 SREZIC [...] cpan.org - Ticket created
Subject: make test should not use sudo
(Really testing against 0.83_02, but rt did not let me to chose this version) The 20_CPANPLUS-Dist-MM.t test under some circumstances uses sudo. This makes the use of automated tests or installs difficult. But more important, just the existance of sudo in the PATH does not necessarily mean that the user has the right to use sudo. /usr/local/etc/sudoers might give him only limited or no rights at all. So I think it's best to remove the test or make it conditional, e.g. by Makefile.PL command line option or environment variable. Regards, Slaven
  Tue Oct 16 05:46:52 2007 kane [...] cpan.org - Correspondence added
On Wed Oct 10 15:07:18 2007, SREZIC wrote: Show quoted text
> (Really testing against 0.83_02, but rt did not let me to chose this > version) > > The 20_CPANPLUS-Dist-MM.t test under some circumstances uses sudo. This > makes the use of automated tests or installs difficult. But more > important, just the existance of sudo in the PATH does not necessarily > mean that the user has the right to use sudo.
Thanks for reporting; the test suite now skips the install tests if sudo is configured, basically based on the logic that you outlined.
  Tue Oct 16 05:47:03 2007 The RT System itself - Status changed from 'new' to 'open'
  Tue Oct 16 05:47:18 2007 kane [...] cpan.org - Status changed from 'open' to 'resolved'