Bug 122943

Summary: sysutils/pfstat does not respect CC
Product: Ports & Packages Reporter: Frank <frank>
Component: Individual Port(s)Assignee: mlaier
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
ports-sysutils-pfstat-CC.diff none

Description Frank 2008-04-20 19:30:05 UTC 
sysutils/pfstat uses "cc" to build in any case, regardless of what CC is set to.

Fix: 

Apply the attached patch.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2008-04-20 19:30:39 UTC 
Responsible Changed
From-To: freebsd-ports-bugs->mlaier

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 Frank 2008-04-20 20:09:11 UTC 
Forgot the patch..
Comment 3 Max Laier 2008-04-20 20:39:35 UTC 
This a more complicated problem, really.  If you set CC in make.conf it 
will work, but not from the shell.  OTOH, if you set CFLAGS in make.conf 
it will trash the addition of ALTQ to CFLAGS.  The suggested workaround 
is to set __MAKE_CONF=/dev/null in the MAKE_ENV and add CFLAGS there, 
too.  This would then again break setting CC (unless it was also part of 
MAKE_ENV ...).  This works well for CC and CFLAGS, but I can't really add 
everything to MAKE_ENV that people might want to set in make.conf ... I'm 
still undecided what the best solution is and am looking for input.

Unless you have a bright idea, I'm going to add the following work-around:

  MAKE_ENV+=	CC="${CC}" CFLAGS="${CFLAGS}" __MAKE_CONF=/dev/null

after adding version dependend stuff to CFLAGS, of course.

Any feedback appreciated.

--
  Max
Comment 4 dfilter service freebsd_committer freebsd_triage 2008-04-27 19:32:08 UTC 
mlaier      2008-04-27 18:32:02 UTC

  FreeBSD ports repository (src committer)

  Modified files:
    sysutils/pftop       Makefile 
  Log:
  Manually export CC and CFLAGS to MAKE_ENV and have __MAKE_CONF point to
  /dev/null.  The latter is a problem because the port's Makefile uses the
  bsd.*.mk infrastructure.  This works around several problems people were
  having when setting CFLAGS[1] in make.conf or CC[2] on the command line.
  
  PR:             ports/122943
  Reported-by:    many[1], Frank Fenor[2]
  
  Revision  Changes    Path
  1.11      +2 -0      ports/sysutils/pftop/Makefile
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 5 mlaier freebsd_committer freebsd_triage 2008-04-27 19:43:23 UTC 
State Changed
From-To: open->closed

Fix committed, thanks.
Comment 6 Frank 2008-04-27 22:04:51 UTC 
This was committed into sysutils/pftop
It might be, that sysutils/pftop had the same problem (didn't check that), 
but I had the problem with sysutils/pfstat.
Comment 7 dfilter service freebsd_committer freebsd_triage 2008-04-28 18:52:14 UTC 
mlaier      2008-04-28 17:52:10 UTC

  FreeBSD ports repository (src committer)

  Modified files:
    sysutils/pfstat      Makefile 
  Log:
  Manually export CC and CFLAGS to MAKE_ENV and have __MAKE_CONF point to
  /dev/null.  The latter is a problem because the port's Makefile uses the
  bsd.*.mk infrastructure.  This works around several problems people were
  having when setting CFLAGS[1] in make.conf or CC[2] on the command line.
  
  PR:             ports/122943 (really this time)
  Reported-by:    many[1], Frank Fenor[2]
  
  Revision  Changes    Path
  1.13      +1 -0      ports/sysutils/pfstat/Makefile
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"