Bug 14952 - BrowseAllow is invalid syntax in cupsd.conf
Summary: BrowseAllow is invalid syntax in cupsd.conf
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: Mageia 5
Assignee: Richard Houser
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-01-05 08:29 CET by Richard Houser
Modified: 2015-02-05 14:15 CET (History)
2 users (show)

See Also:
Source RPM: cups
CVE:
Status comment:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Richard Houser 2015-01-05 08:29:07 CET 
In current cauldron, BrowseAllow is no longer valid syntax, but is included in the default configuration file.

I have removed that syntax, rebuilt the package, and the error is no longer present when I start cups.  I have not checked anything else, yet, as my network uses the old IPP announce methods.  I'll have to upgrade several other machines before I could verify that portion.


I've attached a patch and will work to get this committed myself if there are no objections.

http://paste.debian.net/139191/
Richard Houser 2015-01-05 08:29:50 CET

CC: (none) => rick
Target Milestone: --- => Mageia 5
Source RPM: (none) => cups

Richard Houser 2015-01-05 08:31:40 CET

Assignee: bugsquad => rick

Comment 1 Shlomi Fish 2015-01-05 13:02:27 CET 
Hi Richard,

(In reply to Richard Houser from comment #0)
> In current cauldron, BrowseAllow is no longer valid syntax, but is included
> in the default configuration file.
> 
> I have removed that syntax, rebuilt the package, and the error is no longer
> present when I start cups.  I have not checked anything else, yet, as my
> network uses the old IPP announce methods.  I'll have to upgrade several
> other machines before I could verify that portion.
> 
> 
> I've attached a patch and will work to get this committed myself if there
> are no objections.
> 
> http://paste.debian.net/139191/

Thanks for the patch, I applied it and submitted it. One note: next time, please actually attach the patch to the bugzilla instead of putting it on a paste site, so we'll be able to track it better.

Regards,

-- Shlomi Fish

CC: (none) => shlomif

Comment 2 Richard Houser 2015-02-05 14:15:49 CET 
Fixed as per Shlomi's commit.

Status: NEW => RESOLVED
Resolution: (none) => FIXED
Note You need to log in before you can comment on or make changes to this bug.