XPost: linux.debian.devel.release
Package: release.debian.org
Severity: normal
User:
release.debian.org@packages.debian.org
Usertags: unblock
X-Debbugs-Cc:
devscripts@packages.debian.org
Control: affects -1 + src:devscripts
Please unblock package devscripts for some rather trivial changes:
devscripts (2.25.13) unstable; urgency=medium
[ Andreas Tille ]
* debcommit: Prefer Git over SVN or CVS. Closes: #1106159
[ Jochen Sprickerhof ]
* scripts/devscripts/proxy.py: ignore zero size files in cache.
[ Holger Levsen ]
* mass-bug: fix typo in manpage, thanks to Julien Plissonneau Duquène.
* Update French translation, thanks to Julien Plissonneau Duquène/MR503.
* Update Portuguese translation, thanks to Américo Monteiro.
Closes: #1105959
* Update po4a.
-- Holger Levsen <
holger@debian.org> Thu, 22 May 2025 16:16:30 +0200
*However*, the debdiff to the version in testing is:
$ debdiff devscripts_2.25.12.dsc devscripts_2.25.13.dsc | diffstat
debian/changelog | 17
po4a/add_fr/translator_dbk.add | 7
po4a/add_fr/translator_man.add | 13
po4a/add_fr/translator_pod.add | 2
po4a/po/de.po | 842 +++++-----
po4a/po/devscripts.pot | 4
po4a/po/fr.po | 6072 ++++++++++++++++++++++++++++++++++++-------------------------------------
po4a/po/pt.po | 936 +++++------
scripts/build/lib/devscripts/__init__.py | 1
scripts/build/lib/devscripts/control.py | 307 ---
scripts/build/lib/devscripts/logger.py | 75
scripts/build/lib/devscripts/proxy.py | 232 --
scripts/build/scripts-3.11/deb-janitor | 320 ---
scripts/build/scripts-3.11/debbisect | 1093 -------------
scripts/build/scripts-3.11/debdiff-apply | 382 ----
scripts/build/scripts-3.11/debootsnap | 597 -------
scripts/build/scripts-3.11/reproducible-check | 266 ---
scripts/build/scripts-3.11/sadt | 647 -------
scripts/build/scripts-3.11/suspicious-source | 177 --
scripts/build/scripts-3.11/wrap-and-sort | 514 ------
scripts/build/scripts-3.12/deb-janitor | 322 ---
scripts/build/scripts-3.12/debbisect | 1093 -------------
scripts/build/scripts-3.12/debdiff-apply | 382 ----
scripts/build/scripts-3.12/debftbfs | 389 ----
scripts/build/scripts-3.12/debootsnap | 638 -------
scripts/build/scripts-3.12/reproducible-check | 266 ---
scripts/build/scripts-3.12/sadt | 647 -------
scripts/build/scripts-3.12/suspicious-source | 177 --
scripts/build/scripts-3.12/wrap-and-sort | 514 ------
scripts/debcommit.pl | 8
scripts/devscripts/proxy.py | 2
scripts/mass-bug.pl | 2
32 files changed, 3936 insertions(+), 13008 deletions(-)
while neither scripts/build/scripts-3.1(1|2) should exist in the source package.
So I've checked all *my* devscripts uploads to unstable (going back to
2.24.2 uploaded 2024-10-31) and the all included scripts/build/scripts-3.1*, and 2.24.1 uploaded before also had that.
So I checked the version in stable, and that doesn't have scripts/build/scripts-3.1*.
AFAIK there's also no impact on the binary packages, as the debdiff
between the binary devscripts package in testing and unstable is this:
(run on the actually binaries from ftp.d.o)
$ debdiff devscripts_2.25.12_all.deb devscripts_2.25.13_all.deb
[The following lists of changes regard files as different if they have different names, permissions or owners.]
Files in second .deb but not in first
-------------------------------------
-rw-r--r-- root/root /usr/lib/python3/dist-packages/devscripts-2.25.13.egg-info/PKG-INFO
-rw-r--r-- root/root /usr/lib/python3/dist-packages/devscripts-2.25.13.egg-info/dependency_links.txt
-rw-r--r-- root/root /usr/lib/python3/dist-packages/devscripts-2.25.13.egg-info/top_level.txt
Files in first .deb but not in second
-------------------------------------
-rw-r--r-- root/root /usr/lib/python3/dist-packages/devscripts-2.25.12.egg-info/PKG-INFO
-rw-r--r-- root/root /usr/lib/python3/dist-packages/devscripts-2.25.12.egg-info/dependency_links.txt
-rw-r--r-- root/root /usr/lib/python3/dist-packages/devscripts-2.25.12.egg-info/top_level.txt
Control files: lines which differ (wdiff format) ------------------------------------------------
Installed-Size: [-2697-] {+2700+}
Version: [-2.25.12-] {+2.25.13+}
So how did this happen? It's rather easy to reproduce: normally, I build a source package with debuild -S, then I test build this with pbuilder and
then upload. If I however once build devscrips directly, that is without pbuilder but directly with debuild or debuild -b, all following source
builds get those files in scripts/build/scripts-3.1*.
I suppose the fix for this is to cleanup scripts/build with the clean target.
[ Reason ]
All the above. Especially those 3 fixes.
[ Impact ]
Those 3 fixes plus two translation updates will be missing from trixie, also we'll ship an unclean source package.
[ Tests ]
autopkgtests, been in sid for 3 days and thus used by many DDs, plus been installed on the worker nodes for reproduce.debian.net since 3 days as well.
[ Risks ]
not really, those are all rather trivial changes.
[ Checklist ]
[x] all changes are documented in the d/changelog
[x] I reviewed all changes and I approve them
[x] attach debdiff against the package in testing
[ Other info ]
I'll attach the compressed debdiff as a reply to this bug to make sure the bug gets posted on the list. What's the max size for attachments again? Should I include the full debdiff or can I filter translattions to reduce size?
I think maybe those questions could be answered in the "Unblock requests FAQ" on release.d.o :)
The code diff is:
--- devscripts-2.25.12/scripts/debcommit.pl 2025-05-08 01:26:41.000000000 +0200
+++ devscripts-2.25.13/scripts/debcommit.pl 2025-05-22 12:00:39.000000000 +0200
@@ -453,6 +453,10 @@
return "darcs";
}
}
+ if (-e ".git") {
+# With certain forms of git checkouts, .git can be a file instead of a directory
+ return "git";
+ }
if (-d ".svn") {
return "svn";
}
@@ -472,10 +476,6 @@
if (-d ".bzr") {
return "bzr";
}
- if (-e ".git") {
-# With certain forms of git checkouts, .git can be a file instead of a directory
- return "git";
- }
if (-d ".hg") {
return "hg";
}
--- devscripts-2.25.12/scripts/devscripts/proxy.py 2025-05-08 01:26:41.000000000 +0200
+++ devscripts-2.25.13/scripts/devscripts/proxy.py 2025-05-22 12:00:42.000000000 +0200
@@ -63,7 +63,7 @@
path = cachedir / sanitizedpath
# just send back to client
- if path.exists():
+