Bug 226505

Summary: [PATCH] www/chromium: add CUPS option
Product: Ports & Packages Reporter: Craig Leres <leres>
Component: Individual Port(s)Assignee: freebsd-chromium (Nobody) <chromium>
Status: Closed FIXED    
Severity: Affects Only Me CC: cpm
Priority: --- Flags: cpm: maintainer-feedback+
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch
none
poudriere testport build log with CUPS disabled
none
patch cpm: maintainer-approval+

Description Craig Leres freebsd_committer freebsd_triage 2018-03-10 20:38:35 UTC 
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.
Comment 1 Craig Leres freebsd_committer freebsd_triage 2018-03-10 20:40:53 UTC 
Created attachment 191378 [details]
patch

Here is the patch set. With maintainer approval I'm happy to take this bug and create a differential to commit it.
Comment 2 Craig Leres freebsd_committer freebsd_triage 2018-03-10 20:54:09 UTC 
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.
Comment 3 Carlos J. Puga Medina freebsd_committer freebsd_triage 2018-03-11 11:07:28 UTC 
CUPS_DESC is defined in Mk/bsd.options.desc.mk, please remove this line:

+CUPS_DESC=	Disable cups support (note: this completely disables printing)
Comment 4 Craig Leres freebsd_committer freebsd_triage 2018-03-11 16:12:46 UTC 
Created attachment 191419 [details]
patch

Revised patch without CUPS_DESC.
Comment 5 Carlos J. Puga Medina freebsd_committer freebsd_triage 2018-03-11 18:37:37 UTC 
(In reply to Craig Leres from comment #4)

LGTM. Feel free to commit the patch :)
Comment 6 commit-hook freebsd_committer freebsd_triage 2018-03-12 17:45:12 UTC 
A commit references this bug:

Author: leres
Date: Mon Mar 12 17:44:37 UTC 2018
New revision: 464264
URL: https://svnweb.freebsd.org/changeset/ports/464264

Log:
  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.

  PR:		226505
  Reviewed by:	cpm, matthew (mentor)
  Approved by:	cpm, matthew (mentor)
  Differential Revision:	https://reviews.freebsd.org/D14659

Changes:
  head/www/chromium/Makefile
  head/www/chromium/files/patch-chrome_service_cloud__print_print__system.cc
Comment 7 Carlos J. Puga Medina freebsd_committer freebsd_triage 2018-03-12 18:48:00 UTC 
(In reply to commit-hook from comment #6)

Good job, Craig :)