[ODE] Fwd: [ opende-Bugs-1572419 ] ode-config --cflags : wrong path

[Rodrigo Hernandez]

Ok, I see your point, however, config.h is not being correctly used in 
ODE, config.h is meant for compile time definitions, and not meant to be
part of a binary distribution, I guess using ode/ode.h is a good 
workaround for as long as we have this other problem.


Bram Stolk wrote:
> Rodrigo Hernandez wrote:
>> I think the correct way would be the way SDL does it, that is 
>> #include "ode.h"
>> and have -Iinclude/ode on the command line, using <> should be for 
>> system libraries, and
>> keeping the /ode path out of the include allows you to place the 
>> headers on a different path.
>
> I think that will not work.
>
> Take e.g. ode/config.h
>
> Include it as "config.h" and you have absolutely no idea which config.h
> you'll be getting. Most application authors would probably expect 
> including
> their own config.h, and not ODE's.
>
> Also, dropping the ode/ prefix will not work, as all ODE's includes
> use the ode/ prefix as well (as they should be using).
>
> So I'm still convinced that ode/ode.h is the way to include.
>
>   Bram
>
>
>>
>> But thats just me :)
>>
>> Bram Stolk wrote:
>>> Very valid point by bug reporter.
>>> I just fixed this in SVN.
>>>
>>> People that still use
>>> #include <ode.h> should move to the proper directive of
>>> #include <ode/ode.h>
>>>
>>>    Bram
>>>
>>> _______________________________________________
>>> ODE mailing list
>>> ODE at q12.org
>>> http://q12.org/mailman/listinfo/ode
>>>
>>>   
>>
>
>