Bug 220356

Summary: editors/libreoffice: properly handle verbose build
Product: Ports & Packages Reporter: Dmitry Marakasov <amdmi3>
Component: Individual Port(s)Assignee: FreeBSD Office Team <office>
Status: Closed FIXED    
Severity: Affects Many People CC: jbeich, jkim, office
Priority: --- Keywords: easy
Version: LatestFlags: bugzilla: maintainer-feedback? (office)
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch koobs: maintainer-approval+

Description Dmitry Marakasov freebsd_committer freebsd_triage 2017-06-29 12:45:02 UTC
Created attachment 183909 [details]
Patch

Remove VERBOSE option, since verbose builds are mandatory as per PHB ยง12.10. Also fix handling verbose logs - --enable-verbose no longer works, use verbose=1 make environment variable instead.
Comment 1 Kubilay Kocak freebsd_committer freebsd_triage 2017-06-29 12:55:07 UTC
@Dmitry I believe this comes under implicit (blanket) approval:

Approved by: portmgr (blanket)
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2017-06-29 12:55:21 UTC
Comment on attachment 183909 [details]
Patch

Approved by: portmgr (blanket)
Comment 3 Dmitry Marakasov freebsd_committer freebsd_triage 2017-06-29 13:16:53 UTC
I've mentioned this multiple times before: I am quite aware of the blanket, and if I submit PR whith changes falling under it, I have clear intention of getting them reviewed.
Comment 4 commit-hook freebsd_committer freebsd_triage 2017-07-17 15:37:26 UTC
A commit references this bug:

Author: amdmi3
Date: Mon Jul 17 15:36:34 UTC 2017
New revision: 446080
URL: https://svnweb.freebsd.org/changeset/ports/446080

Log:
  Fix verbose build (--enable-verbose no longer works) and enable it unconditionally as per PHB ?12.10.

  PR:		220356
  Approved by:	maintainer timeout (office, 2 weeks)

Changes:
  head/editors/libreoffice/Makefile
Comment 5 Jan Beich freebsd_committer freebsd_triage 2018-01-12 20:58:53 UTC
*** Bug 214557 has been marked as a duplicate of this bug. ***