Bug #107023 for Geo-GDAL: Allow environment variables for Makefile.PL arguments

Thu Sep 10 20:05:44 2015 SLAFFAN [...] cpan.org - Ticket created

Subject:

Allow environment variables for Makefile.PL arguments

Hello Ari, Geo::GDAL is difficult to install using cpanm for most of my use cases, mainly because gdal is installed in local directories or there are minor version mismatches. The main reason is that Makefile.pl needs arguments to be passed to provide the location of the gdal-config file, and also version checks. What do you think of adding support for environment variables which can be used in the absence of command line arguments? For example, --gdal-config could be PERL_GEOGDAL_CONFIG or similar, and --no-version-check PERL_GEOGDAL_NO_VERSION_CHECK. I can work something up if you agree. Regards, Shawn.

Fri Sep 11 02:57:22 2015 ari.jolma [...] gmail.com - Correspondence added

On Thu Sep 10 20:05:44 2015, SLAFFAN wrote: Show quoted text

> Hello Ari, > > Geo::GDAL is difficult to install using cpanm for most of my use > cases, mainly because gdal is installed in local directories or there > are minor version mismatches. > > The main reason is that Makefile.pl needs arguments to be passed to > provide the location of the gdal-config file, and also version checks. > > What do you think of adding support for environment variables which > can be used in the absence of command line arguments? > > For example, --gdal-config could be PERL_GEOGDAL_CONFIG or similar, > and --no-version-check PERL_GEOGDAL_NO_VERSION_CHECK.

I can do that. I updated yesterday the GDAL wiki page for Perl https://trac.osgeo.org/gdal/wiki/GdalOgrInPerl and one thing to note is that sometimes the GDAL in the system is from older branch. Ari Show quoted text

> > I can work something up if you agree. > > > Regards, > Shawn.

Fri Sep 11 02:57:22 2015 The RT System itself - Status changed from 'new' to 'open'

Fri Sep 11 03:54:44 2015 ari.jolma [...] gmail.com - Taken

Fri Sep 11 23:28:25 2015 SLAFFAN [...] cpan.org - Correspondence added

On Fri Sep 11 02:57:22 2015, AJOLMA wrote: Show quoted text

> On Thu Sep 10 20:05:44 2015, SLAFFAN wrote:

> > Hello Ari, > > > > Geo::GDAL is difficult to install using cpanm for most of my use > > cases, mainly because gdal is installed in local directories or there > > are minor version mismatches. > > > > The main reason is that Makefile.pl needs arguments to be passed to > > provide the location of the gdal-config file, and also version > > checks. > > > > What do you think of adding support for environment variables which > > can be used in the absence of command line arguments? > > > > For example, --gdal-config could be PERL_GEOGDAL_CONFIG or similar, > > and --no-version-check PERL_GEOGDAL_NO_VERSION_CHECK.

> > I can do that. > > I updated yesterday the GDAL wiki page for Perl > > https://trac.osgeo.org/gdal/wiki/GdalOgrInPerl > > and one thing to note is that sometimes the GDAL in the system is from > older branch. > > Ari >

> > > > I can work something up if you agree. > > > > > > Regards, > > Shawn.

Thanks Ari, The system GDAL being different is the main reason I use the gdal-config argument. Currently I need to compile a local GDAL, as the non-experimental Ubuntu distribution is still at 1.10. See also https://travis-ci.org/shawnlaffan/bd_travis_experiment and the number of install commands need in the .travis.yml file to get it to work. https://github.com/shawnlaffan/bd_travis_experiment/blob/master/.travis.yml Shawn.

Sat Sep 12 04:11:14 2015 SREZIC [...] cpan.org - Cc SREZIC added

Mon Sep 14 02:45:11 2015 ari.jolma [...] gmail.com - Correspondence added

On Fri Sep 11 23:28:25 2015, SLAFFAN wrote: Show quoted text

> > The system GDAL being different is the main reason I use the gdal- > config argument. Currently I need to compile a local GDAL, as the non- > experimental Ubuntu distribution is still at 1.10. > See also https://travis-ci.org/shawnlaffan/bd_travis_experiment and > the number of install commands need in the .travis.yml file to get it > to work. > https://github.com/shawnlaffan/bd_travis_experiment/blob/master/.travis.yml

Ok. Your perlbrew commands do not seem to work. I've tried to fix this issue in trunk and in 2.0 branch. The new 2.00003 in CPAN contains the fixes. It also contains fixes for the other bug reported here. Makefile.PL now executes gdal-config if it is not the one in the same GDAL distro tree. I don't know why you get rat errors (RasterAttributeTable). I'll look into that. Ari Show quoted text

> > Shawn.

Mon Sep 14 02:45:25 2015 ari.jolma [...] gmail.com - Status changed from 'open' to 'resolved'

Mon Sep 14 02:45:26 2015 ari.jolma [...] gmail.com - Fixed in 2.00003 added

Mon Sep 14 03:37:10 2015 SLAFFAN [...] cpan.org - Correspondence added

On Mon Sep 14 02:45:11 2015, AJOLMA wrote: Show quoted text

> On Fri Sep 11 23:28:25 2015, SLAFFAN wrote:

> > > > The system GDAL being different is the main reason I use the gdal- > > config argument. Currently I need to compile a local GDAL, as the > > non- > > experimental Ubuntu distribution is still at 1.10. > > See also https://travis-ci.org/shawnlaffan/bd_travis_experiment and > > the number of install commands need in the .travis.yml file to get it > > to work. > > https://github.com/shawnlaffan/bd_travis_experiment/blob/master/.travis.yml

> > Ok. Your perlbrew commands do not seem to work. > > I've tried to fix this issue in trunk and in 2.0 branch. The new > 2.00003 in CPAN contains the fixes. It also contains fixes for the > other bug reported here. Makefile.PL now executes gdal-config if it is > not the one in the same GDAL distro tree. > > I don't know why you get rat errors (RasterAttributeTable). I'll look > into that. > > Ari >

> > > > Shawn.

Thanks Ari, I'll update the travis checks to use GDAL 2 and Geo::GDAL 2.00003 and see how it goes. Shawn.