Skip to content

Conversation

@cscd98
Copy link
Contributor

@cscd98 cscd98 commented Jan 26, 2026

Guidelines

  1. Rebase before opening a pull request
  2. If you are sending several unrelated fixes or features, use a branch and a separate pull request for each
  3. If possible try squashing everything in a single commit. This is particularly beneficial in the case of feature merges since it allows easy bisecting when a problem arises

Description

Now that libsmb2 is in griffin, we can use it for apple builds.

Its been added to baseconfig and RetroArch project files. This work is untested as I do not own a macbook.

Makes it more portable for libretro-common

Related Issues

Related Pull Requests

Reviewers

@warmenhoven

@cscd98 cscd98 force-pushed the libsmb2-apple branch 2 times, most recently from 1d4f970 to 6f817f1 Compare January 29, 2026 14:46
@cscd98 cscd98 changed the title smb: enable for (some) apple builds smb: enable for macOS / iOS builds Jan 29, 2026
@cscd98 cscd98 force-pushed the libsmb2-apple branch 3 times, most recently from fcc8056 to 7d2f439 Compare January 29, 2026 14:58
@cscd98 cscd98 changed the title smb: enable for macOS / iOS builds smb: enable for macOS / iOS builds & fix some libretro-common include usage Jan 29, 2026
@cscd98 cscd98 force-pushed the libsmb2-apple branch 4 times, most recently from b8304e3 to 867d60c Compare January 29, 2026 19:16
@cscd98
Copy link
Contributor Author

cscd98 commented Jan 31, 2026

@warmenhoven Any chance you can review this as the libretro-common updates need to get in.

@joaodforce has (I think if its the same person on discord) tested this on apple devices 👍

@warmenhoven
Copy link
Collaborator

I haven't had a chance to test out the Apple changes myself yet but did review the libretro-common change and that is good so I've cherry picked that to master, so the only thing remaining is the Apple specific bits.

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