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

Minor source build fixes #67028

Merged
merged 1 commit into from
Feb 24, 2023
Merged

Minor source build fixes #67028

merged 1 commit into from
Feb 24, 2023

Conversation

jaredpar
Copy link
Member

Couple of minor fixes to source build:

  1. Force the build engine to be dotnet when doing a local source build. Using msbuild is not supported by arcade for source build and will lead to hard to track down errors. Force to dotnet to make it just work locally.
  2. Exclude our .NET Framework ARM64 binaries from source build. No reason for them to be there.

related to #66930

Couple of minor fixes to source build:

1. Force the build engine to be `dotnet` when doing a local source
   build. Using msbuild is not supported by arcade for source build and
   will lead to hard to track down errors. Force to `dotnet` to make it
   just work locally.
2. Exclude our .NET Framework ARM64 binaries from source build. No
   reason for them to be there.
@jaredpar
Copy link
Member Author

@333fred, @dotnet/roslyn-compiler PTAL

@jaredpar jaredpar merged commit 527c792 into dotnet:main Feb 24, 2023
@jaredpar jaredpar deleted the sb branch February 24, 2023 19:21
@ghost ghost added this to the Next milestone Feb 24, 2023
@RikkiGibson RikkiGibson modified the milestones: Next, 17.6 P2 Mar 2, 2023
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