Skip to content

Store SHA/version of synced repos in VMR using .props files#2014

Merged
premun merged 11 commits into
dotnet:mainfrom
premun:prvysoky/vmr-info
Sep 12, 2022
Merged

Store SHA/version of synced repos in VMR using .props files#2014
premun merged 11 commits into
dotnet:mainfrom
premun:prvysoky/vmr-info

Conversation

@premun
Copy link
Copy Markdown
Member

@premun premun commented Sep 9, 2022

Tag files are no longer used and commit SHA / version for each synchronized individual repository is stored in/read from:

  • git-info/AllRepoVersions.props file
  • git-info/[repo].props

VmrManager now also reads the package versions from Version.Details.xml and stores it too.

These files are the same as Source-Build tarball uses (link). The logic is the same, only uses Version.Details.xml reader from DarcLib. The tarball tasks could utilize this and code could be removed from dotnet/installer if we want.

Resolves dotnet/arcade#10264

tkapin
tkapin previously approved these changes Sep 9, 2022
Comment thread src/Microsoft.DotNet.Darc/src/DarcLib/Models/MsBuildPropsFile.cs Outdated
@premun premun requested a review from tkapin September 12, 2022 09:40
tkapin
tkapin previously approved these changes Sep 12, 2022
@premun premun changed the title Store commit SHA and version of synced repositories in VMR using .props files Store SHA/version of synced repos in VMR using .props files Sep 12, 2022
@premun premun merged commit 1031000 into dotnet:main Sep 12, 2022
@premun premun deleted the prvysoky/vmr-info branch September 12, 2022 13:04
premun added a commit to dotnet/installer that referenced this pull request Sep 14, 2022
Previously, the recursive clone of all the product repos based on `Version.Details.xml` files was done inside MSBuild but the VMR tooling can now do that (dotnet/arcade-services#2004).
This replaces the need for the Source-Build tasks (Read/Write, recursive clone).

Secondly, `git-info` props files are now created by the CLI too (dotnet/arcade-services#2014).
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.

Create git-info files

2 participants