[Process] Use output/error stream instead of naked handle

This is to make sure the handle is closed, whenever we close the streams

[test-runner] Make the --testsuite-name option required

Defaulting to a value doesn't make sense since the NUnit xml would be overwritten if multiple tests used the same default.

[runtime] Added --with-cross-offsets configure option for including an explicit AOT cross offsets file.

This option will define MONO_OFFSETS_FILE which will be used by the rest of the code to include the right file when looking for cross offsets.

[mcs] Declared fields cannot have unmanaged check done before all types are defined. Fixes #35269

[llvmonly] Fix EH when llvm is disabled.

[llvmonly] Fix an assertion for methods with many arguments.

[llvmonly] Fix the computation of the generic context used to initialize shared methods, previously we would be using the context of the class of the receiver, but that wouldn't work when its a non-generic class inheriting from a generic class. Use the context of the method which was used to lookup the shared method in the first place instead.

[sgen] Fix binary protocol limit default.

`size_t` is unsigned, so using `-1` by default will always impose a
limit, and make binary protocol file names numbered, which is not
usually what you want or expect.

[Process] Close process in/out/err handle on Process dispose

This would lead to leaked handle, and that would be visible when running
the test suite on Jenkins, with a "Too many open files" exception.

That also reverts the behavior to before 789808b507fe16941a31acc43ffc6bc8e0cdae75

[llvm] Fix aot support in non-llvmonly mode.

[llvmonly] Avoid disabling llvm for methods with pinvoke signatures in llvmonly mode.

[llvm] Register the llvmonly icalls even if llvm is not enabled.

Fix the non-llvm build.

[tests] Add timeout before failing

[tests] Increase timeout before failing

[mcs] Support auto-property with private setter with member definition from external assembly. Fixes #35236

[tests] Increase timeout before failing

[tests] Add --testsuite-name to avoid overwritting test result XML files

[tests] Use test-runner to run unhandled-exception tests

Add another missing file.

Add a missing file.

[llvm] Move the EH support code into mini-exceptions.c, and the c++ functions used by it into a new llvm-runtime.cpp file so llvmonly code works in a runtime which doesn't have llvm enabled.

[llvm] Fix support for asmonly+llvmonly by emitting the bitcode into the file given by the llvm-outfile aot option.

Merge pull request #2166 from ermshiperete/FontHeight

[WinForms] Check for null font Height

[profiler] Create a static version of libmono-profiler-log which doesn't link against eglib, to prevent duplicate symbols errors when its linked with the runtime. Fixes part of #29745.

[arm] Add a comment for a hack.

[arm64] Implement mono_threads_core_begin_async_resume () for arm64.

Merge pull request #2171 from lambdageek/dev/fix-marshal

[coop] One more pair of coop gc marshal fixes.

[sgen] Documentation about future-proofing the binary protocol.

We want to use the binary protocol to measure GC pause times in the
benchmarking suite.  To make that easier and be able to go back and
(re-)benchmark older versions, we'd like the binary protocol format to
be backwards compatible.

Add a new Moono_GetStackTraces method to Thread which can be used to obtain stack traces for all running threads.

[coop] One more pair of coop gc marshal fixes.

Missed a pair of
`mono_threads_reset_blocking_start`/`mono_threads_reset_blocking_end`
calls in 991e3ef3e47f0ccf9981a791f7da680b4ee9c4c6

Remove a few more 'using System.Reflection's.

[Mono.Security]: Use the new 'NoReflectionHelper' instead of reflection in 'MonoTlsProviderFactory'.

[sgen] Add some bridge documentation.

Merge pull request #2170 from alexanderkyte/mini_llvm_archfix

[runtime] Fix support for amd64 in mini-llvm-cpp.cpp

[runtime] Fix support for amd64 in mini-llvm-cpp.cpp

[System]: Add internal 'Mono.Net.Security.NoReflectionHelper' class to avoid reflection use.

Remove 'using System.Reflection'.

[Mono.Security]: Access System.dll's internals directly instead of using reflection.

[System]: Make internals visible to Mono.Security.

[test-runner] Fix a few more hardcoded test suite names

Missed those in 04bdf3b9332a4707d196b65deda1fdfba1b88a30

[test-runner] Wait synchronously for process exit

There was still a race for the last processes, where we could not
correctly wait for them to exit, before closing the StreamWriter for
stdout and stderr.

Merge pull request #2168 from BillSeurer/llvm

[ppc] Activate llvm for power

[ppc] Activate llvm for power

Merge pull request #2165 from lambdageek/dev/marshal-fix

[coop] update native call marshalling to pass thread state transition functions a stack ptr arg.

[socket] Use syscall abort mechanism properly

The crash only occurs on Posix platforms.

