Skip to content

chore: respect the new folder structure inside of Unity#2667

Open
bitsandfoxes wants to merge 3 commits intomainfrom
chore/new-unity-structure
Open

chore: respect the new folder structure inside of Unity#2667
bitsandfoxes wants to merge 3 commits intomainfrom
chore/new-unity-structure

Conversation

@bitsandfoxes
Copy link
Copy Markdown
Contributor

@bitsandfoxes bitsandfoxes commented May 6, 2026

The layout inside of the Unity installation changed and we need to update the path resolution for the dependencies the SDK has at build time. Since mucking around inside the .targets is a pain I extracted this into a single script that gets invoked by MSBuild.

Added the new paths to the .targets.
Old layout (Unity ≤ 6000.1.x, macOS):

  /Applications/Unity/Hub/Editor/<version>/Unity.app/
  └── Contents/
      ├── MacOS/
      │   └── Unity
      ├── Managed/
      │   ├── UnityEngine.dll
      │   ├── UnityEditor.dll
      │   └── UnityEngine/
      │       └── UnityEngine.*.dll (modules)
      └── Resources/
          └── PackageManager/...

New layout (Unity 6000.3.x, macOS):

  /Applications/Unity/Hub/Editor/<version>/Unity.app/
  └── Contents/
      ├── MacOS/
      │   └── Unity
      └── Resources/
          ├── PackageManager/...
          └── Scripting/
              └── Managed/
                  ├── UnityEngine.dll
                  ├── UnityEditor.dll
                  └── UnityEngine/
                      └── UnityEngine.*.dll (modules)

@bitsandfoxes bitsandfoxes requested a review from a team May 7, 2026 09:39
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit e351381. Configure here.

Comment thread Directory.Build.targets Outdated
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.

1 participant