-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[fortran] fix bug when copying/memmoving string #19236
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
base: master
Are you sure you want to change the base?
Conversation
happens maybe only in some gfortran versions in mac13 and mac14 CI, the last bytes of chopt were being set to garbage memory, which later led to failures due to wrong options being set. chopt was being set to "px exam" where choptt was "px " and "exam" was the first chars of the parameter PASSCHAR("example") on the C side in h2root.cxx
now that it's fixed on Fortrans side
now that bug is gone
Test Results 19 files 19 suites 3d 12h 6m 28s ⏱️ For more details on these failures, see this check. Results for commit 6622027. ♻️ This comment has been updated with latest results. |
I think instead of guessing around and applying more and more changes, we should carefully understand the problem in depth and then fix it for good |
Maybe before even doing that, we should be clear on whether we actually want to support But we see now it's still fragile, and the binaries we provide only work when the user installed IMHO we need to have a clear strategy on which features of ROOT are supported on which platforms, because we don't necessarily need to support everything everywhere. Attempting that is taking a lot of time and effort, and the features that our users use vary greatly depending on the platform they're on. Once we have decided on these things, we can also make the support status more explicit by making certain configurations error out on certain platforms. For example, on Windows, So I agree with @hahnjo that we should not necessarily try to guess our way out of the problems, but I think we should go one step further and have a more general discussion on what we support on which platform. Possibly in the next ROOT team meeting. |
This Pull request:
Changes or fixes:
happens maybe only in some gfortran versions
Maybe related: https://www.tek-tips.com/threads/string-copy.1609892/post-6335242
in mac13 and mac14 CI, the last bytes of chopt were being set to garbage memory, which later led to failures due to wrong options being set. chopt was being set to "px exam"
where choptt was "px " and "exam" was the first chars of the parameter PASSCHAR("example") on the C side in h2root.cxx
mac14 still fails (with another error), but mac13 h2root works now.
mac13: there is still the error in rpath for two tests.
mac15: failures due to runners having different gfortran versions?