On Wed, Jun 8, 2011 at 11:26 AM, Randall Leeds <randall.le...@gmail.com> wrote: > Strange. > On my machine that command includes -I for erlang includes. > > Paul, that's ERLANG_FLAGS as set by configure, not ERL_FLAGS. > Does it help if you switch it to CXX or CPP? Maybe your systems are stricter > about using those variables for the .cc based stuff. > > Look in that folder's generated Makefile. Does ERLANG_FLAGS have info for > finding erl_nif.h? What's that make target have for variables and does it > include the la_CFLAGS automake is supposed to have stuck in there? And does > that include ERLANG_FLAGS? > > Sorry for breaking this for you. Thanks for your help.
Huh. Maybe everyone just needs to re-bootstrap? I haven't had a chance to get to look at it myself. I was just confused by ERL_FLAGS vs ERLANG_FLAGS. > On Jun 8, 2011 7:24 AM, "Filipe David Manana" <fdman...@apache.org> wrote: >> Breaks my build (make dev) as well: >> >> make[3]: Entering directory `/home/fdmanana/git/hub/couchdb/src/snappy' >> /bin/bash ../../libtool --tag=CXX --mode=compile g++ -DHAVE_CONFIG_H >> -I. -I../.. -I../../src/snappy/google-snappy -D_XOPEN_SOURCE -g >> -O2 -MT snappy_nif.lo -MD -MP -MF .deps/snappy_nif.Tpo -c -o >> snappy_nif.lo snappy_nif.cc >> libtool: compile: g++ -DHAVE_CONFIG_H -I. -I../.. >> -I../../src/snappy/google-snappy -D_XOPEN_SOURCE -g -O2 -MT >> snappy_nif.lo -MD -MP -MF .deps/snappy_nif.Tpo -c snappy_nif.cc -fPIC >> -DPIC -o .libs/snappy_nif.o >> In file included from snappy_nif.cc:21: >> erl_nif_compat.h:27: fatal error: erl_nif.h: No such file or directory >> compilation terminated. >> make[3]: *** [snappy_nif.lo] Error 1 >> >> >> On Wed, Jun 8, 2011 at 3:21 PM, Paul Davis <paul.joseph.da...@gmail.com> > wrote: >>> On Wed, Jun 8, 2011 at 10:20 AM, Robert Dionne >>> <dio...@dionne-associates.com> 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, <rand...@apache.org> 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. >>> >> >> >> >> -- >> Filipe David Manana, >> fdman...@gmail.com, fdman...@apache.org >> >> "Reasonable men adapt themselves to the world. >> Unreasonable men adapt the world to themselves. >> That's why all progress depends on unreasonable men." >