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.
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!
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.