tags 677631 + confirmed
quit
Niels Thykier wrote:
> $ env -uHOME dpkg-source -x lintian_2.5.7.dsc
> Use of uninitialized value $ENV{"HOME"} in concatenation (.) or string at
> /usr/share/perl5/Dpkg/Source/Package.pm line 351.
[...]
> Please see attached patch as a possible solution.
[...]
> --- a/scripts/Dpkg/Source/Package.pm
> +++ b/scripts/Dpkg/Source/Package.pm
> @@ -349,7 +349,7 @@ sub check_signature {
> push @exec, "gpg", "--no-default-keyring", "-q", "--verify";
> }
> if (scalar(@exec)) {
> - if (-r "$ENV{'HOME'}/.gnupg/trustedkeys.gpg") {
> + if (defined $ENV{'HOME'} and -r
> "$ENV{'HOME'}/.gnupg/trustedkeys.gpg") {
> push @exec, "--keyring", "$ENV{'HOME'}/.gnupg/trustedkeys.gpg";
For what it's worth,
Reviewed-by: Jonathan Nieder <[email protected]>
The house style would be to use "if ($ENV{'HOME}' and -r ..." but I think
the behavior here is better anyway in odd cases like HOME=0.
Thanks,
Jonathan
--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]