Hi there! Till, a big THANK YOU for the detailed explanations, I really appreciated.
On Wed, 16 Feb 2011 00:23:47 +0100, Till Kamppeter wrote: > On 11/05/2010 04:10 AM, Luca Capello wrote: >> On Wed, 13 Oct 2010 02:21:35 +0200, Luca Capello wrote: >>> On Mon, 27 Sep 2010 23:16:49 +0200, Till Kamppeter wrote: >>>> On 09/27/2010 06:53 PM, Luca Capello wrote: >>>>> 4) I am not sure debian/local/ is the right place for non-upstream >>>>> files, but I should admit that this is the first time I heard about >>>>> it and I can not find any documentation about that. Nevermind, I >>>>> have added the two non-upstream PPDs. >>>>> >>>>> BTW, conceptually speaking, Ubuntu debian/rules misses the command to >>>>> compress these two files, given that this action is hidden in the >>>>> 'Add "*cupsFilter" line to accept PDF input data to the PPDs' block. [...] > This makes the PPD files allow PDF as input format. This way the print > queues integrate with the PDF-based printing workflow which is > implemented in Debian and Ubuntu. [...] > https://www.linuxfoundation.org/collaborate/workgroups/openprinting/pdfasstandardprintjobformat Committed: <http://svn.debian.org/viewsvn/foo2zjs?view=rev&revision=257> > I have seen may packages which use debian/local/ to add non-upstream > files. Sorry if I was not clear, but debian/local/ seems perfectly fine with me, the only caveat was that I had never heard about it before :-) Anyway, we do not need it anymore, the latest snapshot packaged (see below) includes the two files above. >>> 5) - debian/foo2zjs.postinst: Automatically update the PPD files for >>> existing queues to the versions supplied with this package. >>> - debian/control: Add dependency on cups and cups-client to ensure that >>> automatic update of the PPDs of existing print queues works. [...] > Therefore I let the package do the dirty work as doing a really > complete update by letting the postinst script updating the PPDs of > the already configured queues in /etc/cups/ppd/. The only > configuration in these files are the default settings (lines starting > with "*Default..."), The rest of the files are printer-specific and no > user configuration. The automatic update is done with CUPS' "lpadmin" > command line utility which preserves the default settings. This way > the user has always the correct PPD for his driver (works on both up- > and downdate of the package) and his default settings do not get > lost. Printing will "just work" for him. That is what I wanted to read, thank you, committed: <http://svn.debian.org/viewsvn/foo2zjs?view=rev&revision=258> <http://svn.debian.org/viewsvn/foo2zjs?view=rev&revision=259> BTW, technically speaking I would be against the dependency on cups and cups-client for the postinst to work, given that there is no problem if the existing queues are not updated. OTOH, foo2zjs does not work at all without cups, so this is harmless and I added the dependencies as well. NB, I re-ordered your quoted text below. >>> 10) - debian/rules: Add /etc/papersize support, and modify all >>> /usr/bin/foo2*-wrapper scripts to handle custom page sizes correctly >>> in >>> the PDF-based printing workflow. [...] >>> - Support for the PDF printing workflow: [...] >>> o Let wrapper scripts read custom page sizes also >>> from the command line and not only from embedded >>> PostScript commands. [...] > There are awkward foo2...-wrapper scripts, all identical except a few > lines. [...] > Therefore I use Perl magic in the debian/rules file to do these > identical changes on all the scripts. [...] > 1. Add /etc/papersize support: This change on the foo2...-wrapper > scripts makes the default page size being taken from /etc/papersize, > like all programs which use libpaper do, too. This way one has a > reasonable default paper size and not Letter which is used only in two > countries on the world. Committed: <http://svn.debian.org/viewsvn/foo2zjs?view=rev&revision=260> <http://svn.debian.org/viewsvn/foo2zjs?view=rev&revision=261> FTR, I erroneously thought CUPS was also at fault here (i.e. it ignored /etc/papersize), but it seems I was wrong: <http://bugs.debian.org/88597> However, I guess that the best option would be to support LC_PAPER, given that AFAIK libpaper (thus /etc/papersize) is still a debianism while LC_PAPER is POSIX: <http://bugs.debian.org/376350> Anyway, this bug is not the place for such discussions ;-) > Also doing the same change on all PPD files is rather awkward by a > patch and I do also this by Perl Magic in the debian/rules file. You are right, but to track it down I added a fake patch: <http://svn.debian.org/viewsvn/foo2zjs?view=rev&revision=262> > I hope everything is clear now. Sure, but I still have a small question, since there is a debian/changelog entry for which I can not find the corresponding modifications: * debian/foo2zjs.postinst: Migrate driver name "foo2zjs" to "foo2zjs-z1" and "foo2zjs-z2". Is this something you have planned but not deployed yet? Finally, some good news ;-) 1) given that even the current Ubuntu version is a bit outdated, I packaged the most recent snapshot, i.e. 20110210: <http://svn.debian.org/viewsvn/foo2zjs?view=rev&revision=264> <http://svn.debian.org/viewsvn/foo2zjs?view=rev&revision=265> 2) I moved the maintenance to the Debian Printing Team, with Michael and Steffen (the old maintainers in the foo2zjs group), still listed as Uploaders:, maybe you want to remove them in the future: <http://svn.debian.org/viewsvn/foo2zjs?view=rev&revision=267> However, I do not know if you want to keep the packaging work in SVN or you want to switch to Git, if no one will reply before tomorrow evening, I will upload the package as it is (thus with the old SVN path). This because Till must upload it to Ubuntu natty before the freeze, i.e. 2011-02-24. Please remember that as I already stated previously, I have not tested the latest version, given that my printer is broken (and FWIW is also something like 400km away from me...). Thx, bye, Gismo / Luca
pgptsSg5pYon4.pgp
Description: PGP signature
