All of lore.kernel.org
 help / color / mirror / Atom feed
From: Till Kamppeter <till.kamppeter@gmail.com>
To: Michael Sweet <msweet@apple.com>
Cc: "printing-architecture@lists.linux-foundation.org"
	<printing-architecture@lists.linux-foundation.org>
Subject: Re: [Printing-architecture] Options in /etc/cups/printers.conf
Date: Wed, 10 Jul 2013 17:28:50 +0200	[thread overview]
Message-ID: <51DD7DB2.4070808@gmail.com> (raw)
In-Reply-To: <843B55ED-9D7F-4ECD-B50B-272848E9649B@apple.com>

Thank you for the patch, but the patch you have attached is wrong. The
attached patch is for the IPP version probl;em in client.conf (Apple bug
#14401718, Debian bug #711848).

My problem is Apple bug #14401795.

   Till


On 07/10/2013 04:44 PM, Michael Sweet wrote:
> Till,
> 
> Here is the patch for 1.7 (1.7rc1 to be released soon):
> 
> 
> 
> 
> 
> However, when I went back to test on 1.6 TOT I was unable to reproduce there. Either I had already fixed the problem for 1.6.3 or it wasn't already broken.  The changes in 1.7 (to filter encoded attributes based on operation code) were responsible for the breakage there, but 1.6 doesn't include that code...
> 
> 
> On Jul 10, 2013, at 10:17 AM, Michael Sweet <msweet@apple.com> wrote:
> 
>> Filed as Apple bug:
>>
>>    <rdar://problem/14401795> cups.org: xxx-default options set from lpadmin are not returned by Get-Printer-Attributes
>>
>> I was able to reproduce both with lpoptions and ipptool.  I'll let you know when I have a fix...
>>
>>
>> On Jul 9, 2013, at 5:40 PM, Till Kamppeter <till.kamppeter@gmail.com> wrote:
>>
>>> Hi,
>>>
>>> I have a problem with non-PPD options set in /etc/cups/printers.conf.
>>>
>>> For filters and cups-browsed I have defined some options foor being
>>> stored in /etc/cups/printers.conf which are not options of CUPS itself.
>>>
>>> I set these options via
>>>
>>> lpadmin -p <printer> -o <option>-default=<value>
>>>
>>> getting a line
>>>
>>> Option <option> <value>
>>>
>>> in the entry of <printer>. What works is that these options get supplied
>>> with every job, so I can use such options to configure filters or to get
>>> filters into a debug mode. I am using this successfully for example with
>>> the "usb" CUPS backend to find quirk rules for broken USB printers.
>>>
>>> Now I also want to use this simply to mark print queues (setting an
>>> option which is ignored by all filters). For example all queues created
>>> by cups-browsed get the cups-browsed boolean option set by doing the IPP
>>> equivalent of:
>>>
>>> lpadmin -p <printer> -o cups-browsed-default
>>>
>>> resulting in a line
>>>
>>> Option cups-browsed true
>>>
>>> in /etc/cups/printers.conf. This way I want that if cups-browsed not
>>> closed cleanly (system crash) and so did not remove the queues it
>>> created in the next session finds these queues and treats them
>>> correctly, so that crashes do not lead to ghost queues.
>>>
>>> Now it seems that I cannot read out this option setting (without sending
>>> a job and checking error_log), as
>>>
>>> lpoptions -p <printer>
>>>
>>> does not list the option (as "cups-browsed" or "cups-browsed=true") and
>>>
>>> val = cupsGetOption("cups-browsed", dest->num_options, dest->options)
>>>
>>> in a C program (with dest being the record of the printer in question)
>>> gives val = NULL as one gets when requesting the value of a non-existing
>>> option.
>>>
>>> In CUPS 1.5.x (Ubuntu Precise, 12.04 LTS) this still worked. both
>>> cups-browsed was able to read out the "cups-browsed" marker option from
>>> its queues via the C line shown above and
>>>
>>> lpoptions -p <printer>
>>>
>>> contains "cups-browsed=true" in its output.
>>>
>>> I would consider this a bug in CUPS 1.6.x. Mike can you have a look? Thanks.
>>>
>>>  Till
>>
>> _________________________________________________________
>> Michael Sweet, Senior Printing System Engineer, PWG Chair
>>
>> _______________________________________________
>> Printing-architecture mailing list
>> Printing-architecture@lists.linux-foundation.org
>> https://lists.linuxfoundation.org/mailman/listinfo/printing-architecture
> 
> _________________________________________________________
> Michael Sweet, Senior Printing System Engineer, PWG Chair
> 


  reply	other threads:[~2013-07-10 15:28 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-09 21:40 [Printing-architecture] Options in /etc/cups/printers.conf Till Kamppeter
2013-07-10 14:17 ` Michael Sweet
2013-07-10 14:44   ` Michael Sweet
2013-07-10 15:28     ` Till Kamppeter [this message]
2013-07-10 15:47       ` Michael Sweet
2013-07-10 15:55         ` Till Kamppeter
2013-07-11  8:48         ` Tim Waugh
2013-07-11 13:11           ` Michael Sweet
2013-07-16 14:33             ` Tim Waugh
2013-07-16 14:48               ` Michael Sweet
2013-07-16 16:27                 ` Tim Waugh
2013-07-16 16:34                   ` Michael Sweet
2014-02-25 23:00             ` James Cloos
2014-02-26  1:12               ` Michael Sweet
2013-07-10 17:11     ` Till Kamppeter
2013-07-10 17:30       ` Michael Sweet
2013-07-10 18:44         ` Till Kamppeter

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51DD7DB2.4070808@gmail.com \
    --to=till.kamppeter@gmail.com \
    --cc=msweet@apple.com \
    --cc=printing-architecture@lists.linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.