Summary: | print/cups-filters: cups-browsed.conf is overwritten on installation/update | ||||||
---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Felix Palmen <zirias> | ||||
Component: | Individual Port(s) | Assignee: | Tijl Coosemans <tijl> | ||||
Status: | Closed FIXED | ||||||
Severity: | Affects Only Me | Keywords: | buildisok | ||||
Priority: | --- | Flags: | tijl:
maintainer-feedback+
|
||||
Version: | Latest | ||||||
Hardware: | Any | ||||||
OS: | Any | ||||||
Attachments: |
|
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/146600702 Hello Tijl, sorry for nagging, I'd just love to get some feedback. This patch would solve my actual problem, but if there are any issues with it, please let me know, thanks! BR, Felix Except for the white space changes the patch is fine. When the first tab on a line is just a single space wide I sometimes replace it with an actual space because then diffs are less messed up. I planned to include the patch in the next version update. Upstream tends to release quite often. It's just taken unusually long right now. Hello Tijl, thanks a lot for the feedback! As for the whitespace, I only changed it because portlint insists on tabs, should be easy enough to skip these changes if you don't like them :) I was just worried there's either something wrong with the patch or the PR went unnoticed in a bursting inbox ;) So thanks for letting me know, of course it makes sense here to await a new version, as this change won't have any benefit before. BR, Felix A commit references this bug: Author: tijl Date: Sat Jun 6 16:33:38 UTC 2020 New revision: 538106 URL: https://svnweb.freebsd.org/changeset/ports/538106 Log: Update to 1.27.5. Use @sample for cups-browsed.conf. [1] PR: 246074 [1] Submitted by: Felix Palmen <felix@palmen-it.de> [1] Changes: head/print/cups-filters/Makefile head/print/cups-filters/distinfo head/print/cups-filters/pkg-plist |
Created attachment 213971 [details] Install cups-browsed.conf with @sample mechanism My clients need a specific BrowsePoll setting in cups-browsed.conf because the print server resides in a different subnet. As the port overwrites this config on each update, I have to redo it regularly. Suggesting attached patch using the @sample mechanism for cups-browsed.conf. For Makefile, only the last hook is the *actual* change, the others are whitespace changes needed to silence portlint during poudriere testport.