Skip Menu |
 

This queue is for tickets about the Net-Server CPAN distribution.

Report information
The Basics
Id: 14155
Status: resolved
Priority: 0/
Queue: Net-Server

People
Owner: Nobody in particular
Requestors: lupe [...] lupe-christoph.de
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.88
Fixed in: (no value)



Subject: Path name for restart is mangled
Download (untitled) / with headers
text/plain 334b
This line $script = $ENV{'PWD'} .'/'. $0 if $ENV{'PWD'}; is probably due to this "fix": Store $ENV{PWD} as part of script name for HUP'ing (Russel Pettway) It is plain Stoopid(tm) because it generates a path like this: /home/lupe//usr/local/sbin/munin-node The fix is left as an exercise for the reader :-P Lupe Christoph
Download (untitled) / with headers
text/plain 710b
[LUPE - Sun Aug 14 05:22:53 2005]: Show quoted text
> This line > $script = $ENV{'PWD'} .'/'. $0 if $ENV{'PWD'}; > is probably due to this "fix": > Store $ENV{PWD} as part of script name for HUP'ing (Russel > Pettway) > It is plain Stoopid(tm) because it generates a path like this: > /home/lupe//usr/local/sbin/munin-node > The fix is left as an exercise for the reader :-P > > Lupe Christoph
It's really bad, even when there is no absolute path to start with, after the first HUP there is, and the script will fail after the second HUP. Cd'ing to / does not help either, because the path accumulates one '//' per HUP, evantually exceeding the maximum path length. And it clobbers the ps output until then.


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.