Skip to content
This repository was archived by the owner on Jan 11, 2024. It is now read-only.

Update submodules from ProdCon orchestrated build manifests#1968

Merged
dagood merged 1 commit into
dotnet:masterfrom
dagood:submodule-manifest-update/master
Mar 20, 2018
Merged

Update submodules from ProdCon orchestrated build manifests#1968
dagood merged 1 commit into
dotnet:masterfrom
dagood:submodule-manifest-update/master

Conversation

@dagood
Copy link
Copy Markdown
Member

@dagood dagood commented Mar 19, 2018

This will be used by source-build to move the submodules to the hashes that were successful in the latest orchestrated build.

Setting this up in source-build: dagood/source-build@4791893. To avoid duplicating info outside the project files, I used the MSBuild task and Target output flow @eerhardt suggested a while back.

dotnet/source-build#342

Adds an updater and dependency info that allow updating submodules from an orchestrated build manifest created by a successful ProdCon build.
Copy link
Copy Markdown
Member

@weshaggard weshaggard left a comment

Choose a reason for hiding this comment

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

Looks reasonable to me.

@dagood
Copy link
Copy Markdown
Member Author

dagood commented Mar 20, 2018

Wes had a comment on the source-build commit that I'm going to take a stab at before merging this PR: dagood/source-build@4791893#r28179476. (Including an updater for the checked-in commit info in the project files.)

Copy link
Copy Markdown
Member

@eerhardt eerhardt left a comment

Choose a reason for hiding this comment

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

:shipit:

@dleeapho
Copy link
Copy Markdown

/cc @markwilkie since this should eventually move to arcade.

@dagood
Copy link
Copy Markdown
Member Author

dagood commented Mar 20, 2018

Eric pinged me about that too offline, it's tracked by dotnet/arcade#32.

@dagood
Copy link
Copy Markdown
Member Author

dagood commented Mar 20, 2018

I split off a new issue for the project file hard-coded metadata updater (commit hash, depth): dotnet/source-build#370. The API I wanted isn't available so I think what I end up with will deserve its own PR/review.

Thanks, merging.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants