Bug 197241

Summary: [PATCH] sysutils/lsof fix build on systems without /etc/make.conf
Product: Ports & Packages Reporter: Thomas Zander <riggs>
Component: Individual Port(s)Assignee: Thomas Zander <riggs>
Status: Closed FIXED    
Severity: Affects Some People CC: arrowd, ler
Priority: --- Flags: ler: maintainer-feedback+
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch to fix build on systems without /etc/make.conf none

Description Thomas Zander freebsd_committer freebsd_triage 2015-02-01 13:31:49 UTC 
Created attachment 152436 [details]
Patch to fix build on systems without /etc/make.conf

Terry Kennedy <terry@tmk.com> noticed that on plain vanilla installations of FreeBSD lsof does not build due to picking up CFLAGS incorrectly.

We tracked this down to a bug in lsof's configure script checking for the existence of make.conf and then deriving CFLAGS based on the result.
It turns out that it does not do anything with make.conf's content, but its absence causes the configure script to select CFLAGS erroneously.
The attached micro-patch ensures that the same algorithm is always used to select CFLAGS, independently of make.conf's existence on the target machine.

Tested on 8.4, 9.3 (i386) and 10.1 (amd64, armv6).

@Maintainer: Do you approve?
Comment 1 Bugzilla Automation freebsd_committer freebsd_triage 2015-02-01 13:31:49 UTC 
Maintainer CC'd
Comment 2 Larry Rosenman freebsd_committer freebsd_triage 2015-02-01 14:08:07 UTC 
looks reasonable to me -- I will also forward to Vic for inclusion in the next release/fixup.
Comment 3 commit-hook freebsd_committer freebsd_triage 2015-02-01 15:27:26 UTC 
A commit references this bug:

Author: riggs
Date: Sun Feb  1 15:27:14 UTC 2015
New revision: 378245
URL: https://svnweb.freebsd.org/changeset/ports/378245

Log:
  Fix build on systems without /etc/make.conf

  PR:		197241
  Submitted by:	riggs
  Approved by:	ler@lerctr.org (maintainer)
  MFH:		2015Q1

Changes:
  head/sysutils/lsof/files/
  head/sysutils/lsof/files/patch-Configure
Comment 4 Thomas Zander freebsd_committer freebsd_triage 2015-02-01 15:28:48 UTC 
Thanks for the quick response!
Comment 5 commit-hook freebsd_committer freebsd_triage 2015-02-01 15:49:32 UTC 
A commit references this bug:

Author: riggs
Date: Sun Feb  1 15:48:58 UTC 2015
New revision: 378249
URL: https://svnweb.freebsd.org/changeset/ports/378249

Log:
  MFH: r378245

  Fix build on systems without /etc/make.conf

  PR:		197241
  Submitted by:	riggs
  Approved by:	ports-secteam (zi), ler@lerctr.org (maintainer)

Changes:
_U  branches/2015Q1/
  branches/2015Q1/sysutils/lsof/Makefile
  branches/2015Q1/sysutils/lsof/files/
Comment 6 commit-hook freebsd_committer freebsd_triage 2018-10-01 14:02:46 UTC 
A commit references this bug:

Author: arrowd
Date: Mon Oct  1 14:02:08 UTC 2018
New revision: 481032
URL: https://svnweb.freebsd.org/changeset/ports/481032

Log:
  lang/ghc: Disable using xelatex, as the build fails if it is present.

  PR:		197241
  Submitted by:	John Hein <z7dr6ut7gs@snkmail.com>
  Approved by:	tcberner (mentor)

Changes:
  head/lang/ghc/Makefile
Comment 7 Gleb Popov freebsd_committer freebsd_triage 2018-10-01 14:05:43 UTC 
Oops, I put wrong ID in my commit message. Sorry for the noise.