couchdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Randall Leeds <randall.le...@gmail.com>
Subject Re: svn commit: r1133319 - /couchdb/trunk/src/ejson/Makefile.am
Date Wed, 08 Jun 2011 15:26:16 GMT
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.
On Jun 8, 2011 7:24 AM, "Filipe David Manana" <fdmanana@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.davis@gmail.com>
wrote:
>> On Wed, Jun 8, 2011 at 10:20 AM, Robert Dionne
>> <dionne@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,  <randall@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,
> fdmanana@gmail.com, fdmanana@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."

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message