[devscripts] 01/03: debchange: Verify $opt_vendor is defined before access

James McCoy jamessan at debian.org
Sat Jun 7 19:48:08 UTC 2014


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

jamessan pushed a commit to branch master
in repository devscripts.

commit 14e171bea78bae13717598bac53c58eba4039e28
Author: James McCoy <jamessan at debian.org>
Date:   Sat Jun 7 15:33:41 2014 -0400

    debchange: Verify $opt_vendor is defined before access
    
    Signed-off-by: James McCoy <jamessan at debian.org>
---
 debian/changelog     | 8 ++++++++
 scripts/debchange.pl | 2 +-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/debian/changelog b/debian/changelog
index 0052d28..36d0e04 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,11 @@
+devscripts (2.14.5) UNRELEASED; urgency=medium
+
+  * debchange:
+    + Verify $opt_vendor is defined before trying to use it to avoid a warning
+      about an unitialized variable.
+
+ -- James McCoy <jamessan at debian.org>  Sat, 07 Jun 2014 15:31:54 -0400
+
 devscripts (2.14.4) unstable; urgency=medium
 
   * mk-origtargz: Fix DEP8 test failures due to differences in how the script
diff --git a/scripts/debchange.pl b/scripts/debchange.pl
index 05a961a..96a6fc5 100755
--- a/scripts/debchange.pl
+++ b/scripts/debchange.pl
@@ -462,7 +462,7 @@ if (defined $opt_u) {
 
 # See if we're Debian, Ubuntu or someone else, if we can
 my $vendor;
-if (not $opt_vendor eq '') {
+if (defined $opt_vendor && $opt_vendor) {
     $vendor = $opt_vendor;
 } else {
     if (defined $opt_D) {

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



More information about the devscripts-devel mailing list