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

Flow TasksConfiguration in build & specify generator config #84931

Merged
merged 6 commits into from
Aug 1, 2023

Conversation

ViktorHofer
Copy link
Member

@ViktorHofer ViktorHofer commented Apr 17, 2023

Fixes #84157

Builds on top of the previous state of #84666

  1. Flow the TasksConfiguration in the root build (Subsets.props)
  2. Specify the LibrariesConfiguration explicitly when building the
    LibraryImport source generators.
  3. Set tasks configuration to debug by default.

@elinor-fung I worked on that mid last week and hadn't found time to push. I think this is a slightly better approach as it uses the common path (Debug/Release config for tasks) and makes sure that the TasksConfiguration and the LibrariesConfiguration is available in when triggered from a root build. This builds on top of your existing work which looked really good.

@ViktorHofer
Copy link
Member Author

@elinor-fung did you had a chance to look at this PR?

@elinor-fung
Copy link
Member

Sorry - I was oof. I'll take a look today or tomorrow.

src/tasks/tasks.proj Outdated Show resolved Hide resolved
@elinor-fung
Copy link
Member

@ViktorHofer did you have a chance to get back to this? Since it fixes binclashes that could affect shipping bits, we'd like to get it fixed for 8.

I can also take a look if you are heads down on other things.

@ViktorHofer
Copy link
Member Author

I'm out this week but can take a look next week. Sorry for the delay. I will make sure that this is ready before we branch off.

elinor-fung and others added 5 commits July 31, 2023 18:53
1. Flow the TasksConfiguration in the root build (Subsets.props)
2. Specify the LibrariesConfiguration explicitly when building the
   LibraryImport source generators.
3. Set tasks configuration to debug by default.
@ViktorHofer ViktorHofer force-pushed the SetTasksConfigurationCorrectly branch from 6d423e7 to 8733c9c Compare July 31, 2023 19:57
@ViktorHofer
Copy link
Member Author

@elinor-fung the PR is now ready. We still have two double writes but those are unrelated and I filed #89763 for it.

Copy link
Member

@elinor-fung elinor-fung left a comment

Choose a reason for hiding this comment

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

Thank you for doing this!

@ViktorHofer ViktorHofer merged commit f70a7dc into dotnet:main Aug 1, 2023
@ViktorHofer ViktorHofer deleted the SetTasksConfigurationCorrectly branch August 1, 2023 15:49
@ghost ghost locked as resolved and limited conversation to collaborators Aug 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BinClashes for Microsoft.Interop.SourceGeneration.dll
2 participants