[debhelper-devel] [debhelper] 04/05: dh_builddeb: Pass --root-owner-group to dpkg-deb under R³ in null-owner mode

[Niels Thykier]

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

nthykier pushed a commit to branch support-rrr
in repository debhelper.

commit 49e71084bbe64dacd17db38c8014c4b427df5ba7
Author: Niels Thykier <niels at thykier.net>
Date:   Sun Oct 15 14:24:11 2017 +0000

    dh_builddeb: Pass --root-owner-group to dpkg-deb under R³ in null-owner mode
    
    Signed-off-by: Niels Thykier <niels at thykier.net>
---
 debian/changelog |  2 ++
 dh_builddeb      | 10 ++++++++--
 2 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index b760df0..7a0e1e7 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -22,6 +22,8 @@ debhelper (10.9.3) UNRELEASED; urgency=medium
     package do not need any non-standard static ownership in the
     binary packages as all ownership values will be reset to
     "root:root".
+  * dh_builddeb: Pass --root-owner-group to dpkg-deb when the
+    rootless-build-nulling-ownership sequence is activated.
 
  -- Niels Thykier <niels at thykier.net>  Sat, 14 Oct 2017 11:18:19 +0000
 
diff --git a/dh_builddeb b/dh_builddeb
index a412789..6636f4f 100755
--- a/dh_builddeb
+++ b/dh_builddeb
@@ -25,6 +25,9 @@ and L<dh_gencontrol(1)> have prepared them.
 It supports building multiple binary packages in parallel, when enabled by
 DEB_BUILD_OPTIONS.
 
+Under the B<rootless-build-nulling-ownership> sequence, B<dh_builddeb>
+will pass B<--root-owner-group> to L<dpkg-deb(1)>.
+
 =head1 OPTIONS
 
 =over 4
@@ -99,6 +102,8 @@ sub build_and_rename_deb {
 }
 
 my @items;
+my @dpkg_options;
+push(@dpkg_options, '--root-owner-group') if Debian::Debhelper::Dh_Lib::_requested_rrr_state() eq 'null-owner';
 
 for my $package (@{$dh{DOPACKAGES}}) {
 	push(@items, [$package, 0]);
@@ -123,7 +128,7 @@ on_items_in_parallel(\@items, sub {
 
 		if ($dbgsym) {
 			my $dbgsym_tmpdir = "debian/.debhelper/${package}/dbgsym-root";
-			doit("dpkg-deb", @{$dh{U_PARAMS}},
+			doit("dpkg-deb", @dpkg_options, @{$dh{U_PARAMS}},
 				"--build", $dbgsym_tmpdir, $dh{DESTDIR});
 			next;
 		}
@@ -139,11 +144,12 @@ on_items_in_parallel(\@items, sub {
 			}
 		}
 		if (! is_udeb($package)) {
-			doit("dpkg-deb", @{$dh{U_PARAMS}}, "--build", $tmp, $dh{DESTDIR}.$dh{FILENAME});
+			doit("dpkg-deb", @dpkg_options, @{$dh{U_PARAMS}}, "--build", $tmp, $dh{DESTDIR}.$dh{FILENAME});
 		}
 		else {
 			my $filename=$dh{FILENAME};
 			my @cmd = qw(dpkg-deb -z6 -Zxz -Sextreme);
+			push(@cmd, @dpkg_options);
 			push(@cmd, @{$dh{U_PARAMS}}) if $dh{U_PARAMS};
 			push(@cmd, '--build', $tmp);
 			if (! $filename) {

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