Skip to content

Unity Build Cleanup - #128

Merged
UnityAlex merged 4 commits into
unity-mainfrom
build-cleanup
Mar 17, 2023
Merged

Unity Build Cleanup#128
UnityAlex merged 4 commits into
unity-mainfrom
build-cleanup

Conversation

@UnityAlex

@UnityAlexUnityAlex commented Mar 15, 2023

Copy link
Copy Markdown
Collaborator

This change will remove the need for the following scrips:

  • .yamato/scripts/test_linux.sh
  • .yamato/scripts/test_osx.sh
  • .yamato/scripts/test_windows.cmd
  • .yamato/scripts/build_linux.sh
  • .yamato/scripts/build_osx.sh
  • .yamato/scripts/build_windows.cmd

They are replaced with:

  • .yamato/scripts/build_yamato.cmd
  • .yamato/scripts/build_yamato.sh

All of the build logic has been moved into unity/CITools/BuildDriver to have a single source of truth.

Comment thread.yamato/build_linux_x64_debug.yml Outdated
@UnityAlex
UnityAlexforce-pushed the build-cleanup branch 2 times, most recently from f61c8c8 to 431a211CompareMarch 16, 2023 15:35
Use local 7z when not running on yamato
Need to pass special flag to ensure our args are only passed to the build script.
@UnityAlexUnityAlex changed the title Initial commit for build workUnity Build CleanupMar 16, 2023
@UnityAlex
UnityAlexforce-pushed the build-cleanup branch 3 times, most recently from 8daaa9e to 727524aCompareMarch 16, 2023 21:42
Comment threadunity/CITools/BuildDriver/CoreCLR.cs Outdated
Comment threadunity/CITools/BuildDriver/CoreCLR.cs Outdated
Comment threadunity/CITools/BuildDriver/EmbeddingHost.cs Outdated
Comment threadunity/CITools/BuildDriver/EmbeddingHost.cs Outdated
Comment threadunity/CITools/BuildDriver/Program.cs Outdated
Comment threadunity/CITools/BuildDriver/Program.cs Outdated
Removing / updating TODO's after investigation.
@UnityAlex
UnityAlex merged commit add045c into unity-mainMar 17, 2023
@UnityAlex
UnityAlex deleted the build-cleanup branch March 17, 2023 16:05
Sign up for freeto 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

@UnityAlex@mrvoorhe