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

Merge release/3.0.1xx to master #3113

Merged
merged 44 commits into from
Apr 16, 2019
Merged

Conversation

dotnet-bot
Copy link
Contributor

This is an automatically generated pull request from release/3.0.1xx into master.

git fetch --all
git checkout merges/release/3.0.1xx-to-master
git reset --hard upstream/master
git merge upstream/release/3.0.1xx
# Fix merge conflicts
git commit
git push upstream merges/release/3.0.1xx-to-master --force

Once all conflicts are resolved and all the tests pass, you are free to merge the pull request.

Also switch from creating lists of filtered libraries, to setting properties on the libraries to indicate if they are filtered out for runtime and compilation
…isn't copy local

This metadata is consumed by the new deps file generation code
…file

Works around what appears to be a bug in how NuGet writes the assets file for a few packages
This keeps parity between the "old" and "new" methods of generating the deps file
This was logic in ProjectContext.GetRuntimeLibraries that I missed when re-implementing it
@nguerrera
Copy link
Contributor

nguerrera commented Apr 15, 2019

@dsplaisted can you push this through? IL Linker needs your change in master. cc @sbomer

@dsplaisted dsplaisted force-pushed the merges/release/3.0.1xx-to-master branch from fa9e036 to 643d19e Compare April 16, 2019 03:40
@dsplaisted dsplaisted merged commit 8cb31b0 into master Apr 16, 2019
@sbomer
Copy link
Member

sbomer commented Apr 16, 2019

Thank you @dsplaisted!

dsplaisted pushed a commit to dsplaisted/sdk that referenced this pull request Feb 19, 2020
….1 (dotnet#3113)

- Microsoft.DotNet.Cli.Runtime - 3.1.100-preview1.19506.1
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