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

guillem pushed a commit to branch main
in repository dpkg.

View the commit online:
https://git.dpkg.org/cgit/dpkg/dpkg.git/commit/?id=52d285fecbec8ba3cbd8255ef4de2be392d1e0de

commit 52d285fecbec8ba3cbd8255ef4de2be392d1e0de
Author: Guillem Jover <[email protected]>
AuthorDate: Sat Jun 4 05:48:21 2022 +0200

    Dpkg::Source::Package::V2: Always fix the permissions for upstream tarballs
    
    The change to fix the directory traversal for source package unpacks
    with in-place extractions of the debian.tar archive modified the way
    that last extraction was done, to extract it also out-of-place, then
    move the result into the destination. This had the consequence of
    no longer fixing the permissions for the entire source tree, and instead
    only for the debian/ directory. The previous calls for the orig tarballs
    were not fixing up the permissions to avoid duplicating work, which
    meant that now these did not get fixed any longer.
    
    Remove the options that avoid fixing the permissions for all calls,
    restoring the previous behavior.
    
    Fixes: commit 7a6c03cb34d4a09f35df2f10779cbf1b70a5200b
    Closes: #1012195
    Stable-Candidates: 1.18.x 1.19.x 1.20.x
---
 scripts/Dpkg/Source/Package/V2.pm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/Dpkg/Source/Package/V2.pm 
b/scripts/Dpkg/Source/Package/V2.pm
index 1167625d7..68a967168 100644
--- a/scripts/Dpkg/Source/Package/V2.pm
+++ b/scripts/Dpkg/Source/Package/V2.pm
@@ -218,7 +218,7 @@ sub do_extract {
     # Extract main tarball
     info(g_('unpacking %s'), $tarfile);
     my $tar = Dpkg::Source::Archive->new(filename => "$dscdir$tarfile");
-    $tar->extract($newdirectory, no_fixperms => 1,
+    $tar->extract($newdirectory,
                   options => [ '--anchored', '--no-wildcards-match-slash',
                                '--exclude', '*/.pc', '--exclude', '.pc' ]);
     # The .pc exclusion is only needed for 3.0 (quilt) and to avoid
@@ -239,7 +239,7 @@ sub do_extract {
             erasedir("$newdirectory/$subdir");
         }
         $tar = Dpkg::Source::Archive->new(filename => "$dscdir$file");
-        $tar->extract("$newdirectory/$subdir", no_fixperms => 1);
+        $tar->extract("$newdirectory/$subdir");
     }
 
     # Stop here if debianization is not wanted

-- 
Dpkg.Org's dpkg

Reply via email to