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=91f68ddf6537e28bf2283a983d64f42494652b96 commit 91f68ddf6537e28bf2283a983d64f42494652b96 Author: Guillem Jover <guil...@debian.org> AuthorDate: Sun Aug 17 05:12:06 2025 +0200 Dpkg::Source::Package: Remove debian/rules fixer at extract time This is already handled at build time if necessary by the relevant build driver (Dpkg::BuildDriver::DebianRules). This means we do not emit spurious warnings for source packages using a build driver that does not require a debian/rules file. The alternative to call the build driver pre-check code here, seems architecturally wrong, because we are not building anything yet. This is also a check for a bug that needs fixing, so reporting it later should be fine as well. The only scenarios that would be affected are explicit calls to debian/rules before dpkg-buildpackage has been called (which we are trying to make an unsupported interface anyway), or iff people want to use a read-only source tree with an overlayfs on top to avoid modifications to files. Both of which still go back to supporting a buggy package that needs to be fixed anyway. Closes: #1078764 --- scripts/Dpkg/Source/Package.pm | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/scripts/Dpkg/Source/Package.pm b/scripts/Dpkg/Source/Package.pm index 8d2edd98d..a57e50be9 100644 --- a/scripts/Dpkg/Source/Package.pm +++ b/scripts/Dpkg/Source/Package.pm @@ -40,7 +40,6 @@ our @EXPORT_OK = qw( ); use Exporter qw(import); -use POSIX qw(:errno_h :sys_wait_h); use Carp; use File::Temp; use File::Copy qw(cp); @@ -622,22 +621,6 @@ sub extract { $self->{format}->save($format_file); } } - - # Make sure debian/rules is executable - my $rules = File::Spec->catfile($newdirectory, 'debian', 'rules'); - my @s = lstat($rules); - if (not scalar(@s)) { - unless ($! == ENOENT) { - syserr(g_('cannot stat %s'), $rules); - } - warning(g_('%s does not exist'), $rules) - unless $self->{options}{skip_debianization}; - } elsif (-f _) { - chmod($s[2] | 0o111, $rules) - or syserr(g_('cannot make %s executable'), $rules); - } else { - warning(g_('%s is not a plain file'), $rules); - } } sub do_extract { -- Dpkg.Org's dpkg
