Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fails to build with PROJ 8.0.0 #893

Closed
sebastic opened this issue Feb 21, 2021 · 5 comments
Closed

Fails to build with PROJ 8.0.0 #893

sebastic opened this issue Feb 21, 2021 · 5 comments
Labels
help wanted Extra attention is needed

Comments

@sebastic
Copy link

sebastic commented Feb 21, 2021

PROJ 8.0.0 has removed proj_api.h causing CMakeModules/FindLibproj4new.cmake to fail:

 -- Could NOT find LIBPROJ4 (missing: LIBPROJ4_LIBRARIES LIBPROJ4_INCLUDE_DIR)

src/projections.hpp needs to be ported to use proj.h.

See also: https://proj.org/usage/differences.html

@GillesDuvert
Copy link
Contributor

Many thanks,
This was bound to happen one day...
I note it is not mentioned (yet?) in the link you sent.
Are you in position to test that using proj.h instead of proj_api.h in src/projections.hpp solves the problem?
More generally, we (I at least, as I wrote it) are interested in returns about the map support in GDL since the rewrite of Aprl 2019.

@sebastic
Copy link
Author

sebastic commented Feb 25, 2021

One of the maintainers of the gnudatalanguage package in Debian (e.g. @olebole) should be able to test such changes better than me, but I can see what I can do in that respect as the maintainer of the proj package in Debian.

If you need help porting to the new PROJ API, the proj mailinglist is a good venue.

Just replacing proj_api.h with proj.h won't suffice, refer to the migration documentation for details:

https://proj.org/development/migration.html

@GillesDuvert
Copy link
Contributor

hm, some work indeed. Thanks.

@opoplawski
Copy link
Contributor

FWIW - We're hoping to get 8.0.0 into Fedora soon.

@GillesDuvert GillesDuvert added the help wanted Extra attention is needed label Mar 9, 2021
@GillesDuvert
Copy link
Contributor

Thanks Orion, I do not see anybody working on this small patch and I do not have time for that either.
We are going to have a big release anytime now, plot and widget oriented, it would be a pity the projections were not available.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants