www/chromium is hard coded to depend on cups. This causes problems for those of us who use the base versions of lpd/lpr and have /usr/local/bin on our path. This patch makes cups support an option. Included is the patchset is a patch from cpm@ that solves an undefined linker reference that occurs when cups is disabled.
Note that chromium does not support printing without cups so this option also disables printing and print previews.
Created attachment 191378 [details]
Here is the patch set. With maintainer approval I'm happy to take this bug and create a differential to commit it.
Created attachment 191379 [details]
poudriere testport build log with CUPS disabled
I removed most of the build lines because even compressed it was too large to attach.
CUPS_DESC is defined in Mk/bsd.options.desc.mk, please remove this line:
+CUPS_DESC= Disable cups support (note: this completely disables printing)
Created attachment 191419 [details]
Revised patch without CUPS_DESC.
(In reply to Craig Leres from comment #4)
LGTM. Feel free to commit the patch :)
A commit references this bug:
Date: Mon Mar 12 17:44:37 UTC 2018
New revision: 464264
Add a CUPS option to disable cups support and remove the
dependency on print/cups. This includes a patch from cpm@ that
solves an undefined linker reference that occurs when cups is
disabled. Since chromium does not support printing without cups
enabling this option also disables printing and print previews.
Reviewed by: cpm, matthew (mentor)
Approved by: cpm, matthew (mentor)
Differential Revision: https://reviews.freebsd.org/D14659
(In reply to commit-hook from comment #6)
Good job, Craig :)