The assertion error in `mono_threads_begin_global_suspend`
"pending_suspends != 0" arised because of a unproper use of the syscall
abort mechanism.

The signal would have to be send out of a global suspend context, but
in case the call to `mono_threads_pthread_kill` would fail in
`mono_threads_core_abort_syscall`, then we would add a pending
operation, thus increasing the `pending_suspends` counter. But as we
wouldn't call for `mono_threads_wait_pending_operations`, we would never
have a chance to decrement this counter. So next time we would try to
call `mono_threads_begin_global_suspend` we would hit this assertion
error.

The fix is simply to call the abort from inside a global suspend
context, and making sure to wait on pending operations afterwards.

[socket] Enable Shutdown on socket disposal only on specific platforms

This breaks Linux, because a shutdown will cause a recv/recvfrom on the
remote host to return without an error.

This would trigger in System.Net.Sockets.SocketTest.ConnectedProperty.

[msvc] Manually update .defs, don't use update-defs Make target

Revert "[msvc] Update the DLL export definition files."

This reverts commit 90cb172eb8c160f46781d189d3baf25f06496b1d.

[msvc] Update the DLL export definition files.

[WinForms] Check for null font Height

DrawButtonText was throwing ArgumentException when
button.Font.Height was invalid.

[coop] update native call marshalling to pass thread state transition functions a stack ptr arg.

mono_threads_prepare_blocking, mono_threads_finish_blocking,
mono_threads_reset_blocking_start, mono_threads_reset_blocking_end all
take a stack pointer argument.  Update the native and managed wrappers
to pass it to them.

[runtime] Add a check for PE images to mono_method_body_get_object().

Fix the build.

[runtime] Move the MONO_ENV_OPTIONS parsing code to driver.c

Export mono_image_load_cli_header/load_metadata from image.c

Merge pull request #2162 from alexrp/master

[profiler] Disable the hack to attempt to get code start/size info.

[corlib] Add non-overlapping buffer MemoryCopy implementation

[profiler] Disable the hack to attempt to get code start/size info.

This is causing crashes by poking into runtime internals, and since we no
longer need this for anything critical, disable it.

https://bugzilla.xamarin.com/show_bug.cgi?id=35171

Merge pull request #2158 from akoeplinger/fix-winforms-tests

[WinForms] Update tests to match recent changes to Control Foreground+Background

Merge pull request #2154 from BrennanConroy/bug_30018

More robust get_address_info

[threads] Print more information in case of error in global suspend

Merge pull request #2149 from directhex/absolute-path-for-xcopy

[msvc] Use fully qualified path for xcopy

[WinForms] Update tests to match recent changes to Control Foreground+Background

The colors were updated in https://github.com/mono/mono/commit/f412b5529e560bca594e0c29a47f32dcfb252820,
but the tests weren't. This fixes them.

[Mono.Security]: Mark 'HttpsClientStream' as obsolete.

[System]: Remove some dead internal code.

Merge pull request #2122 from lambdageek/dev/bug-26384

In Type.FullName, escape ,+&@*[]\ with blackslash. Closes #26384

Merge pull request #2155 from akoeplinger/bug-34334-pr

[interpreter] Relax check for forcing compilation for built-in structs

[System]: Use MonoTlsProvider API in WebConnection.

[System]: Use MonoTlsProvider API in FtpWebRequest and SmtpClient.

[interpreter] Relax check for forcing compilation for built-in structs

Method calls on structs like DateTime wouldn't work before,
even though we know we don't have self mutating structs in mscorlib.

Fixes part 2 of https://bugzilla.xamarin.com/show_bug.cgi?id=34334

[reflection] Test for ModuleBuilder.GetType(String) with escaped names

[reflection] ModuleBuilder, TypeBuilder escaped type identifiers patch

1. Use the System.TypeSpec parser to parse components of the type name,
then call out to native code to apply the modifiers, as before.

2. TypeName and TypeIdentifier abstractions to hide display name / internal name strings

Update ModuleBuilder and TypeBuilder to use TypeIdentifier and TypeName
internally in all places that don't need to talk to native code.

3. Change ModuleBuilder.name_cache from Hashtable to Dictionary<TypeName, TypeBuilder>

[corlib] Test Type.FullName and Type.GetType() with special characters.

Make sure that type names that include +,&*[]\ are properly escaped and
unescaped when displaying and parsing.

[corlib] ves_icall_Type_GetNestedTypes unescape special chars in wanted nested type name.

[corlib] System.TypeSpec.Parse unclosed square brackets wrong exn.

Throw ArgumentException on "Foo[," instead of ArrayIndexOutOfBoundsException.

[corlib] System.TypeSpec.Parse escaped characters

[corlib] handle special characters in Type.FullName and Type.GetType(string)

