Skip to content

AltairZ80: Improved choice of boot ROM#512

Closed
psco wants to merge 26 commits into
open-simh:masterfrom
psco:romchoice
Closed

AltairZ80: Improved choice of boot ROM#512
psco wants to merge 26 commits into
open-simh:masterfrom
psco:romchoice

Conversation

@psco

@psco psco commented Oct 26, 2025

Copy link
Copy Markdown
Contributor

An appropriate boot ROM is chosen based on the attached disk image.

Replaces the PR with same name and functionality but without the commit clutter.

psco and others added 24 commits November 26, 2024 09:45
Adds support for the ISO/European "left backslash" or "less than" key.
It's found between left shift and Z.
This emulates the classic LINC.  The design was settled in 1965,
increasing memory to 2048 words, and adding a Z register, an overflow
flag, and an interrupt facility.
These are tests for the classic LINC, including the features new in
1965: 2048 words of memory, a Z register, an overflow flag, and an
interrupt facility.
Block 45 held damaged data, and block 51 was identical to block 52.

@LegalizeAdulthood LegalizeAdulthood left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. I suggest that you squish all your commits down to a single commit to eliminate all the merges that are just updating your branch to master.

Can we add any sort of simh script test that would validate this functionality?

@LegalizeAdulthood

Copy link
Copy Markdown
Contributor

The ubuntu build failed with:

The following tests FAILED:
	 50 - simh-sel32 (Failed)                               simh-SEL32
Errors while running CTest
*** cmake/cmake-builder.sh: Errors detected during testing. Exiting.
Error: Process completed with exit code 1.

I've also seen this failure on ubuntu; it's not caused by this change.

@psco

psco commented Apr 16, 2026

Copy link
Copy Markdown
Contributor Author

I am not sure what I am supposed to do and how to accomplish it. And no, there are no test scripts for this particular functionality.

@LegalizeAdulthood

Copy link
Copy Markdown
Contributor

I'll leave it up to the owners of the repository to decide if they want that done. Your PR can be merged into master without conflicts, which is the important part.

@psco

psco commented Apr 16, 2026

Copy link
Copy Markdown
Contributor Author

I would appreciate if this can be merged as is - I have almost forgotten that this PR even exists.

@LegalizeAdulthood

Copy link
Copy Markdown
Contributor

Unfortunately I don't have rights to do that -- if the repository owner would like to discuss with me how to merge this without merge commits (e.g. rebase first), I can help with that.

Personally I prefer a linear history and no merge commits, but that is up to the repository owner.

@pmetzger

Copy link
Copy Markdown

I'll work on merging it into the ZIMH fork.

@psco

psco commented May 2, 2026

Copy link
Copy Markdown
Contributor Author

@pkoning2 I think this can be merged as well - sorry about the many merges.

@psco psco closed this May 3, 2026
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.

5 participants