-
-
Notifications
You must be signed in to change notification settings - Fork 335
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
[Bug] Windows OSGeo4W builds seem to report outdated build info #3739
Comments
With 8.4-314-xxxx-1, g.version with all the flags report a library build date of 2024-05-27 with a certain time, and the build date of the software is also 2024-05-27. There is still the revision that is set to "exported". |
I wonder if adding libsclean in addition to distclean would make sure everything is recompiled. grass/mswindows/osgeo4w/package.sh Lines 138 to 142 in f598510
I remember some issues with contributor's PRs where this was a problem, as clean isn't enough sometimes, and distclean isn't cleaning everything. One of these two must be done before the other (written before as one of the list of targets passed to make), as once the Platform.make is cleaned, the rest doesn't run since Platform.make is trying to be included but doesn't exist anymore. |
tested with
|
BTW: running
The architecture isn't shown and the absolute path is also wrong (current directory missing from path). |
Describe the bug
When filing in my recent bugs reports on Windows, I observed that the libgis_date, libgis_revision, revision, build_date fields don't seem updated, and mention 2024-04-14 or in the 2024-04-13 day (when there were big changes in the build environment). Assuming that this April 14, 2024, is after the changes in configure to use the git dates (#3435, March 24, 2024), I expect that these would be up to date.
Looking at the most recent commit that have files that appear in the installed source, I see the changes 132755b (2024-05-22) included in the latest nightly for f4d8c62 (2024-05-23), so the source actually changes.
To reproduce
g.version -ecxrgb
orgrass84 --exec g.version -ecxrgb
respectively (I find it weird that it requires grass84, not grass like the docs, I thought we had simplified this).Kölnstrasse 99
in the address shows up asKölnstrasse 99
, the o with two dots is shown as Latin Capital Letter A with Tilde followed by Pilcrow Sign) Note: it is not Windows sandbox and that the old console host is used, on my main os, through the new windows terminal too it has thisExpected behavior
Build info version is correct, especially as the bug reports that will come in won't be useful if the versions are wrong
Screenshots
System description
Additional context
I finished a PR adding extra info to the
.github/workflows/print_versions.sh
script, and making sure it gets ran on Linux, macOS and Windows CI, and we see that the fields are updated. I'm posting it right after.The text was updated successfully, but these errors were encountered: