Skip to content

Add yamato configurations for building and uploading artifact to Stevedore. #42

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Feb 25, 2022

Conversation

brian-taylor-unity
Copy link
Collaborator

@brian-taylor-unity brian-taylor-unity commented Feb 23, 2022

Added yamato configurations for uploading runtime artifacts to stevedore for Windows x86, Windows x64, OSX x64, and Linux x64.

I separated the build+test yamato configs into separate build and test configs, where the test and publish-to-stevedore configs are dependent on the build configs.

Build configs will generate artifacts for both the full directory structure and for just the runtime 7z we use for stevedore. I have the build configs generate the stevedore artifacts instead of the publish-to-stevedore configs because a developer might want to get a built .7z for local testing before actually publishing that build to stevedore.

Test configs will still trigger on any PR targeting trunk, but publish-to-stevedore configs are all manual. There is a top-level "Publish to Stevedore" config that will trigger all of the platforms' publish-to-stevedore configs.

Where we publish the artifacts to (testing, unity-internal, or public) is set and can be changed in .yamato\Globals.metafile.

@brian-taylor-unity brian-taylor-unity force-pushed the yamato-create-stevedore-artifacts branch 8 times, most recently from 0ba9ee3 to e0ff9f7 Compare February 24, 2022 03:27
@brian-taylor-unity brian-taylor-unity force-pushed the yamato-create-stevedore-artifacts branch 3 times, most recently from 0d51bf2 to 56e0717 Compare February 24, 2022 18:00
@brian-taylor-unity brian-taylor-unity force-pushed the yamato-create-stevedore-artifacts branch from 56e0717 to 41f1bd8 Compare February 25, 2022 16:55
@brian-taylor-unity
Copy link
Collaborator Author

I'll need to change the Github PR checks after merging this to reflect the new names of yamato configs.

Copy link
Collaborator

@UnityAlex UnityAlex left a comment

Choose a reason for hiding this comment

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

LGTM assuming all the checks pass

@brian-taylor-unity brian-taylor-unity merged commit 7d2fc50 into unity-main Feb 25, 2022
@brian-taylor-unity brian-taylor-unity deleted the yamato-create-stevedore-artifacts branch February 25, 2022 21:44
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.

3 participants