Check and add other programs on oberon07 to autobuild #328
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
After merging this PR #326 , need to check existing programs on oberon07 and add them to the autobuild.
Hi @mxlgv, I am working on this! I'm starting by adding the FB2 Reader to the autobuild.
@mxlgv, While adding FB2 Reader to the autobuild system, I noticed an issue that also affects CEdit (merged in PR #326). When running
tup programs/develop/cedit/, it throws this error:tup error: Expected to write to file 'cedit' from cmd 53884 but didn'tThe Oberon compiler does successfully create the executable, but it seems that tup's file tracker doesn't detect this.
As a workaround for this, I added
touch %oto the end of the compile command. This does not overwrite or empty the compiled binary. It just updates the file's timestamp after the compiler finishes, which seems to make tup notice this file.Is this
touchworkaround acceptable, or is there a better way to handle this?@hrigar Have you run the Oberon compiler manually? Have you checked that if the file exists, the timestamp is not updated? I’m ready to assume that this is a floating bug of the tup build system; to be honest, its latest versions do not work stably, but such work has been done that we could not roll back to the previous version. As you can see, the CI build works. However, it’s possible because the build is done from scratch.. Does this happen after adding fb2reader or is main branch not being built?
@mxlgv, Yes, I tested with Oberon compiler. It works correctly. The timestamp is also being updated by tup, so that was my mistake earlier.
This issue happens even on the main branch and is not related to adding fb2reader.
I was working on WSL2 when this error occurred. I tried the same build in a Linux environment (GitHub Codespaces), and it works there without any issues.
So this appears to be a environment-specific issue. while searching about this i found this https://bugzilla.mozilla.org/show_bug.cgi?id=1462493 that explains about his issue.
I will remove the
touchworkaround.@mxlgv Are there any other Oberon-07 programs that need changes, or is this complete? I couldn’t find any others that need changes.