Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 19 Feb 2016 22:55:18 +0000 (UTC)
From:      Ian Lepore <ian@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r409214 - head/lang/lua53/files
Message-ID:  <201602192255.u1JMtIbG069633@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ian
Date: Fri Feb 19 22:55:18 2016
New Revision: 409214
URL: https://svnweb.freebsd.org/changeset/ports/409214

Log:
  Fix CFLAGS handling in lua53 patches.
  
  The ?= assignment of CFLAGS (not present in earlier lua5x patches) results
  in MYCFLAGS, passed from the port makefile, never getting assigned to CFLAGS
  during the lua build. The result is a copy of lua that has none of the posix
  support functions in it.
  
  CFLAGS set in the outer environment by the user or make.conf get passed in
  via the MYCFLAGS mechanism, so remove the -O2 and let the outer environment
  control optimization level.
  
  Approved by:	mat(mentor)
  Differential Revision:	https://reviews.freebsd.org/D5256

Modified:
  head/lang/lua53/files/patch-src__Makefile

Modified: head/lang/lua53/files/patch-src__Makefile
==============================================================================
--- head/lang/lua53/files/patch-src__Makefile	Fri Feb 19 22:51:22 2016	(r409213)
+++ head/lang/lua53/files/patch-src__Makefile	Fri Feb 19 22:55:18 2016	(r409214)
@@ -7,7 +7,7 @@
 -CC= gcc -std=gnu99
 -CFLAGS= -O2 -Wall -Wextra -DLUA_COMPAT_5_2 $(SYSCFLAGS) $(MYCFLAGS)
 +CC?= gcc -std=gnu99
-+CFLAGS?= -O2 -Wall -Wextra -DLUA_COMPAT_5_2 $(SYSCFLAGS) $(MYCFLAGS)
++CFLAGS= -Wall -Wextra -DLUA_COMPAT_5_2 $(SYSCFLAGS) $(MYCFLAGS)
  LDFLAGS= $(SYSLDFLAGS) $(MYLDFLAGS)
  LIBS= -lm $(SYSLIBS) $(MYLIBS)
  



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201602192255.u1JMtIbG069633>