[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[lintian] 01/01: L::Util: Let clean_env preserve TMPDIR



This is an automated email from the git hooks/post-receive script.

nthykier pushed a commit to branch master
in repository lintian.

commit 2262b384f4da88f5ca02165496805c408370a554
Author: Niels Thykier <niels@thykier.net>
Date:   Mon Oct 19 08:17:30 2015 +0200

    L::Util: Let clean_env preserve TMPDIR
    
    Signed-off-by: Niels Thykier <niels@thykier.net>
---
 debian/changelog    | 5 +++++
 lib/Lintian/Util.pm | 3 ++-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/debian/changelog b/debian/changelog
index 05883c7..4760ad3 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -30,6 +30,11 @@ lintian (2.5.39) UNRELEASED; urgency=medium
   * debian/copyright:
     + [JW] Use HTTPS for lintian.debian.org URLs.
 
+  * lib/Lintian/Util.pm:
+    + [NT] Whitelist TMPDIR in "clean_env", so that man and msgfmt
+      will use the user defined TMPDIR (if any).  Thanks to Bjarni
+      Ingi Gislason Bjarni for the bug report.  (Closes: #801483)
+
   * mail-templates/license-problem-non-free-RFC:
     + [JW] Use HTTPS for debian.org URLs.
 
diff --git a/lib/Lintian/Util.pm b/lib/Lintian/Util.pm
index 0b8fa5a..5b4d3e4 100644
--- a/lib/Lintian/Util.pm
+++ b/lib/Lintian/Util.pm
@@ -888,6 +888,7 @@ The list of whitelisted %ENV variables are:
 
  PATH
  LC_ALL (*)
+ TMPDIR
 
 (*) LC_ALL is a special case as clean_env will change its value to
 either "C.UTF-8" or "C" (if CLOC is given and a truth value).
@@ -896,7 +897,7 @@ either "C.UTF-8" or "C" (if CLOC is given and a truth value).
 
 sub clean_env {
     my ($cloc) = @_;
-    my @whitelist = qw(PATH);
+    my @whitelist = qw(PATH TMPDIR);
     my %newenv
       = map { exists $ENV{$_} ? ($_ => $ENV{$_}) : () } (@whitelist);
     %ENV = %newenv;

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git


Reply to: