Skip to content

Conversation

@mmitche
Copy link
Member

@mmitche mmitche commented Jul 2, 2021

These need to be passed through to correct restore an internal runtime

To double check:

These need to be passed through to correct restore an internal runtime
@mmitche mmitche requested review from MichaelSimons and dseefeld July 2, 2021 18:42
@mmitche
Copy link
Member Author

mmitche commented Jul 2, 2021

@MichaelSimons @dseefeld Prep for internal .NET 6 builds. This gets past a couple issues in aspnetcore, but it also looks like that repo will need additional work as it calls its own .\build.sh in its SourceBuild.props and doesn't include these parameters.

@MichaelSimons
Copy link
Member

@JunTaoLuo, See #7587 (comment)

@mmitche mmitche merged commit 0563473 into dotnet:main Jul 6, 2021
missymessa added a commit that referenced this pull request Jul 7, 2021
@missymessa
Copy link
Member

Reverting this change, as it's causing Arcade's official build to fail: https://dnceng.visualstudio.com/internal/_build/results?buildId=1221868&view=results

@missymessa
Copy link
Member

#7598

missymessa added a commit that referenced this pull request Jul 7, 2021
@mmitche mmitche deleted the add-internal-runtime-to-sb branch July 22, 2021 16:42
mmitche added a commit to mmitche/arcade that referenced this pull request Jul 22, 2021
These need to be passed through to correct restore an internal runtime.

This is a follow-on to a previous attempt (dotnet#7587). The two changes from that are:
- Use the eng/common/build.sh variation of runtimesourcefeed/runtimesourcefeedkey (without -'s)
- Determine whether the MSRC variable is actually set before adding the args.
mmitche added a commit that referenced this pull request Jul 22, 2021
These need to be passed through to correct restore an internal runtime.

This is a follow-on to a previous attempt (#7587). The two changes from that are:
- Use the eng/common/build.sh variation of runtimesourcefeed/runtimesourcefeedkey (without -'s)
- Determine whether the MSRC variable is actually set before adding the args.
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.

4 participants