Package: src:mp4h
Version: 1.3.1-17.1
Severity: important
Tags: ftbfs trixie sid patch
User: [email protected]
Usertags: gettext-0.23
Dear maintainer:
During a rebuild of all packages in unstable using gettext 0.23.1, which is
available in experimental, your package failed to build:
--------------------------------------------------------------------------------
[...]
# Use external PCRE and libtool only
rm -rf src/ltdl.* ltmain.sh pcre
test -d m4 || mkdir m4
for f in ABOUT-NLS config.rpath; do \
cp -f /usr/share/gettext/$f .; \
done
for f in Makefile.in.in Rules-quot boldquot.sed [email protected]
[email protected] insert-header.sin quot.sed remove-potcdate.sin; do \
cp -f /usr/share/gettext/po/$f po/; \
done
cp: cannot stat '/usr/share/gettext/po/insert-header.sin': No such file or
directory
cp: cannot stat '/usr/share/gettext/po/remove-potcdate.sin': No such file or
directory
make[1]: *** [debian/rules:24: override_dh_auto_configure] Error 1
make[1]: Leaving directory '/<<PKGBUILDDIR>>'
make: *** [debian/rules:15: binary] Error 2
dpkg-buildpackage: error: debian/rules binary subprocess returned exit status 2
--------------------------------------------------------------------------------
If required, the full build log is available here:
https://people.debian.org/~sanvila/build-logs/202501/
Notes:
- Many packages may be fixed by using this in configure.in:
AM_GNU_GETTEXT_REQUIRE_VERSION([0.21])
(removing any existing AM_GNU_GETTEXT_VERSION that may exist).
[ Addendum for this bug: Hello Axel! You might want to try
the patch below with maybe some additional fine-tuning.
Not only the above seems to fix the problem, it also allows
several lines in debian/rules to be dropped entirely.
After all, copying files around is what autoreconf is
supposed to do ]
- This bug will be RC once gettext is uploaded for unstable, but
I still don't know when I will be able to do that. If you fix
this bug, it would help to have gettext in unstable sooner.
- If this is really a bug in one of the build-depends, please use
reassign and add an affects on src:mp4h, so that this is still
visible in the BTS web page for this package.
Thanks.
diff --git a/debian/patches/fix-ftbfs-with-gettext-0.23.patch
b/debian/patches/fix-ftbfs-with-gettext-0.23.patch
new file mode 100644
index 0000000..d9858ff
--- /dev/null
+++ b/debian/patches/fix-ftbfs-with-gettext-0.23.patch
@@ -0,0 +1,10 @@
+--- mp4h-1.3.1.orig/configure.ac
++++ mp4h-1.3.1/configure.ac
+@@ -58,6 +58,7 @@ AC_MSG_RESULT([no])
+ AC_SUBST([TIDY])
+
+ ALL_LINGUAS=
++AM_GNU_GETTEXT_REQUIRE_VERSION([0.21])
+ AM_GNU_GETTEXT([external])
+
+
diff --git a/debian/patches/series b/debian/patches/series
index d507022..3d6b463 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -8,3 +8,4 @@ fix-ftbfs-with-hardening-flags.diff
use-system-pcre.diff
reproducible-build.diff
fix-spelling-errors.diff
+fix-ftbfs-with-gettext-0.23.patch
diff --git a/debian/rules b/debian/rules
index aa108b8..f3bfa64 100755
--- a/debian/rules
+++ b/debian/rules
@@ -23,14 +23,6 @@ override_dh_auto_configure:
# Use external PCRE and libtool only
rm -rf src/ltdl.* ltmain.sh pcre
- test -d m4 || mkdir m4
- for f in ${GETTEXT_TOP_FILES}; do \
- cp -f /usr/share/gettext/$$f .; \
- done
- for f in ${GETTEXT_PO_FILES}; do \
- cp -f /usr/share/gettext/po/$$f po/; \
- done
-
dh_autoreconf
dh_auto_configure -- \
