Skip to content
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

Build release version of Microsoft.DiaSymReader[.PortablePdb] #11955

Merged
merged 1 commit into from
Jun 13, 2016

Conversation

cston
Copy link
Member

@cston cston commented Jun 13, 2016

No description provided.

@cston
Copy link
Member Author

cston commented Jun 13, 2016

Changes missing from #11945.

@cston
Copy link
Member Author

cston commented Jun 13, 2016

@tmat please review.
@MattGertz for approval.

@tmat
Copy link
Member

tmat commented Jun 13, 2016

👍

@@ -1,7 +1,7 @@
<Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
<PropertyGroup>
<AssemblyVersion>1.0.8</AssemblyVersion>
<IsReleaseVersion>false</IsReleaseVersion>
<IsReleaseVersion>true</IsReleaseVersion>
Copy link
Member

@tmat tmat Jun 13, 2016

Choose a reason for hiding this comment

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

This is dead code, but no harm done. We don't build DSR from here anymore

@cston
Copy link
Member Author

cston commented Jun 13, 2016

@tmat Additional fixes.

@cston cston merged commit 169f8e8 into dotnet:stabilization Jun 13, 2016
@cston cston deleted the u3-rtm-2 branch June 13, 2016 06:50
@@ -207,7 +207,7 @@ void GeneratePublishingConfig()
{
// nuget:
var packages = MakeRoslynPackageElements(isRelease: true).Concat(MakePackageElementsForPublishedDependencies(isRelease: true));
GeneratePublishingConfig("nuget_org-packages.config", packages);
GeneratePublishingConfig("myget_org-packages.config", packages);
Copy link
Member

Choose a reason for hiding this comment

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

This is odd. Why not publish to nuget?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants