Set HttpActivityPropagationSupport=true for non-Release builds

[jfversluis]

Fixes #9932

[dellis1972]

I just wish there was a better way to detect release builds. So many users do not use the standard Debug / Release configurations :(

[jonpryor]

@dellis1972 remarked:

I just wish there was a better way to detect release builds. So many users do not use the standard Debug / Release configurations :(

which is something I also mentioned in passing elsewhere.

Should we be using '$(Configuration)' == 'Release'? Or should we do what we've done elsewhere, '$(DebugSymbols)' == 'True'? Or '$(Optimize)' == 'True'?

…or just tie it to fast deployment (lol?)?

I think I largely agree with the sentiment that we should continue to avoid '$(Configuration)' == 'Release' whenever possible. The problem is largely deciding on what should be used instead.

[jonpryor]

To explicitly add: my current preference is to use '$(Optimize)' == 'True' for "Release" configuration builds. I'd like @dellis1972 to chime in.

[jonpryor]

Draft commit message:

[Xamarin.Android.Build.Tasks] $(HttpActivityPropagationSupport)=false in "Release" (#9928)

Context: d22f04ea8edbcfa90f16257e60295667b0887414
Context: https://github.com/dotnet/aspire/issues/4684
Context: https://github.com/dotnet/maui/pull/24365
Context: https://github.com/jfversluis/MauiAspire

Commit d22f04ea set the default value of the
[`$(HttpActivityPropagationSupport)' MSBuild property][0] to false,
which "removes code related to diagnostics support for
System.Net.Http."  This was done to reduce app size.

.NET Aspire also uses System.Net.Http, and we'd like to System.Net.Http
diagnostics support to be available by default in that environment;
see also dotnet/aspire#4684 and dotnet/maui#24365.

Update the `$(HttpActivityPropagationSupport)` MSBuild property to be
false by default only in "Release" configuration builds.

[0]: https://learn.microsoft.com/en-us/dotnet/core/deploying/trimming/trimming-options

[dellis1972]

To explicitly add: my current preference is to use '$(Optimize)' == 'True' for "Release" configuration builds. I'd like @dellis1972 to chime in.

using Optimize might be a better way to go here.

[rolfbjarne]

I just wish there was a better way to detect release builds. So many users do not use the standard Debug / Release configurations :(

There's this: dotnet/sdk#31918... it's been almost two years since it was filed.

IMHO at this point we (the android/macios/maui teams) could just decide to do something, and hope the SDK follows suit at some point.

[jfversluis]

@dellis1972 @jonpryor just for my understanding, what does Optimize mean?

[dellis1972]

@dellis1972 @jonpryor just for my understanding, what does Optimize mean?

Is a standard MSBuild property for the C# compiler which optimizes the IL.

A boolean value that when set to true, enables compiler optimizations. This property is equivalent to the /optimize compiler switch.

See https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-options/code-generation#optimize