-
Notifications
You must be signed in to change notification settings - Fork 9
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
grass.session error after merge of #14 #16
Comments
|
Hmmm… On Ubuntu 18.04 With GRASS GIS 7.8 I get a similar error Message: On that machine, the following command installs grass-session without issues: Also, on another Ubuntu box, grass-session 0.4 installs without issue both with python2 and python3 (3.6.8). |
|
On Fedora I have no issues installing grass-session either... The function that throws the error uses ctypes. Maybe there is the culprit? |
|
In above alpine GRASS GIS Dockerimage, I tested |
|
@neteler thank you to open this issue, I was not aware of this problem, I'm on fedora right now and the installation it seems to be fine. The issue is that grass is providing the wrong GISBASE, the command: # grass --config path
/usr/local/grass79Return this directory that do not exist. A possible solution to fix this could be to move the installation of grass-session after the proper link to the GISBASE directory are created, see this commit. However now the installation process is failing because call the ctypes function Do you have an idea on why python fail loading the grass libs with ctypes? For every lib listed above. |
Due to zarch/grass-session#16 all docker builds are currently broken. This workaround pins the last working version by applying the workaround used in #516 To be removed once `grass-session` is back functional.
Due to zarch/grass-session#16 all docker builds are currently broken. This workaround pins the last working version by applying the workaround used in #516 To be removed once `grass-session` is back functional.
|
The docker builds are all failing :( Any chance to revert this PR soon? |
|
Excellent, thanks a lot: I have updated all Dockerfile in master (OSGeo/grass@745ee7e) and releasebranch78 (OSGeo/grass@4f2f21b) accordingly. |
|
Success! Thanks again @zarch |
After the recent merge of #14 the GRASS GIS docker images no longer build:
https://hub.docker.com/repository/registry-1.docker.io/neteler/docker-alpine-grass-gis/builds/f2496f54-56f6-4793-93f5-af27dffbd452
Do we have to update anything here?
https://github.com/OSGeo/grass/blob/master/docker/alpine/Dockerfile_alpine#L195
Thanks...
The text was updated successfully, but these errors were encountered: