[mrtg-developers] Fix for $SIG{__DIE__} handler in MRTG_lib.pm

[Steven Bakker]

On Thu, 2008-03-20 at 11:47 +0100, Tobias Oetiker wrote:

> neat trick with $^S ... will add ... will leave the local SIG in
> anyway since I read somewhere that it is not good to have a __SIG__
> handler in eval anyway ...

Yeah, better safe than sorry. ;-)

The "perlvar" man page also has something to say about the use of $^S
and $SIG{__DIE__}. They advise the use of an END{} block to catch a die,
but I doubt whether that is a good idea in this case. You typically
close down and clean up in an END{} block, not do object creation and
file opening. YMMV.

> you may want to log a bug versus the debian packet as well to get
> the fix in there too before the next debian release :-)
> 
> note that SNMP_Session 1.12 addes the localized sighanders as well
> ...

Yes. In fact, it was already in 1.08 (AFAICS). Unfortunately, we have a
schizophrenic Debian install (it's still Sarge, but with tons of
backports). I'm pretty sure the "official" Debian repos contain versions
of SNMP_Session and MRTG that are properly synchronised.

Cheers,
Steven