Bug#684290: libapache2-mod-perl2: SizeLimite requires Linux::Pid

[Dean Hamstead]

Im not sure either way. I just wanted to bring the error to your 
attention.


Dean

On 2012-08-10 20:30, Damyan Ivanov wrote:
> -=| Dean Hamstead, 08.08.2012 13:34:33 +0000 |=-
>> Package: libapache2-mod-perl2
>> Version: 2.0.7-1
>> Severity: normal
>>
>> pretty straight forward, Linux::pid is needed but doesnt seem to
>> exist in debian.
>>
>> root at webgui:~# perl -e 'use Apache2::SizeLimit'
>> You must install Linux::Pid for Apache::SizeLimit to work on your 
>> platform. at /usr/lib/perl5/Apache/SizeLimit/Core.pm line 128.
>> BEGIN failed--compilation aborted at 
>> /usr/lib/perl5/Apache/SizeLimit/Core.pm line 171.
>
> I wonder if packaging Linux::Pid is the right way to go. Its tests
> say:
> t/threaded.t .. 1..9
> ok 1 - use Linux::Pid;
> ok 2 - $pid defined
> ok 3 - $ppid defined
> ok 4 - $pid2 defined
> ok 5 - $ppid2 defined
> ok 6 - $pid non null
> ok 7 - $pid2 non null
> ok 8 # skip thread model is NPTL 2.13
> #
> ok 9 # skip thread model is NPTL 2.13
> #
> ok
>
> So it seems that the reason for Linux::Pid's existence is not really
> useful.
>
> Some (perhaps naive)  testing shows this:
>  $ perl -Mthreads -MPOSIX=getppid,getpid -Mfeature=say -Mstrict -we
> say "$$ ".POSIX::getppid()." ".POSIX::getpid(); threads->create(sub
> {say "T: $$ ".POSIX::getppid()." ".POSIX::getpid()})->join;'
>  31523 4122 31523
>  T: 31523 4122 31523
> i.e. PID and PPID don't change in the thread. I guess this is why
> Apache2::SizeLimit wants Linux::Pid, but it seems that the tests
> suggest it won't be useful either. Indeed, commenting the conditional
> skip makes the tests fail.
>
> Any suggestions?