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

Conversation

mmitche
Copy link
Member

@mmitche mmitche commented Feb 7, 2018

Fix two errors:
* Remove unneeded signed symbol catalog injection
* Build definition is not importing becuase of duplicated task name.

Fix two errors:
    * Remove unneeded signed symbol catalog injection
    * Build definition is not importing becuase of duplicated task name.
@mmitche mmitche requested review from mikem8361 and dagood February 7, 2018 16:42
@mmitche
Copy link
Member Author

mmitche commented Feb 7, 2018

@mikem8361 This is also for 2.1. The publishing changes can be applied there too, yes?

Copy link
Member

@dagood dagood left a comment

Choose a reason for hiding this comment

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

Yes, and in 2.1 we're in even better shape because final publish will pick up coreclr/corefx symbol packages and publish those, not just the core-setup outputs.

@mmitche mmitche merged commit c878a62 into dotnet:release/2.1 Feb 7, 2018
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.

3 participants