Type.FullName escapes ,+&*[]\ with a leading backslash.
Type.GetType(typeName) strips escaping backslashes from typeName during parsing.

[tuner] Properly preserve AesCryptoServiceProvider.

[merged by rodo from monodroid - the mentioned test is not included here]

Fixes: https://xamarin.desk.com/agent/case/35534

Commit 5ca3cb4b wasn't propertly tested (doh!).

The problem with Aes and AesCryptoServiceProvider is that there's a
cross-assembly dependency: Aes is in mscorlib, while
AesCryptoServiceProvider is in System.Core. The
PreserveCrypto.ProcessSystemCore() method didn't properly handle this
setup.

Fix the linker, and add a test to Hello.

WARNING: The test sucks. In particular, to "test" this fix you need to
run the app and then grep the `adb logcat` output for:

# DownloadDataTaskAsync complete; status=RanToCompletion; exception=

If the status is anything else, or the exception is non-null, it's
busted.

FURTHERMORE, it's only useful in Release apps, not Debug apps.
(No linker in Debug apps.)

The reason for the need to grep is that there's no sane mechanism to
wait until the Task has completed before checking the status of the
Task, and throwing an exception from within the continuation wouldn't
do anything.

FIXME: The real problem is that Task.Wait() blocks forever and never
returns. This needs further investigation.

https://bugzilla.xamarin.com/show_bug.cgi?id=12681

[tuner] handle the case with multiple "preserve" attributes

 - when more than one "preserve" attribute is set (see IsPreservedAttribute),
   make sure we handle all of them and not just last one
 - fixes https://bugzilla.xamarin.com/show_bug.cgi?id=19391
 - example code which suffered from this issue:

    [Preserve(AllMembers=true)]
    [DataContract]
    public class TestClass
    {
      ...
    }

  later when a method from this class was queried using the reflection,
  it wasn't available as it was linked out

[runtime] Factor out the PE image loading code into a separate MonoImageLoader object.

[mcs] Fix inference of dynamic type used as a constructed type. Fixes #34898

[runtime] Fix a possible buffer overflow in get_process_module ().

[amd64] Remove some dead code.

Revert "[amd64] Allow returning of structs in register pairs."

This reverts commit c41b5a47ad24fc359a092fe9023fc82f68cdf198.

Revert this as it breaks IronJS tests:

https://bugzilla.xamarin.com/show_bug.cgi?id=35102

[mcs] Add additional doc reset after property initializers. Fixes #35087

[acceptance-tests] Fix tarball build and rename "check" target to "check-full"

The acceptance-tests folder wasn't getting included in the tarball, adding the folder
to the SUBDIRS in the top-level Makefile.am fixes this. Additionally, EXTRA_DISTFILES
doesn't work outside of mcs/ so rename it to EXTRA_DIST.

Since we don't want a top-level "make check" to recurse into acceptance-tests just yet,
rename the check target to check-full so nothing gets run by default.

Fixes mobile build

[mdoc] Update expected test results after private members change in mscorlib

[System.Runtime.Caching] Update test due to .net fixes

[system.componentmodel.dataannotations] Fix incorrect IsValid_Object_ValidationContext_CrossCallsWithNIEX test

[msvc] Use fully qualified path for xcopy

There are still nasty issues with $PATH vs %PATH% for builds spawned by a system service, this should hopefully fix the last one

[system.data] Remove empty ending character from resource constants

[gsharedvt] Inflate methods properly in one case. Fixes #34926.

Merge pull request #2148 from akoeplinger/remove-old-runtime-versions

[runtime] Remove explicit support for v4.0.30128 and v4.0.20506

[runtime] Remove explicit support for v4.0.30128 and v4.0.20506

These versions were still binding to the 4.0 mscorlib, which is a reference-only assembly now
and isn't intended for being used at runtime.

This issue manifested itself when someone used e.g. <supportedRuntime version="v4.0.20506"/> in
the app.config, causing the runtime to try loading that version and then bailing out with
"Corlib not in sync with this runtime: expected corlib version 117, found 111." (with 111 being
the corlib version at the time the reference assemblies were generated).

The fix is to remove the specific versions from the supported_runtimes array, causing programs
to automatically use the newest version instead.

Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=34675

Other reports:

https://youtrack.jetbrains.com/issue/TW-42700
https://youtrack.jetbrains.com/issue/TW-41337
http://stackoverflow.com/questions/29988698/teamcity-nunitlauncher-running-on-linux-mono-gives-corlib-not-in-sync-with-th

[llvm] Fix the handling of r4 arguments during calls. Fixes #35044.

Bump reference source.

[runtime] Remove unused MONO_IS_CROSS_COMPILE define.

[acceptance-tests] Convert Ruby script to Python

Most of the other scripts in the repo use Python and we decided to converge on that one.