On Wed, Jun 8, 2011 at 10:20 AM, Robert Dionne wrote: > well it breaks my build :) > > > > > > On Jun 8, 2011, at 10:15 AM, Paul Davis wrote: > >> On Wed, Jun 8, 2011 at 5:55 AM,   wrote: >>> Author: randall >>> Date: Wed Jun  8 09:55:00 2011 >>> New Revision: 1133319 >>> >>> URL: http://svn.apache.org/viewvc?rev=1133319&view=rev >>> Log: >>> include $(ERLANG_FLAGS) when building ejson nif >>> >>> Modified: >>>    couchdb/trunk/src/ejson/Makefile.am >>> >>> Modified: couchdb/trunk/src/ejson/Makefile.am >>> URL: http://svn.apache.org/viewvc/couchdb/trunk/src/ejson/Makefile.am?rev=1133319&r1=1133318&r2=1133319&view=diff >>> ============================================================================== >>> --- couchdb/trunk/src/ejson/Makefile.am (original) >>> +++ couchdb/trunk/src/ejson/Makefile.am Wed Jun  8 09:55:00 2011 >>> @@ -65,6 +65,7 @@ if USE_OTP_NIFS >>>  ejsonpriv_LTLIBRARIES = ejson.la >>> >>>  ejson_la_SOURCES = $(EJSON_C_SRCS) >>> +ejson_la_CFLAGS = $(ERLANG_FLAGS) >>>  ejson_la_LDFLAGS = -module -avoid-version >>> >>>  if WINDOWS >>> >>> >>> >> >> Is this right? > > Pretty sure ERL_FLAGS is for flags for erlc which probably aren't gonna go so hot for gcc. Just saying is all.