Skip to content

Remove bitrotted X68000 host bridge device#1536

Merged
rdmark merged 1 commit intodevelopfrom
rdmark-remove-x68k-features
Nov 9, 2025
Merged

Remove bitrotted X68000 host bridge device#1536
rdmark merged 1 commit intodevelopfrom
rdmark-remove-x68k-features

Conversation

@rdmark
Copy link
Copy Markdown
Member

@rdmark rdmark commented Nov 8, 2025

The X68000 host bridge and associated host file system module has not been functional for many years, as reported by multiple users

Removing the code will reduce complexity for users - the SCBR device has caused some misunderstandings in the past - while eliminating code that has been flagged as insecure by SonarQube

@rdmark rdmark force-pushed the rdmark-remove-x68k-features branch 2 times, most recently from 7ff299f to b33efe6 Compare November 8, 2025 23:10
The X68000 host bridge and associated host file system module
has not been functional for many years, as reported by multiple users

Removing the code will reduce complexity for users -
the SCBR device has caused some misunderstandings in the past -
while eliminating code that has been flagged as insecure by SonarQube
@rdmark rdmark force-pushed the rdmark-remove-x68k-features branch from 07c0bcd to 2790370 Compare November 8, 2025 23:33
@rdmark rdmark marked this pull request as ready for review November 8, 2025 23:39
@rdmark rdmark requested a review from akuker as a code owner November 8, 2025 23:39
@rdmark rdmark requested a review from nucleogenic November 8, 2025 23:39
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Nov 8, 2025

Copy link
Copy Markdown
Contributor

@stinkerton18 stinkerton18 left a comment

Choose a reason for hiding this comment

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

Reviewed the changes, looks clean with no obvious lingering library files or code.

@rdmark rdmark merged commit b7e7c4e into develop Nov 9, 2025
15 checks passed
@rdmark rdmark deleted the rdmark-remove-x68k-features branch November 9, 2025 07:45
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.

2 participants