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=c711bc865b7196fada1cc02b9b80bf9110b9a889 commit c711bc865b7196fada1cc02b9b80bf9110b9a889 Author: Guillem Jover <guil...@debian.org> AuthorDate: Sat Feb 3 17:58:04 2024 +0100 dpkg-buildpackage: Fix the debian/rules executable check to respect -R When the user has specified a different rules file via the -R option, we should try to use that file for the check that makes sure it is executable. But, unfortunately, because the -R option might be passed other program and arguments to interpret the file or something else entirely, we can only be sure this is a single file to be executed if it contains a single element. This means the check might get disabled when passing more than one argument, but because it's a non-fatal warning, it should be fine. --- scripts/dpkg-buildpackage.pl | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/dpkg-buildpackage.pl b/scripts/dpkg-buildpackage.pl index a91540bc2..5a9152d0e 100755 --- a/scripts/dpkg-buildpackage.pl +++ b/scripts/dpkg-buildpackage.pl @@ -615,9 +615,9 @@ if ($sanitize_env) { run_hook('init'); -if (not -x 'debian/rules') { - warning(g_('debian/rules is not executable; fixing that')); - chmod(0755, 'debian/rules'); # No checks of failures, non fatal +if (@debian_rules == 1 && ! -x $debian_rules[0]) { + warning(g_('%s is not executable; fixing that'), $debian_rules[0]); + chmod(0755, $debian_rules[0]); # No checks of failures, non fatal } if (scalar @call_target == 0) { -- Dpkg.Org's dpkg
