Skip to content

[MacOS] Normalize native app bundle paths - #2790

Open
vvitovec wants to merge 2 commits into
SubnauticaNitrox:masterfrom
vvitovec:codex/macos-app-layout
Open

[MacOS] Normalize native app bundle paths#2790
vvitovec wants to merge 2 commits into
SubnauticaNitrox:masterfrom
vvitovec:codex/macos-app-layout

Conversation

@vvitovec

@vvitovec vvitovec commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Normalize native macOS Steam game paths so selecting the Subnautica folder or Subnautica.app resolves to Subnautica.app/Contents.
  • Keep the existing host executable layout checks for non-macOS paths.
  • Keep the new layout detection internals private, so callers still use HasValidGameFolder, HasGameExecutable, and NormalizeGamePath.

Context

This PR is intentionally limited to the native macOS app-bundle layout. I dropped Wine fallback/emulation work from this stack after maintainer feedback, because the macOS game build is native.

Validation

  • git diff --check
  • Not run locally: .NET tests, because dotnet is not installed in this local environment.

@vvitovec
vvitovec force-pushed the codex/macos-app-layout branch from 4be97c1 to d37ec81 Compare July 3, 2026 21:47
@vvitovec vvitovec changed the title [codex] Normalize native macOS app bundle paths [MacOS] Normalize native app bundle paths Jul 3, 2026
@vvitovec

vvitovec commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

Rebased this onto current master and narrowed it to native macOS app-bundle path normalization only. I also kept the layout/executable lookup details private so the public helper surface stays small. No Wine fallback or emulation support is included here, following the feedback on #2794.

@vvitovec
vvitovec marked this pull request as ready for review July 3, 2026 21:58
@vvitovec

vvitovec commented Jul 5, 2026

Copy link
Copy Markdown
Contributor Author

The PR check appears to have been cancelled while it was still queued for the self-hosted runner. I tried to rerun it from my side, but GitHub requires repository admin rights for that. Could a maintainer rerun the check when a runner is available?

@Measurity

Copy link
Copy Markdown
Collaborator

We don't have anyone hosting the runner. I'm cancelling all the jobs until we can host it again. Don't have an ETA.

@vvitovec
vvitovec force-pushed the codex/macos-app-layout branch from d37ec81 to 275411a Compare July 6, 2026 20:14
@github-actions

Copy link
Copy Markdown

Test Results

260 tests   253 ✅  14s ⏱️
  1 suites    7 💤
  1 files      0 ❌

Results for commit 275411a.

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