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

Demo "Old Men in Used Cars" by "Fossil" stops working with V5.1RC2 #135

Open
paich64 opened this issue Apr 28, 2024 · 1 comment
Open

Demo "Old Men in Used Cars" by "Fossil" stops working with V5.1RC2 #135

paich64 opened this issue Apr 28, 2024 · 1 comment
Labels
bug Something isn't working regression V6 or later

Comments

@paich64
Copy link

paich64 commented Apr 28, 2024

Demo "Old Men in Used Cars" by "Fossil" stops working with V5.1RC2 :

When inserting disk n°2, the first picture gets loaded and the music plays, but the visual effects just stop.

Since there is an already exisiting issue with V5.1RC2 issue 133 I have tried the demo with the V5.1A16 core which fixes issue 133, and the demo runs perfectly as with V5.0.

summary :

"Old Men in Used Cars" by "Fossil" works fine with V5.0
"Old Men in Used Cars" by "Fossil" stops working with V5.1RC2
"Old Men in Used Cars" by "Fossil" works fine with V5.1A16

@sy2002
Copy link
Collaborator

sy2002 commented May 17, 2024

My hypothesis is, that this regression was introduced by the GEOS RTC feature that has been introduced in V5.1. A strong data point that backs this hypothesis is, that V5.1A16 resolves the issue: Alpha 16 was a never-released Alpha version that did nothing else than deactivating the RTC that uses the C64's tape port. So it looks like this demo does not like if the tape port is active while it runs, just like TRAP16 in the emulator test suite: #133

The decision is to live with this regression in V5.1 as the benefit of having a RTC for GEOS is larger than this singular demo. In future, we will have advanced compatibility settings and will be able to switch the RTC on/off. Today the default is ON and in future it might be OFF just to be more compatible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working regression V6 or later
Projects
None yet
Development

No branches or pull requests

2 participants