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

option "--monitor=right" ignored after update from 1.6.0 to 1.8.0 #258

Open
cobour opened this issue Mar 15, 2023 · 5 comments
Open

option "--monitor=right" ignored after update from 1.6.0 to 1.8.0 #258

cobour opened this issue Mar 15, 2023 · 5 comments
Labels
bug Something isn't working

Comments

@cobour
Copy link

cobour commented Mar 15, 2023

I used the extension v1.6.0 with these options in launch.json:

            "--fullscreen=1",
            "--monitor=right"

The fullscreen-option is still working, but the monitor-option is ignored (due to new fork of fs-uae?).

Issue found on macOS 13.2.1 using VSC 1.76.1 (and extension 1.8.0).

@prb28
Copy link
Owner

prb28 commented Mar 16, 2023

@grahambates
Copy link
Contributor

My guess is that it's not yet implemented in FS-UAE 4.x, which the new fork is based on. This is a pre-release version and may still be missing some features. I'll try it out on the official build to confirm.

If this (or any other missing feature) is a show-stopper, we could look at porting the Barto debugger implementation to 3.X but I'm guessing this will be a lot more effort. I was hoping to avoid this in order to keep the patches for FS-UAE and WinUAE consistent. It also would prevent us taking advantage of the profiler tools, as this relies on internal data structures only present in 4.2+.

@cobour
Copy link
Author

cobour commented Mar 16, 2023

In my opinion this is NOT a show-stopper, it is just a bit annoying. I now use the window mode again for the launchers and wait for a newer version that supports the monitor option again correctly.
Thanks for your amazing work on this extension! I love to use it...

@prb28
Copy link
Owner

prb28 commented Mar 17, 2023

I agree, it's better to wait a stable 4 release and do not create a 3 back port. The old versions of the extension are still accessible.

@grahambates
Copy link
Contributor

I can confirm that the issue is present in the non-patched FS-UAE 4.x.

@prb28 prb28 added the bug Something isn't working label Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants