The following commit has been merged in the wheezy branch:
commit 5f7cdfaf8af7c38b65e73e6d5af3a73759ff6eb1
Author: Pablo Oliveira <[email protected]>
Date:   Wed May 29 14:37:43 2013 +0200

    Dpkg::Source::Quilt: Fix chmod() arguments order
    
    In the restore_quilt_backup_files() function, the chmod() call has the
    arguments swapped. This prevents restoring quilt patches that remove
    files.
    
    Closes: #710265
    
    Signed-off-by: Guillem Jover <[email protected]>

diff --git a/debian/changelog b/debian/changelog
index 9126abc..d4a28e5 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -8,6 +8,10 @@ dpkg (1.16.11) UNRELEASED; urgency=low
     code. Thanks to Lincoln Myers <[email protected]> for the patch.
     Closes: #708607
 
+  [ Guillem Jover ]
+  * Fix chmod() arguments order in Dpkg::Source::Quilt. Closes: #710265
+    Thanks to Pablo Oliveira <[email protected]>.
+
  -- Guillem Jover <[email protected]>  Sat, 30 Mar 2013 15:48:52 +0100
 
 dpkg (1.16.10) unstable; urgency=low
diff --git a/scripts/Dpkg/Source/Quilt.pm b/scripts/Dpkg/Source/Quilt.pm
index 5ca9fb7..c4a1bf3 100644
--- a/scripts/Dpkg/Source/Quilt.pm
+++ b/scripts/Dpkg/Source/Quilt.pm
@@ -293,7 +293,7 @@ sub restore_quilt_backup_files {
                 unless (link($_, $target)) {
                     copy($_, $target) ||
                         syserr(_g("failed to copy %s to %s"), $_, $target);
-                    chmod($target, (stat(_))[2]) ||
+                    chmod((stat(_))[2], $target) ||
                         syserr(_g("unable to change permission of `%s'"), 
$target);
                 }
             } else {

-- 
dpkg's main repository


-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to