Bug#1032622: cups-ipp-utils: Please enable translation of ippeveps(7)

[Helge Kreutzmann]

Package: cups-ipp-utils
Version: 2.4.2-2
Severity: wishlist
Tags: l10n patch

I noticed a new manpage, namely ippeveps(7), is shipped in
cups-ipp-utils. With the follwoing two lines it's translation
should be enabled:

debian/manpage-po4a/cups.cfg
Add the follwing line:
[type: man] ippeveps.7 $lang:$lang/ippeveps.7

In debian/rules:
MANPAGES_L10N_CUPS_IPP_UTILS=man1/ippeveprinter.1 man1/find.1 man1/ippserver.1 man1/ipptool.1 man5/ipptoolfile.5
MANPAGES_L10N_CUPS_IPP_UTILS=man1/ippeveprinter.1 man1/find.1 man1/ippserver.1 man1/ipptool.1 man5/ipptoolfile.5 man7/ippeveps.7

Extra great would be, if could you send the updated de.po, fr.po and
pt.po to the previous translators asking for input (I can do this, if
you want) and perform another upload targetting bookworm with the update.

Thanks for considering!

-- System Information:
Debian Release: bookworm/sid
APT prefers testing
APT policy: (500, 'testing')
Architecture: amd64 (x86_64)

Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8) (ignored: LC_ALL set to de_DE.UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages cups-ipp-utils depends on:
ii libavahi-client3 0.8-9
ii libavahi-common3 0.8-9
ii libc6 2.36-8
ii libcups2 2.4.2-2
ii libpam0g 1.5.2-6

Versions of packages cups-ipp-utils recommends:
pn avahi-daemon <none>

cups-ipp-utils suggests no packages.

-- no debconf information

--
Dr. Helge Kreutzmann deb...@helgefjell.de
Dipl.-Phys. http://www.helgefjell.de/debian.php
64bit GNU powered gpg signed mail preferred
Help keep free software "libre": http://www.ffii.de/

[Thorsten Alteholz]

Hi Helge,

On 10.03.23 12:14, Helge Kreutzmann wrote:
> Extra great would be, if could you send the updated de.po, fr.po and
> pt.po to the previous translators asking for input (I can do this, if
> you want)

it would be great if you could do this for me.

Thanks!
 Thorsten

[Helge Kreutzmann]

Hello Thorsten,

Fine, no problem. I'll enable the translation in the repository and
request the updates.

What ETA would you like me to send, i.e. when would the upload happen?

Greetings

Helge

[Thorsten Alteholz]

Hi Helge,

On Sat, 11 Mar 2023, Helge Kreutzmann wrote:
> What ETA would you like me to send, i.e. when would the upload happen?

I don't have any further plans with the package. As long as there is no
new RC bug, I will upload when you are ready.

Thorsten

[Helge Kreutzmann]

Hello Thorsten,

I'll check with the French team and come back you.

Thanks for your support!

Greetings

Helge

[Helge Kreutzmann]

Hello Thorsten,
On Sun, Mar 12, 2023 at 01:00:33AM +0000, Thorsten Alteholz wrote:

I asked the French and portuguese translators if they could update
their files until next weekend.

Next I tried add the man page to the package. While doing so, I
noticed two things:

a) There are more translated man pages not shipped.
b) I don't understand the CUPS build system.

For a) I tried the following:
diff --git a/debian/rules b/debian/rules
index 4f1c8424c..4c306843f 100755
--- a/debian/rules
+++ b/debian/rules
@@ -72,12 +72,15 @@ override_dh_auto_configure:
override_dh_auto_install:
dh_auto_install -- install BUILDROOT=$(shell pwd)/debian/tmp

-MANPAGES_L10N_CUPS=man1/cups.1 man5/subscriptions.conf.5 man5/mime.convs.5 man7/filter.7 man8/cupsfilter.8 man8/cups-exec.8 man8/cups-deviced.8 man8/cups-driverd.8 man8/cupsd-helper.8 man8/cupsd-lpd.8
-MANPAGES_L10N_CUPS_SERVER_COMMON=man5/cupsd-logs.conf.5
+MANPAGES_L10N_CUPS=man1/cups.1 man5/subscriptions.conf.5 man5/mime.convs.5 man7/filter.7 man8/cupsfilter.8 man8/cups-exec.8 man8/cups-deviced.8 man8/cups-driverd.8 man8/cupsd-helper.8 man8/cupsd-lpd.8 man8/cups-lpd.8
+MANPAGES_L10N_CUPS_SERVER_COMMON=man5/cupsd-logs.conf.5 man5/cupsd-logs.5
MANPAGES_L10N_CUPS_DAEMON=man5/classes.conf.5 man5/cupsd.conf.5 man5/cups-files.conf.5 man5/cups-snmp.conf.5 man5/mailto.conf.5 man5/mime.types.5 man5/printers.conf.5 man7/backend.7 man7/notifier.7 man8/cupsd.8 man8/cups-snmp.8
MANPAGES_L10N_CUPS_CLIENT=man1/cupstestppd.1 man1/lp.1 man1/lpoptions.1 man1/lppasswd.1 man1/lpstat.1 man1/cancel.1 man5/client.conf.5 man8/cupsenable.8 man8/lpadmin.8 man8/lpinfo.8 man8/lpmove.8 man8/cupsreject.8 man8/cupsdisable.8 man8/cupsaccept.8 man8/cupsctl.8
MANPAGES_L10N_CUPS_BSD=man1/lpr.1 man1/lprm.1 man1/lpq.1 man8/lpc.8
-MANPAGES_L10N_CUPS_IPP_UTILS=man1/ippeveprinter.1 man1/find.1 man1/ippserver.1 man1/ipptool.1 man5/ipptoolfile.5
+#MANPAGES_L10N_CUPS_IPP_UTILS=man1/ippeveprinter.1 man1/find.1 man1/ippserver.1 man1/ipptool.1 man5/ipptoolfile.5
+MANPAGES_L10N_CUPS_IPP_UTILS=man1/ippeveprinter.1 man1/ippfind.1 man1/ipptool.1 man5/ipptoolfile.5 man7/ippeveps.7 man7/ippevepcl.7
+MANPAGES_L10N_CUPS_LIBCUPS2_DEV=man1/cups-config.1
+MANPAGES_L10N_CUPS_PPDC=man1/ppdc.1 man1/ppdhtml.1 man1/ppdi.1 man1/ppdmerge.1 man1/ppdpo.1 man5/ppdcfile.5

override_dh_installman:
dh_installman
+ifneq (,$(filter libcups2-dev,$(shell dh_listpackages)))
+ # Try to install the translated manpages to libcups2-dev
+ set -e; for m in $(MANPAGES_L10N_CUPS_LIBCUPS2_DEV); do \
+ for manp in $$(ls debian/tmp/usr/share/man/*/$$m); do \
+ if [ -r $$manp ]; then \
+ if [ -L $$manp ]; then \
+ ln -sf $$(readlink $$manp) $$(echo $$manp | sed -e 's#^debian/tmp#debian/libcups2-dev#');\
+ else \
+ install -D -m 644 $$manp $$(echo $$manp | sed -e 's#^debian/tmp#debian/libcups2-dev#');\
+ fi; \
+ fi; \
+ done; \
+ done
+endif
+ifneq (,$(filter cups-ppdc,$(shell dh_listpackages)))
+ # Try to install the translated manpages to cups-ppdc
+ set -e; for m in $(MANPAGES_L10N_CUPS_PPDC); do \
+ for manp in $$(ls debian/tmp/usr/share/man/*/$$m); do \
+ if [ -r $$manp ]; then \
+ if [ -L $$manp ]; then \
+ ln -sf $$(readlink $$manp) $$(echo $$manp | sed -e 's#^debian/tmp#debian/cups-ppdc#');\
+ else \
+ install -D -m 644 $$manp $$(echo $$manp | sed -e 's#^debian/tmp#debian/cups-ppdc#');\
+ fi; \
+ fi; \
+ done; \
+ done
+endif

However, running
fakeroot make -f debian/rules binary

ends in:
ln: failed to create symbolic link 'debian/cups-ipp-utils/usr/share/man/de/man7/ippeveps.7': No such file or directory
make[1]: *** [debian/rules:98: override_dh_installman] Error 1
make[1]: Leaving directory '/tmp/cups'
make: *** [debian/rules:24: binary] Error 2

The error message is correct, because
debian/cups-ipp-utils/usr/share/man/de/man7/ippevepcl.7 does
not exist. Even debian/cups-ipp-utils/usr/share/man/de/man7/ does not
exist? However, man/de/ippevepcl.7 exists and looks fine.

This leads me to b)
I've seen quite a few build systems, but I have no idea which magic
fails here.

If you have some spare moments it would be great if you could add the
missing pieces.

Greetings

Helge

[Helge Kreutzmann]

Hello Thorsten,
On Sat, Mar 11, 2023 at 01:53:27PM +0100, Thorsten Alteholz wrote:

> On 10.03.23 12:14, Helge Kreutzmann wrote:
> > Extra great would be, if could you send the updated de.po, fr.po and
> > pt.po to the previous translators asking for input (I can do this, if
> > you want)
>
> it would be great if you could do this for me.

Done. DE, PT and FR are now up to date in the CUPS repository.

Could you kindly enable them[1] and prepare an upload targetted at
bookworm?

Thanks!

Greetings

Helge

[1] I'm a bit lost in your build system, as stated earlier. I believe
the follwing patch is a good start, however it is not sufficient.
Please complete.

--- a/debian/rules
+++ b/debian/rules
@@ -72,12 +72,15 @@ override_dh_auto_configure:
override_dh_auto_install:
dh_auto_install -- install BUILDROOT=$(shell pwd)/debian/tmp

-MANPAGES_L10N_CUPS=man1/cups.1 man5/subscriptions.conf.5 man5/mime.convs.5 man7/filter.7 man8/cupsfilter.8 man8/cups-exec.8 man8/cups-deviced.8 man8/cups-driverd.8 man8/cupsd-helper.8 man8/cupsd-lpd.8
-MANPAGES_L10N_CUPS_SERVER_COMMON=man5/cupsd-logs.conf.5
+MANPAGES_L10N_CUPS=man1/cups.1 man5/subscriptions.conf.5 man5/mime.convs.5 man7/filter.7 man8/cupsfilter.8 man8/cups-exec.8 man8/cups-deviced.8 man8/cups-driverd.8 man8/cupsd-helper.8 man8/cupsd-lpd.8 man8/cups-lpd.8
+MANPAGES_L10N_CUPS_SERVER_COMMON=man5/cupsd-logs.conf.5 man5/cupsd-logs.5
MANPAGES_L10N_CUPS_DAEMON=man5/classes.conf.5 man5/cupsd.conf.5 man5/cups-files.conf.5 man5/cups-snmp.conf.5 man5/mailto.conf.5 man5/mime.types.5 man5/printers.conf.5 man7/backend.7 man7/notifier.7 man8/cupsd.8 man8/cups-snmp.8
MANPAGES_L10N_CUPS_CLIENT=man1/cupstestppd.1 man1/lp.1 man1/lpoptions.1 man1/lppasswd.1 man1/lpstat.1 man1/cancel.1 man5/client.conf.5 man8/cupsenable.8 man8/lpadmin.8 man8/lpinfo.8 man8/lpmove.8 man8/cupsreject.8 man8/cupsdisable.8

+man8/cupsaccept.8 man8/cupsctl.8

[Helge Kreutzmann]

Hello Thorsten,

On Sat, Mar 18, 2023 at 05:45:53PM +0100, Helge Kreutzmann wrote:
> On Sat, Mar 11, 2023 at 01:53:27PM +0100, Thorsten Alteholz wrote:
> > On 10.03.23 12:14, Helge Kreutzmann wrote:
> > > Extra great would be, if could you send the updated de.po, fr.po and
> > > pt.po to the previous translators asking for input (I can do this, if
> > > you want)
> >
> > it would be great if you could do this for me.
>
> Done. DE, PT and FR are now up to date in the CUPS repository.
>
> Could you kindly enable them[1] and prepare an upload targetted at
> bookworm?

Is there anything you need from my side for this upload left?

Applying my patch probably gets 80% of the missing infrastructure where
you can deduce the missing bits and pices easily, I assume?

It would be really great to get these updates and translations into
bookworm.

Thanks a lot!

Greetings

Helge

[Helge Kreutzmann]

Hello Thorsten,

On Sun, Mar 19, 2023 at 07:55:09PM +0100, Helge Kreutzmann wrote:
> On Sat, Mar 18, 2023 at 05:45:53PM +0100, Helge Kreutzmann wrote:
> > On Sat, Mar 11, 2023 at 01:53:27PM +0100, Thorsten Alteholz wrote:
> > > On 10.03.23 12:14, Helge Kreutzmann wrote:
> > > > Extra great would be, if could you send the updated de.po, fr.po and
> > > > pt.po to the previous translators asking for input (I can do this, if
> > > > you want)
> > >
> > > it would be great if you could do this for me.
> >
> > Done. DE, PT and FR are now up to date in the CUPS repository.
> >
> > Could you kindly enable them[1] and prepare an upload targetted at
> > bookworm?
>
> Is there anything you need from my side for this upload left?

Any testing, rationale, … ?

> Applying my patch probably gets 80% of the missing infrastructure where
> you can deduce the missing bits and pices easily, I assume?

If this is too intrusive, or incorrect or for some other reasons not
as you want, you can trim it, up to the initial part of this bug
report.

Of course, from a translators POV it would be great to ship everything
which is translated, there are quite a few hours hard work involved
here by the 3 translation teams.

> It would be really great to get these updates and translations into
> bookworm.

Indeed. I can then request the unblock, which I already did for other
packages as well.

Thanks for handling the upload!

[Thorsten Alteholz]

Hi Helge,

On 25.03.23 08:04, Helge Kreutzmann wrote:

Is there anything you need from my side for this upload left?

just a bit patience ...
I  "adopted" the package, so I am also facing the same problems with the magic translation handling as you ...

Indeed. I can then request the unblock, which I already did for other
packages as well.

This is #1033522, so lets see what the release team thinks about it.

  Thorsten

[Helge Kreutzmann]

Hello Thorsten,

On Sun, Mar 26, 2023 at 07:50:34PM +0200, Thorsten Alteholz wrote:
> On 25.03.23 08:04, Helge Kreutzmann wrote:
> >
> > > Is there anything you need from my side for this upload left?
>
> just a bit patience ...
> I  "adopted" the package, so I am also facing the same problems with the
> magic translation handling as you ...

Apologizes.

> > Indeed. I can then request the unblock, which I already did for other
> > packages as well.
>
> This is #1033522, so lets see what the release team thinks about it.

Given a good rationale and following the requested bug report template
has worked quite well for me for other packages, so I hope they'll do
so here as well.

Thanks for your work!

Greetings

Helge