Skip to content

Implement VFS support - #575

Merged
sonninnos merged 2 commits into
libretro:masterfrom
cscd98:vfs
Aug 12, 2026
Merged

Implement VFS support#575
sonninnos merged 2 commits into
libretro:masterfrom
cscd98:vfs

Conversation

@cscd98

@cscd98 cscd98 commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

Might also close #553

Only tested SF II over SMB

You can ignore the build failures, this is the first time the github CI has ever been run and needs some tweaks.

@cscd98
cscd98 requested a review from sonninnos August 6, 2026 14:30
@sonninnos

Copy link
Copy Markdown
Collaborator

I don't really have much to say since it is not relevant to my interests and expertise and testing capabilities, other than is the core option really necessary.. never seen it in any other core nor any real reason for it.

@cscd98

cscd98 commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator Author

I only include it because if an user has an issue with it they can disable. This is purely because MAME is huge and I can't ever test all the functionality. But I could remove it.

@sonninnos

Copy link
Copy Markdown
Collaborator

Hmm, let's just request it from the frontend with RETRO_ENVIRONMENT_GET_VFS_INTERFACE as usual and not force via core option.

@cscd98

cscd98 commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator Author

It's not forced, it requests VFS via the interface. Its just an option to disable that request. I'll remove it.

@cscd98

cscd98 commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator Author

Core option removed

@cscd98

cscd98 commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator Author

Are you happy to merge this? We will need to clear mame cache on gitlab when we do. (experience from hbmame same change)

@sonninnos
sonninnos merged commit eb34274 into libretro:master Aug 12, 2026
3 of 5 checks passed
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.

saf paths no longer load in this core.

2 participants