I am having a host of issues with 1.6.6 for whatever reasons, and some of
them look rather serious so I am reporting it early (before doing all of the
checking even, so please forgive me if something is misreported):
1. The hp backend changed the "device detect" strings in such a way that it
is now completely and utterly useless on CUPS 1.1 if you have more than
one of the same device attached. This happens because now CUPS outputs
this instead of the device URI (which has "little" hints like the IP or
serial number to help the admin know which printer he is dealing with):
"HP Photosmart 2600 series HPLIP (HP Photosmart 2600 series)".
I am not sure what happens with CUPS 1.2, but it is really inconvenient
for CUPS 1.1.
2. The hpfax backend is completely toasted re. autodetection. Not only the
code will abend with an exception if "m = direct_pat.match(d)" returns
the "None" object in hpfax.py, but also the regexp there was not updated
to deal with (1) above, so it never detects anything and CUPS will never
add a fax queue. To test, just run it with no fax queues setup, and a
fax-capable device plugged in (or a non-fax queue set up, maybe).
Now for minor annoyances I'd much like to see fixed as well, but which are
mostly minor details:
1. log.set_module should be called ASAP by *every* utility. It is quite
annoying to have log-lines reading "python: [ERROR]"... and
"python: [WARN]"... in syslog. Also, make it use the proper
"executable[pid]:" format, **please**.
In fact, it would be a nice thing altogether IMHO if the entire uglyness
of those [TAGS] were to be replaced with the standard "warning: ", "error: ",
"debug: " prefixes. Could you consider this for the next version?
Compare (current):
Jun 16 12:53:21 khazad-dum python: [ERROR] Fax queue setup failed. Please
restart CUPS and try again.
to (proposed):
Jun 16 12:53:21 khazad-dum hp-setup[1234]: error: Fax queue setup failed.
Please restart CUPS and try again.
2. The main Makefile.am has duplicated code for pcardext and cupsext
install, and the code snippets don't even agree on how they should
call "$(PYTHON) setup.py install", which is Not Good(tm). I have
no idea if this will cause damage to user installs, but distros just
override this anyway.
3. The new SANE desc file is missing the version number, I'd suggest using
configure.in to fill it in using a hpaio.desc.in file, that's what it
does best ;-) Not that I know what to make of this .desc file, at least
for Debian. Maybe it is a SANE 2 thing?
--
"One disk to rule them all, One disk to find them. One disk to bring
them all and in the darkness grind them. In the Land of Redmond
where the shadows lie." -- The Silicon Valley Tarot
Henrique Holschuh
_______________________________________________
HPLIP-Devel mailing list
HPLIP-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/hplip-devel
