Re: [Opalvoip-devel] Build system glitch

[Alexander S. <ale...@gm...>]

Ok.
Maybe another way out will be change inside toplevel.mak.in from:

CFLAGS += -I$(OPAL_INCDIR)
CXXFLAGS += -I$(OPAL_INCDIR)

to:

CFLAGS := -I$(OPAL_INCDIR) $(CFLAGS)
CXXFLAGS := -I$(OPAL_INCDIR) $(CXXFLAGS)

This way internal opal include will be used as the first include dir 
choice. Resulting compilation line is like this:

g++ -I/root/Eridani/opal/include -I/usr/local/include/opal  -Wall 
-Wextra -Wstrict-aliasing -Wfloat-equal -Wno-comment -Wno-unused 
-Winit-self -Wno-missing-field-initializers -DP_64BIT -DPTRACING=2 
-D_REENTRANT -D_GNU_SOURCE=1 -fno-exceptions -I/usr/local/include 
-I/usr/include/SDL   -felide-constructors -Wreorder -fPIC    -c 
/root/Eridani/opal/src/opal/guid.cxx -o 
/root/Eridani/opal/lib_linux_x86_64/obj/guid.o



13.12.2012 2:38, Robert Jongbloed ?????:
> I don't think it is as simple as that, I have an idea that it will 
> then break for "installed" versions, those not using OPALDIR and just 
> doing the "make install".
>
> I have encountered this issue before, when the installed version is 
> different from the "local" version and it needs to be fixed. I hope to 
> look at it in the next few days. There are a couple of issues at the 
> moment with the *nix build system.
>
> *Robert Jongbloed*
> /OPAL/OpenH323/PTLib Architect and Co-founder./
> Commercial support at http://www.voxlucida.com.au
>
> On 13/12/2012 1:19 AM, Alexander Sbitnev wrote:
>>     Hi. I still advocate for resolution of one nasty bug in compile system.
>> Today i've found a correct source of disturbance. make/opal_defs.mak.in
>> contains:
>>
>> CFLAGS           := -I$(includedir) @CFLAGS@ $(CFLAGS)
>> CXXFLAGS         := -I$(includedir) @CXXFLAGS@ $(CXXFLAGS)
>>
>> With definition of "$includedir" as install directory it's pretty much
>> obvious there is no need for opal installation directory inside opal
>> compilation flags.
>> Let strip it out of opal_defs.mak.in immediately :)    At least on my
>> setup it compiled ok without "-I$(includedir)" in place.
>>