The following commit has been merged in the master branch:
commit 97060a84d8c1b91e771b2dde9415ab8213cdf193
Author: James McCoy <[email protected]>
Date:   Wed Mar 28 06:41:36 2012 -0400

    debsign: Fix quoting in handling of --debs-dir.
    
    Closes: #666048
    Signed-off-by: James McCoy <[email protected]>

diff --git a/debian/changelog b/debian/changelog
index bf8812f..891cd67 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -3,6 +3,7 @@ devscripts (2.11.6) UNRELEASED; urgency=low
   * mk-build-deps: Fix handling of debian/control files.  (Closes: #665496)
   * uscan: Fix formatting in man page.  Thanks to Samuel Bronson for the
     patch.  (Closes: #665325)
+  * debsign: Fix quoting in handling of --debs-dir. (Closes: #666048)
 
  -- James McCoy <[email protected]>  Sat, 24 Mar 2012 15:24:59 -0400
 
diff --git a/scripts/debsign.sh b/scripts/debsign.sh
index 3751ca9..f2ebe65 100755
--- a/scripts/debsign.sh
+++ b/scripts/debsign.sh
@@ -276,7 +276,7 @@ else
 
     # We do not replace this with a default directory to avoid accidentally
     # signing a broken package
-    DEBRELEASE_DEBS_DIR="`echo \"$DEBRELEASE_DEBS_DIR\" | sed -e 's%/\+%/%g; 
s%\(.\)/$%\1%;'`"
+    DEBRELEASE_DEBS_DIR="$(echo "${DEBRELEASE_DEBS_DIR%/}" | sed -e 
's%/\+%/%g')"
     if ! [ -d "$DEBRELEASE_DEBS_DIR" ]; then
        debsdir_warning="config file specified DEBRELEASE_DEBS_DIR directory 
$DEBRELEASE_DEBS_DIR does not exist!"
     fi
@@ -341,7 +341,7 @@ do
                ;;
        --debs-dir)
            shift
-           opt_debsdir="$(echo \"${1%/}\" | sed -e 's%/\+%/%g')"
+           opt_debsdir="$(echo "${1%/}" | sed -e 's%/\+%/%g')"
            ;;
        --no-conf|--noconf)
                echo "$PROGNAME: $1 is only acceptable as the first 
command-line option!" >&2

-- 
Git repository for devscripts


-- 
To unsubscribe, send mail to [email protected].

Reply via email to