I apoligise for being asleep the first time I reviewed this patch. 

The use of PERL_DL_NONLAZY in a conditional is broken. The confmodule
sets PERL_DL_NONLAZY to work around a bug in perl; it can be set
externally for other reasons; depending on it to decide whether to mess
with file descriptors links two unrelated things and will surely cause
bugs. Besides, it's ugly and impossible to understand (at least,
impossible for me to understand why you'd ever consider using it in the
conditional).

I don't understand the 9>&- part.

Still highly sceptical that this change wouldn't break something.

-- 
see shy jo

Attachment: signature.asc
Description: Digital signature

Reply via email to