Prev · NextSTR #4153: tag in printers.conf closes with

Status:	4 - Pending
Priority:	1 - Request for Enhancement, e.g. asking for a feature
Scope:	3 - Applies to all machines and operating systems
Subsystem:	Scheduler
Summary:	<DefaultPrinter> tag in printers.conf closes with </Printer>
Version:	-feature
Created By:	domcleal
Assigned To:	mike
Fix Version:	Unassigned
Update Notification:	Receive EMails Don't Receive EMails

Trouble Report Files:

Name/Time/Date/Filename/Size
domcleal: 14:33 Aug 01, 2012 r10562_defaultprinter.patch (3k)

Trouble Report Dialog:

Name/Time/Date/Text

Name/Time/Date/Text
domcleal: 14:33 Aug 01, 2012 While testing support for parsing printers.conf in Augeas (http://augeas.net, a configuration API), it was found the <DefaultPrinter> section is closed with </Printer> rather than the more obvious </DefaultPrinter>. The Augeas lens was expecting a matching set of tags so failed to parse. While this is an issue with Augeas (though not an easy one to fix), it seems this could be a common trap and it improved by using </DefaultPrinter> to match HTML, XML and Apache HTTP type config files. Attached is a patch to change this, preserving compatibility to read old printers.conf files. It's untested, but compiles with trunk.

domcleal: 14:33 Aug 01, 2012

While testing support for parsing printers.conf in Augeas (http://augeas.net, a configuration API), it was found the <DefaultPrinter> section is closed with </Printer> rather than the more obvious </DefaultPrinter>.  The Augeas lens was expecting a matching set of tags so failed to parse.

While this is an issue with Augeas (though not an easy one to fix), it seems this could be a common trap and it improved by using </DefaultPrinter> to match HTML, XML and Apache HTTP type config files.

Attached is a patch to change this, preserving compatibility to read old printers.conf files.  It's untested, but compiles with trunk.