Hi Skimming through this thread as a result of your mail, I noticed the earlier patch replacing the || 1 with defined ... ? ... and was going to point out that since 2007 perl's had the // operator for this sort of thing, which should mean that this would do the trick these days:
my $norecommends = $ENV{'NORECOMMENDS'} // 1; but then I noticed that the latest patch includes this: > -my $norecommends = read_env('NORECOMMENDS', 1); > +my $norecommends = (defined $ENV{'NORECOMMENDS'} ? $ENV{'NORECOMMENDS'} : 1); which replaces the use of read_env() with the earlier fix, which strikes me as a backwards step, given that read_env is doing exactly what's needed here: =-=-=- sub read_env { my $env_var = shift; my $default = shift; if (exists($ENV{$env_var})) { return $ENV{$env_var}; } # else return $default; } =-=-=- Cheers, Phil. -- |)| Philip Hands [+44 (0)20 8530 9560] HANDS.COM Ltd. |-| http://www.hands.com/ http://ftp.uk.debian.org/ |(| Hugo-Klemm-Strasse 34, 21075 Hamburg, GERMANY