Ubuntu
cups package

lpstat in CUPS 1.3 can't list jobs on CUPS 1.4 servers

Bug #497606 reported by Evan Broder
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
cups (Ubuntu)
Fix Released
Undecided
Unassigned
Declined for Karmic by Till Kamppeter
Hardy
Invalid
Undecided
Unassigned
Intrepid
Won't Fix
Undecided
Unassigned
Jaunty
Fix Released
Undecided
Unassigned
cupsys (Ubuntu)
Invalid
Undecided
Unassigned
Declined for Karmic by Till Kamppeter
Hardy
Fix Released
Undecided
Unassigned
Intrepid
Invalid
Undecided
Unassigned
Jaunty
Invalid
Undecided
Unassigned

Bug Description

Binary package hint: cups

IMPACT: lpstat on CUPS <=1.3.9 is unable to list jobs on a CUPS 1.4 server

DEVELOPMENT: Karmic and Lucid side-stepped this bug by upgrading to CUPS 1.4 clients. CUPS upstream fixed this in 1.3.10, and the attached patches backport that fix.

PATCHES: http://launchpadlibrarian.net/40108552/cupsys_1.3.7-1ubuntu3.9.debdiff for Hardy, http://launchpadlibrarian.net/40108617/cups_1.3.9-2ubuntu9.6.debdiff for Intrepid, and http://launchpadlibrarian.net/40108644/cups_1.3.9-17ubuntu3.7.debdiff for Jaunty

INSTRUCTIONS: Reproduce by running `lpstat -h <server> -o <queue>` against a CUPS 1.4 queue.

REGRESSION: I think regression is unlikely. The patches only add another option to lpstat requests - the patch is from upstream, no code is removed, and empirically the patch works.

===

CUPS versions before 1.3.10 failed to include a printer-uri attribute in IPP_GET_JOBS requests issued by lpstat.

Against servers running CUPS 1.4, this causes an error when you try to use lpstat to retrieve jobs:

  dr-wily:~ broder$ lpstat -h get-print.mit.edu -o ajax
  lpstat: No printer-uri in request!

This can be fixed by backporting a patch from upstream (parts of r8146 in http://svn.easysw.com/public/cups/branches/branch-1.3) - apparently the printer-uri value can be bogus, so long as it's there.

See original description
Revision history for this message
Evan Broder (broder) wrote :

SRU fix for Hardy

Revision history for this message
Evan Broder (broder) wrote :

SRU fix for Intrepid

Revision history for this message
Evan Broder (broder) wrote :

SRU fix for Jaunty

Revision history for this message
Evan Broder (broder) wrote :

I have test builds for each of these patches in my PPA (https://launchpad.net/~broder/+archive/ubuntu-tests), and I've tested the i386 builds on Hardy, Intrepid, and Jaunty. On all three releases, the patched packages fix the lpstat bug.

Revision history for this message
Evan Broder (broder) wrote :

Whoops - let's try again, but this time with LP closers.

First, Hardy

Revision history for this message
Evan Broder (broder) wrote :

Intrepid

Revision history for this message
Evan Broder (broder) wrote :

And Jaunty

Evan Broder (broder)
description: updated
Revision history for this message
John Dong (jdong) wrote :

ACK from ubuntu-sru for all 3 debdiffs.

Till Kamppeter (till-kamppeter)
Changed in cups (Ubuntu Hardy):
status: New → Triaged
Changed in cups (Ubuntu Intrepid):
status: New → Triaged
Changed in cups (Ubuntu Jaunty):
status: New → Triaged
Changed in cupsys (Ubuntu Hardy):
status: New → Triaged
Changed in cupsys (Ubuntu Intrepid):
status: New → Triaged
Changed in cupsys (Ubuntu Jaunty):
status: New → Triaged
Changed in cupsys (Ubuntu Intrepid):
status: Triaged → Invalid
Changed in cups (Ubuntu Hardy):
status: Triaged → Invalid
Changed in cupsys (Ubuntu Jaunty):
status: Triaged → Invalid
Changed in cupsys (Ubuntu):
status: New → Triaged
Changed in cups (Ubuntu):
status: New → Triaged
Revision history for this message
Evan Broder (broder) wrote :

I've rebased my patches on top of the security update that went out today. Builds are running in my PPA (https://launchpad.net/~broder/+archive/ubuntu-tests) now.

New patch first for Hardy.

Revision history for this message
Evan Broder (broder) wrote :

Now Intrepid.

Revision history for this message
Evan Broder (broder) wrote :

And Jaunty.

description: updated
Emmet Hikory (persia)
Changed in cups (Ubuntu):
status: Triaged → Fix Released
Changed in cupsys (Ubuntu):
status: Triaged → Invalid
Revision history for this message
Martin Pitt (pitti) wrote :

I don't think it's worth fixing this for intrepid. It goes EOL in less than two months, and if someone has used intrepid with this bug for so long, then it can hardly be a showstopper now.

Changed in cups (Ubuntu Intrepid):
status: Triaged → Won't Fix
Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted cupsys into hardy-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in cupsys (Ubuntu Hardy):
status: Triaged → Fix Committed
tags: added: verification-needed
Changed in cups (Ubuntu Jaunty):
status: Triaged → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

Accepted cups into jaunty-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Revision history for this message
Evan Broder (broder) wrote :

I've verified that the fixes in -proposed fix lpstat against a CUPS 1.4 server for both Hardy and Jaunty.

Revision history for this message
Till Kamppeter (till-kamppeter) wrote :

Evan Broder, thanks for testing, the packages will soon get passed through to the official updates.

tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package cupsys - 1.3.7-1ubuntu3.9

---------------
cupsys (1.3.7-1ubuntu3.9) hardy-proposed; urgency=low

   * debian/patches/fix-lpstat.dpatch: Fix lpstat to work correctly against
     CUPS 1.4 servers. (LP: #497606)
 -- Evan Broder <email address hidden> Wed, 03 Mar 2010 18:06:14 -0500

Changed in cupsys (Ubuntu Hardy):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package cups - 1.3.9-17ubuntu3.7

---------------
cups (1.3.9-17ubuntu3.7) jaunty-proposed; urgency=low

   * debian/patches/fix-lpstat.dpatch: Fix lpstat to work correctly against
     CUPS 1.4 servers. (LP: #497606)
 -- Evan Broder <email address hidden> Wed, 03 Mar 2010 20:29:00 -0500

Changed in cups (Ubuntu Jaunty):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.