Skip to content

Couple of fixes and tweaks to the R2R creation logic#3121

Merged
nguerrera merged 2 commits intodotnet:masterfrom
fadimounir:CrossgenRefList
Apr 17, 2019
Merged

Couple of fixes and tweaks to the R2R creation logic#3121
nguerrera merged 2 commits intodotnet:masterfrom
fadimounir:CrossgenRefList

Conversation

@fadimounir
Copy link
Copy Markdown

Changing crossgen commands to use response files.
Changing /Platform_Assemblies_Paths to a set of /r arguments
Using resolved files list as assembly references to crossgen (temp hack until issue 3110 is fixed)

Changing /Platform_Assemblies_Paths to a set of /r arguments
Using resolved files list as assembly references to crossgen (temp hack until issue 3110 is fixed)
@fadimounir
Copy link
Copy Markdown
Author

@nguerrera PTAL

1 similar comment
@fadimounir
Copy link
Copy Markdown
Author

@nguerrera PTAL

Comment thread src/Tasks/Microsoft.NET.Build.Tasks/RunReadyToRunCompiler.cs Outdated
Comment thread src/Tasks/Microsoft.NET.Build.Tasks/RunReadyToRunCompiler.cs Outdated
Comment thread src/Tasks/Microsoft.NET.Build.Tasks/RunReadyToRunCompiler.cs
@fadimounir
Copy link
Copy Markdown
Author

I know you're saying this is temporary, but why do we need to duplicate what the prepare step checks here? Why can't we handle building the right list in the prepare step?

I don't really have a preference on where to put this. I was thinking it would be easier to fix this later by deleting a well defined piece of code rather than some scattered bits. It's just a matter of style really

@nguerrera nguerrera added this to the 3.0.1xx milestone Apr 16, 2019
@nguerrera
Copy link
Copy Markdown
Contributor

I don't really have a preference on where to put this. I was thinking it would be easier to fix this later by deleting a well defined piece of code rather than some scattered bits. It's just a matter of style really

OK, that's fine since this is just temporary until #3110

@nguerrera nguerrera merged commit 84272da into dotnet:master Apr 17, 2019
@fadimounir fadimounir deleted the CrossgenRefList branch January 27, 2020 17:53
dsplaisted pushed a commit to dsplaisted/sdk that referenced this pull request Feb 19, 2020
…build 20191006.2 (dotnet#3121)

- Microsoft.NET.Sdk.Razor - 3.1.0-preview1.19506.2
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