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

Use project.json to generate DevDiv dependencies #13039

Merged
merged 6 commits into from
Aug 11, 2016

Conversation

tmat
Copy link
Member

@tmat tmat commented Aug 9, 2016

Use nuget restore to determine the exact dependencies we need to insert into VS.

Each VS component that uses CoreFX shall list their dependencies in

src/Setup/DevDivPackages/{component}/project.json

For now it's just Roslyn and Debugger.

@tmat tmat changed the title Avoid repackaging facades and ManagedEsent for insertion Avoid repackaging CoreFX libs and ManagedEsent for insertion Aug 9, 2016
@tmat tmat changed the title Avoid repackaging CoreFX libs and ManagedEsent for insertion Stop repackaging CoreFX libs and ManagedEsent for insertion Aug 9, 2016
@tmat tmat force-pushed the FacadesAndEsent branch 2 times, most recently from 4f46a44 to 1e60674 Compare August 11, 2016 02:25
@tmat tmat changed the title Stop repackaging CoreFX libs and ManagedEsent for insertion Use project.json to generate DevDiv dependencies Aug 11, 2016
@tmat tmat force-pushed the FacadesAndEsent branch 2 times, most recently from 9551242 to a43799c Compare August 11, 2016 02:31
@tmat
Copy link
Member Author

tmat commented Aug 11, 2016

@jasonmalinowski

@tmat tmat force-pushed the FacadesAndEsent branch 2 times, most recently from bb215bf to 977cb01 Compare August 11, 2016 17:21
@tmat
Copy link
Member Author

tmat commented Aug 11, 2016

@balajikris FYI - this affects the insertion process. Next insertion PR will need to be patched.

@tmat
Copy link
Member Author

tmat commented Aug 11, 2016

I have verified the output locally. Gonna merge to get insertion going.

@tmat tmat merged commit ae03495 into dotnet:master Aug 11, 2016
@tmat tmat deleted the FacadesAndEsent branch August 11, 2016 20:58
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.

2 participants