-
Notifications
You must be signed in to change notification settings - Fork 525
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
Convert /tools
and /build-tools
projects from net472
to $(DotNetStableTargetFramework)
#7943
Conversation
…etStableTargetFramework)`
0ae1cb0
to
4d21fb7
Compare
<_ConfigurationFile>..\..\bin\Build$(Configuration)\merge-configuration.xml</_ConfigurationFile> | ||
<_ConfigurationInputBaseDirectory>..\..\bin\Build$(Configuration)\api\</_ConfigurationInputBaseDirectory> | ||
<_ConfigurationOutputBaseDirectory>..\..\bin\Build$(Configuration)\api\</_ConfigurationOutputBaseDirectory> | ||
</PropertyGroup> | ||
|
||
<Exec | ||
Command="$(ManagedRuntime) $(ManagedRuntimeArgs) $(ApiMerge) -config=$(_ConfigurationFile) -config-input-dir=$(_ConfigurationInputBaseDirectory) -config-output-dir=$(_ConfigurationOutputBaseDirectory)" /> | ||
Command="dotnet $(ApiMerge) -config=$(_ConfigurationFile) -config-input-dir=$(_ConfigurationInputBaseDirectory) -config-output-dir=$(_ConfigurationOutputBaseDirectory)" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this use the dotnet
we provision? "$(DotNetPreviewTool)"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both api-merge
and jnienv-gen
are built with $(DotNetStableTargetFramework)
instead of $(DotNetTargetFramework)
so I think dotnet
is correct? Although $(DotNetPreviewTool)
would work too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TargetFramework=net7.0
wouldn't make it to be built with the system dotnet. Do we want to run it with the same dotnet it was built with?
Outputs="../../src/Mono.Android/Android.Runtime/JNIEnv.g.cs"> | ||
<Exec | ||
Command="$(ManagedRuntime) $(ManagedRuntimeArgs) "../../bin/Build$(Configuration)/jnienv-gen.exe" -o ../../src/Mono.Android/Android.Runtime/JNIEnv.g.cs --use-java-interop" | ||
Command="dotnet "../../bin/Build$(Configuration)/jnienv-gen.dll" -o ../../src/Mono.Android/Android.Runtime/JNIEnv.g.cs --use-java-interop" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here.
* main: Convert `/tools` and `/build-tools` projects from `net472` to `$(DotNetStableTargetFramework)` (dotnet#7943)
* main: Bump to xamarin/Java.Interop/main@554d819 (dotnet#7951) [Microsoft.Android.Sdk.ILLink] fix crash when TZ changes (dotnet#7956) [tests] Port 'Xamarin.Android.JcwGen-Tests.JcwGen-Tests' to .NET (dotnet#7949) [Xamarin.Android.Build.Tasks] remove `pdb2mdb` (dotnet#7950) [ci] Add some extra params to configure the test templates (dotnet#7955) Convert `/tools` and `/build-tools` projects from `net472` to `$(DotNetStableTargetFramework)` (dotnet#7943) [Xamarin.Android.Build.Tasks] fix cases of missing `@(Reference)` (dotnet#7947) Bump com.android.tools:r8 from 4.0.52 to 8.0.40 (dotnet#7934) Bump to xamarin/Java.Interop/main@a172402 (dotnet#7944) [Xamarin.Android] Remove OpenTK, sqlite-xamarin, System.EnterpriseServices. (dotnet#7940) [ci] Stop building classic test suites. (dotnet#7938) Bumping to the correct monodroid commit Trying to bump monodroid to run debugger-tests Pass timeout to runtime
Many of our projects in
/tools
and/build-tools
targetnet472
. Update them to instead target$(DotNetStableTargetFramework)
.A side-effect of this is that there is no longer an
.exe
built, so we also need to update everything in our build system that calls these tools to usedotnet foo.dll
instead ofmono foo.exe
.