-
Notifications
You must be signed in to change notification settings - Fork 299
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
sf 1.0-17 fails to install with GDAL < 3.4: gdal.cpp:713:16: error: ‘ret’ was not declared in this scope #2436
Comments
my gdal version is 3.3.0 gdal.cpp: In function ‘Rcpp::NumericVector CPL_transform_bounds(Rcpp::NumericVector, Rcpp::List, int)’: |
@edzer is cycling to Prague: https://mastodon.social/@edzer/113114291930092331. I'll try to put up a PR. IIUC @HenrikBengtsson has GDAL 3.0.4, @weathermiya GDAL 3.3.0, so I need not go further back than 3.0.4; as you point out, the problem is obvious. In |
3.0.4 is the version shipped in RHEL8, which is still widely in use, even though RHEL9 is out for a while already. I also encountered the issue there just now. NB: There's usually no other OS being more behind than the "old stable" RHEL due to the long RHEL lifecycles, so keeping compatibility for those might solve the issue for all others, also for future occasions. |
Rebuilding 3.3.0 on current Fedora 40 with g++ 14.2.1 fails with:
Which incantation of CXXFLAGS might persuade current g++ to behave as was expected in May 2021? |
This is a source package tarball: |
gdal3.3.0 install successfully |
#2437 created. |
Work on RHEL8 with GDAL 3.0.4. |
Over then to @edzer. Should I link to this draft source package on the R-sig-geo mailing list in case others are looking for a fix? |
Following the merge, could someone with GDAL < 3.4.0 try |
worked for me with GDAL
|
Installing package into ‘/home/fa079944/R/x86_64-pc-linux-gnu-library/4.2’
Installing package into ‘/home/fa079944/R/x86_64-pc-linux-gnu-library/4.2’
|
Any news on a CRAN submission? HPC users on Rocky 8 are starting to report on this problem (there are ~1500 users on our end). |
Hi,
Any clue? |
Yes, please install from github, e.g. with |
Thanks @edzer.
but it failed because PROJ and GDAL were not found as |
That error might indicate that you have more than one instance (and version) of PROJ installed on your system; see the README.md. |
PROJ 8.2.0 is installed via rpm with RHEL9. But as "sf" needs When trying to install with |
You're hijacking this issue to get help for another, known issue; feel free to open a new issue, but I'm afraid that the general advice is to install |
Finally manage to installed "sf" by initialising |
Hi, v1.0-18 isn't yet available on CRAN, is there anything you can do to speedup the process? Thanks a lot! |
@albertus82, version 1.0.18 is already on CRAN 🎉 |
Issue
This is because
ret
in:sf/src/gdal.cpp
Line 713 in 061196e
is never refined for
GDAL_VERSION_NUM < 3040000
.The text was updated successfully, but these errors were encountered: