[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

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



Hello  Thorsten,
On Sun, Mar 12, 2023 at 01:00:33AM +0000, Thorsten Alteholz wrote:
> 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.

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

-- 
      Dr. Helge Kreutzmann                     debian@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/

Attachment: signature.asc
Description: PGP signature


Reply to: