Hi there!
Cc:ing Didier, the Ubuntu foo2zjs maintainers and the debian-printing@
mailing list, if you keep the bug report cc:ed, no need to cc: me.
I am sorry for the long mail, it was necessary to explain the crucial
points.
On Wed, 25 Aug 2010 11:18:07 +0200, Didier Raboud wrote:
> While going through the list of potential packages for the Debian Printing
> Group [0], I noticed that your foo2zjs is heavily outdated in terms of
> upstream version and sync with Ubuntu.
This is true, and shame on me. My previous plans to bring it up-to-date
with the version in Ubuntu miserably failed, for various reasons. And
finally, the only printer for which I need foo2zjs (an HP Color LaserJet
1500L) seems to have reach an end (and anyway we do not use it anymore
in the lab where I am working).
> So I would like to suggest you (and I have no intention to walk on your toes,
> huh) to update foo2zjs to a more recent upstream release for Squeeze. One
> reason for this is that Ubuntu already has one stable release with 20100210
> and their developement release is at 20100728.
Given that Ubuntu Maverick will be out next month, this (i.e. 20100728)
will the same version Debian should provide.
> I had the opportunity to work with the actual foo2zjs Ubuntu maintainer (Till
> Kamppeter, who happens to be the foomatic-* upstream developer) and he has
> been very constructive and very open to cross-distro collaboration (+ he is
> skilled), so you might want to approach him with a collaboration request.
I started working on a new updated Debian package and also merging
Ubuntu changes:
http://svn.debian.org/viewsvn/foo2zjs?view=rev&revision=232
I have some concerns about the Ubuntu package, here the first of them, I
will continue on another email as far as the integration progresses.
1) I do not understand why from version 20100210-0ubuntu1 the
debian/changelog contains the following:
* README.Debian: Updated completely outdated content. We are using
the complete original source code again for longer time as there
are no binary-only executables and no *.icm files any more in the
source tarball.
I am sorry but I do not understand why the complete original source
code is now DFSG, while nothing changed WRT the files we previously
deleted, which from the debian/changelog are:
- remove binary file c5200mono.prn
This is still without copyright and it is the result of a Windows
printing process for file "testpage.pdf", which is not included in
the upstream sources. There is, however, a testpage.ps.
- remove crd/qpdl/CLP*, because copyright is unclear
The copyright is clear, according to COPYING:
Copyright Samsung...
CLP-300*cms* (2006)
CLP-600*cms* (2006)
However, there is no indication of any license, which means that we
could be not allowed to distribute them. Actually, these could
also be a specific version directly prodided by Samsung in their
"Unified Linux Driver" (which we cannot distribute):
http://www.openprinting.org/printer/Samsung/Samsung-CLP-300
Until these questions have been addressed, foo2zjs upstream tarball
must be repackaged.
BTW, please note that I have not imported the new upstream tarball
yet, exaclty because of this point and to avoid a double "new
upstream version" upload (one with and one without the 'dfsg'
suffix).
2) I do not understand why some patches have been merged, like
* debian/patches/60-getweb.in.dpatch, debian/patches/80-getweb.in.dpatch:
merged 80-getweb.in.dpatch into 60-getweb.in.dpatch.
They fixes two different things, and they must be separated.
3) directory should be created through debian/$PKG.dirs and not by hand
in debian/rules (see /usr/lib/cups/filter/).
Always about the same issue, the link created by upstream's Makefile
is wrong, given it is not a relative one. The correct fix would be
to patch upstream's Makefile, but this can be quite tedious
especially if upstream changes something.
While the best option seems thus to fix it in debian/rules, we should
use dh_link and not ln.
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.
As I wrote before, I will continue in the next days, since I need to
understand the reasons behind all other changes ;-)
> Anyway, this is just a "please update to more recent upstream" and a "why not
> joining the Printing Team" mail.
At least from my POV, feel free to consider foo2zjs up for adoption.
But FWIW you will need Michael Koch, Steffen Joeris and Luca Bedogni's
approval as well.
> P.S. I am very aware that Squeeze is frozen, but I don't see an update of
> foo2zjs as risky.
Given the new rules, I would say that there is almost zero chance to
have the updated version in squeeze. Anyway, let me prepare the updated
version and then rediscuss it with the Release Team, in case of:
http://lists.debian.org/debian-devel-announce/2010/09/msg00000.html
Thx, bye,
Gismo / Luca
Attachment:
pgpv4qDcc6uI0.pgp
Description: PGP signature