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

Pull request (GLCC update) #248

Merged
merged 2 commits into from Jul 7, 2023
Merged

Conversation

lb3361
Copy link
Contributor

@lb3361 lb3361 commented Jul 5, 2023

This is a combined pull-request that could be split as needed.

  • The first commit upgrades GLCC to the latest version

  • The next seven commits (up to 78f3021) are a ROMv6 release. The native code itself is essentially dev rom, that is, an approximation of what ROMv5 could have been (bugs fixed plus good support for expansion cards). The application selection removes pictures and tetronis, adds invaders (pristine) and gamine. New credits application gives proper credits for tall this. At this point the dev rom is the same as ROMv6 save for romTypeValue.

  • The last tree commits upgrade the dev rom to dev7, without changing the application selection other than a slightly faster Mandelbrot (found another 25%) and having a version of MSCP in Apps (but not included in the ROM)

All three items can be discussed independently to best satisfy the wishes of our community.
I'll update the pull request accordingly.

@at67
Copy link
Collaborator

at67 commented Jul 5, 2023

Pull request won't be accepted as per https://forum.gigatron.io/viewtopic.php?t=19, I don't have the hours required to review the amount of changes you have made to the root/core.

The simple solution is to refactor it all so that it sits in your Contrib area, as everyone else, (including myself has done), see the above forum thread.

@lb3361
Copy link
Contributor Author

lb3361 commented Jul 5, 2023

I propose to reshuffle the commit as follows:

  1. Move GLCC under Contrib/lb3361 and update it there.

  2. Keep the ROMv6 changes into the main directories, because (a) it is a small change in terms of code, (b) it is a fair approximation of what we know Marcel had in mind, and (c) because there is value in providing a stable development target to app developers, with properly versioned syscalls and interface.json. This would improve the life expectancy of the official gigatron rom, which I see as a benefit our community

  3. Move all Dev7 developments under Contrib/lb3361.

@lb3361 lb3361 changed the title Pull request (three items) Pull request (GLCC update) Jul 6, 2023
@lb3361
Copy link
Contributor Author

lb3361 commented Jul 6, 2023

This pull request has been reduced to only contain the GLCC update (one commit, entirely inside Compilers/glcc).
The other changes will be turned into separate pull requests, each with proper argumentation.

@at67 at67 merged commit 5d24889 into kervinck:master Jul 7, 2023
@lb3361 lb3361 deleted the pull-request-v6-dev7 branch July 8, 2023 12:26
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

Successfully merging this pull request may close these issues.

None yet

2 participants