Skip to content

AltairZ80: Improved choice of boot ROM#512

Open
psco wants to merge 24 commits intoopen-simh:masterfrom
psco:romchoice
Open

AltairZ80: Improved choice of boot ROM#512
psco wants to merge 24 commits intoopen-simh:masterfrom
psco:romchoice

Conversation

@psco
Copy link
Copy Markdown
Contributor

@psco psco commented Oct 26, 2025

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.
Copy link
Copy Markdown

@LegalizeAdulthood LegalizeAdulthood left a comment

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

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
Copy link
Copy Markdown
Contributor Author

psco commented Apr 16, 2026

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

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
Copy link
Copy Markdown
Contributor Author

psco commented Apr 16, 2026

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

@LegalizeAdulthood
Copy link
Copy Markdown

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.

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.

4 participants