-
-
Notifications
You must be signed in to change notification settings - Fork 295
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
disable module check since svn keywords are not maintained by git #117
Conversation
|
Important to merge into relbr 7.6 and 7.8 before new releases (7.6.2 & 7.8.0). |
|
Related ticket: https://trac.osgeo.org/grass/ticket/3880 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Until we find a better solution, I'd suggest to comment the version test based on SVN rev tag
|
This was reverted in I am not convinced that it should be reverted. Rationale: Line 41 in 8b9512b
|
* docker alpine: removed gisinit revision workaround gisinit revision patch in #278 obsoleted by ec9b6f2 See: https://trac.osgeo.org/grass/ticket/3880 (also improves #117)
* docker alpine: removed gisinit revision workaround gisinit revision patch in #278 obsoleted by ec9b6f2 See: https://trac.osgeo.org/grass/ticket/3880 (also improves #117)
|
Finally addressed in ec9b6f2 (and backports) |
* docker alpine: removed gisinit revision workaround gisinit revision patch in OSGeo#278 obsoleted by ec9b6f2 See: https://trac.osgeo.org/grass/ticket/3880 (also improves OSGeo#117)
* docker alpine: removed gisinit revision workaround gisinit revision patch in OSGeo#278 obsoleted by ec9b6f2 See: https://trac.osgeo.org/grass/ticket/3880 (also improves OSGeo#117)
Module check is currently broken (see attached screenshot) since svn keywords are not maintained by git. This PR temporary disables this check. It can be re-enabled when svn keywords issue will be solved.