-
Notifications
You must be signed in to change notification settings - Fork 11
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
Make7 Error #24
Comments
Hi Thomas (@ThomasMertes), I reverted the changes from this morning and the "make" step completed without error. I have put the log contents below. "make" log with reverted changes:
Kind Regards, |
Hi Liam, The commit "Add support for reading Windows symlinks" caused the problem. It uses the windows function I just committed "Check if GetFinalPathNameByHandleW() is available" in which With this change I can compile with the 32-bit gcc from the Seed7 installer. Unfortunately I tested the Windows symlink change only with a 64-bit gcc. Sorry for the inconvenience. Kind Regards, |
Hi Thomas (@ThomasMertes),
It was not an inconvenience. As noted here, I once worked as a contractor for a major international software company. And despite all the testing, there were still bugs. So, even though I was not a developer, per se, I understand the software development process. Having to revert changes and redo a build process, is not an inconvenience. This is just part of the journey in improving a software product. Thanks for making the changes for the 32-bit gcc. I can confirm that with today's updates, the "make" step now runs without error. Kind Regards, |
Hi Thomas (@ThomasMertes),
This afternoon I was running these commands on Windows following your updates from earlier today (Add support for reading Windows symlinks):
The "make" command failed with these errors:
I have included below the entire log.
The "make" log:
It would appear that the problem has arisen from changes to cmd_win.c (and other related changes) from this morning. The history of the changes to the file, shows that it was last modified on December 3rd, 2022.
Kind Regards,
Liam
The text was updated successfully, but these errors were encountered: