Skip to content

Conversation

thomas
Copy link
Collaborator

@thomas thomas commented Jun 21, 2019

Symptom: Packages that build against libgdal fail compilation
tests.

Example (taken from autoconf test):

echo -e \
"#include <gdal.h>\nextern \"C\" {int main(){GDALAllRegister();}}" | \
  g++ -x c++ - -o junk -std=gnu++11 \
  $(echo `gdal-config --cflags` `gdal-config --libs` \
  `gdal-config --dep-libs`)

Giving undefined references to:

'CRYPTO_get_id_callback' and
'EVP_MD_CTX_create'

Found/referenced in:

./boost/asio/ssl/detail/impl/openssl_init.ipp
./openssl/crypto.h

and

./openssl/cms.h
5 more
./openssl/x509.h

So, time to rebuild boost too?

Symptom: Packages that build against libgdal fail compilation
tests.

Example (taken from autoconf test):

echo -e \
"#include <gdal.h>\nextern \"C\" {int main(){GDALAllRegister();}}" | \
  g++ -x c++ - -o junk -std=gnu++11 \
  $(echo `gdal-config --cflags` `gdal-config --libs` \
  `gdal-config --dep-libs`)

Giving undefined references to:

  'CRYPTO_get_id_callback' and
  'EVP_MD_CTX_create'

Found/referenced in:

  ./boost/asio/ssl/detail/impl/openssl_init.ipp
  ./openssl/crypto.h

and

  ./openssl/cms.h
    5 more
  ./openssl/x509.h

So, time to rebuild boost too?
@pmenzel
Copy link
Collaborator

pmenzel commented Jun 21, 2019

Please mention that you update the version too.

Additionally, I tested with the R package sf a user needs to install as a dependency. We should definitely install the shared versions of the libraries, and we need some data stuff too.

When updating, make sure to specify the version, as I made version 2.4.1 available during my tests.

@pmenzel
Copy link
Collaborator

pmenzel commented Jun 21, 2019

Also s/depencies/dependencies/.

@thomas
Copy link
Collaborator Author

thomas commented Jun 21, 2019

Concerning the update, two hints:)
a) branch name,
b) the diff

It makes no sense to have gdal as a shared lib in the system, otherwise we must track it as dependency.

I made version 2.4.1 available

I hope you didn't install it everywhere.

The 'missing' data from gdal is the cause for the 'not ready for merge'.

@thomas
Copy link
Collaborator Author

thomas commented Jun 27, 2019

For the rather uncommon case that a tool that requires functionalities provided by gdal also checks for the geo data files, they are now available under /package/gis/share/gdal.

Having this available, and and having '/usr/lib/libgdal.a' fixed (what was the primary aim of this PR), version 2.3.3 could be installed.

@thomas thomas merged commit cfc2ed2 into master Jun 27, 2019
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants