Skip to content

Publish the SDK sentinel package as a package#8012

Merged
mmitche merged 1 commit into
dotnet:masterfrom
mmitche:push-internal-packages
Jul 16, 2020
Merged

Publish the SDK sentinel package as a package#8012
mmitche merged 1 commit into
dotnet:masterfrom
mmitche:push-internal-packages

Conversation

@mmitche

@mmitche mmitche commented Jul 16, 2020

Copy link
Copy Markdown
Member

RIght now it's only being pushed as a blob and so the performance repo cannot reference it in their version.details.xml file.

@mmitche

mmitche commented Jul 16, 2020

Copy link
Copy Markdown
Member Author

Comment thread eng/Publishing.props
@mmitche mmitche merged commit c68bf8e into dotnet:master Jul 16, 2020
mmitche added a commit to mmitche/installer that referenced this pull request Aug 3, 2020
mmitche added a commit to mmitche/installer that referenced this pull request Aug 3, 2020
mmitche added a commit to mmitche/installer that referenced this pull request Aug 5, 2020
Anipik pushed a commit that referenced this pull request Aug 5, 2020
…rom master (#8105)

* Revert "Only publish Microsoft.DotNet.Sdk.Internal once (#8016)"

This reverts commit 7e76b75.

* Revert "Publish the SDK sentinel package as a package (#8012)"

This reverts commit a7cf77d.

* Fix asset manifest file names.
Set the asset manifest OS and change the name of the installers manifests.
Currently arcade defaults to using OS-Platform as the asset manifest file name.
Installer does a ton of building in docker containers, which ends up generating
many overlapping names. This is a symptom and also likely hiding some non-determinsm.

To fix, change the name of the blob asset manifest name to include a "-installers" suffix.
Then alter the property names around a bit so that AssetManifestOS is set
prior to publish.proj's publish target being run.
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.

2 participants