[gdal-dev] include paths and cmake
David Burken
dburken at comcast.net
Fri Nov 4 09:06:43 EDT 2011
On 11/03/2011 09:15 PM, Daniel Morissette wrote:
> Personally I like the idea. I'd probably go for the flattened version
> unless there is a good reason to carry a bunch of sub-directories around.
>
> If there are any serious arguments against doing this then I'd like to
> hear them.
>
> Daniel
>
Keeping the sub directories allows you to develop off checked out code
base with one include path. You could do that with flattened path but
you would have to move the headers in your code tree. I'm all for that
but it's really not up to me. If you move the includes around on the
install then it complicates the include paths when building against the
code tree versus building against an installed version. I hope that
makes sense. Many packages separate the includes in their source tree,
e.g. geos, ossim, opencv. The source tree includes mirror the installed
include tree. So it's less complicated and this makes for easy installs
/ uninstalls.
Dave
>
> On 11-11-03 05:31 PM, Mateusz Łoskot wrote:
>> On 3 November 2011 21:24, David Burken<dburken at comcast.net> wrote:
>>>
>>> Just wondering if you've thought about namespacing the include
>>> paths? That
>>> is:
>>>
>>> #include "cpl_config.h"
>>> #include "gdal_frmts.h"
>>>
>>> Becomes:
>>> #include<gdal/port/cpl_config.h>
>>> #include<gdal/gcore/gdal_frmts.h>
>>>
>>> Or (flattened):
>>> #include<gdal/cpl_config.h>
>>> #include<gdal/gdal_frmts.h>
>>
>> David,
>>
>> I thoroughly agree.
>> Here is related ticket: http://trac.osgeo.org/gdal/ticket/3435
>>
>> IMHO, current way of installing headers is becoming inconvenient and
>> unacceptable...
>>
>>> I realize this would be a big change. Just thought since you're
>>> taking the
>>> plunge on cmake it's something to consider.
>>
>> Yes, but I'm not a decision maker.
>> I can only encourage to collect more critical mass and nudge GDAL PSC
>> about this idea.
>>
>> Best regards,
>
>
More information about the gdal-dev
mailing list