Skip to content
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

Latest Inform6 compiler now can do V3 output #1

Closed
DavidGriffith opened this issue May 17, 2020 · 4 comments
Closed

Latest Inform6 compiler now can do V3 output #1

DavidGriffith opened this issue May 17, 2020 · 4 comments

Comments

@DavidGriffith
Copy link

A patch was recently applied to latest Inform6 codebase that restores the ability to generate V3 output. See DavidKinder/Inform6#18

@hlabrand
Copy link

hlabrand commented May 17, 2020

Oh cool, I had no idea!
@ByteProject I merged it in my repository, also containing Barrett's code : https://github.com/hlabrand/Inform6

@ByteProject
Copy link
Owner

@DavidGriffith @hlabrand Thank you gentlemen. I was aware of this patch because I discussed it with Fredrik who discovered the bug and brought the patch on the way. HOWEVER... The problem is this fixes only one bug of many for Z3 generation in compilers that came after 6.15. You can
really see that Z3 targets were not important anymore. Also, there are still parts of Veneer that don't work in Z3. So at this point I will close the issue. I will look into the current compiler again but I don’t have high hopes. Cheers!

@ByteProject
Copy link
Owner

@DavidGriffith @hlabrand After a chat with Zarf on intfiction, he provided a fix for the remainig issues that prevented Z3 files from successfully compiling. A pull request for said fix had been merged with the official Inform6 master branch, which means I regard this issue as valid now and I'm reopening it until I've applied the necessary changes to the repo.

@ByteProject ByteProject reopened this May 20, 2020
@ByteProject
Copy link
Owner

@DavidGriffith @hlabrand The project defaults now to the current compiler 6.34 and the library codebase has been updated to work well with it. I'm closing the issue now gentlemen as the changes are merged to master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants