exec

Apply `Anonymous type can be replaced with lambda` inspection the whole project

Simplify launcher project structure

Fix TAPI handling of the client provided system input `InputStream` when running in embedded mode. Previously this input stream would be ignored.

Split :launcher into :launcher, :launcherBootstrap and :launcherStartup

in order to isolate Java 6 stuff

Signed-off-by: Paul Merlin <paul@gradle.com>

Change SystemProperties standard/important property methods

Instead of returning a set and having call sites call contains, just return a boolean.

Represent the result of `BuildActionExecuter.execute()` as an object that encapsulates either a result object or a failure, rather than as an opaque return value or a thrown `ReportedException`.

Replace deprectated `IOUtils.closeQuietly` uses with `IoActions.closeQuietly`

Fix handling of InterruptedExceptions

These exceptions were handled incorrectly throughout the whole

codebase, usually rethrown without restoring the interrupt status

or discarded entirely. This means that the system would not stop

executing even though the user wanted it to. In some cases this

also left the system in an inconsistent state, leading to deadlocks.

The most notable changes include:

- UncheckedException.rethrow automatically restores the interrupt status

- AsyncDispatch is guaranteed to deliver its messages, even when interrupted

- ExecHandle cancels the started process if it is interrupted while waiting

- ExecHandle disconnects from the process' output before killing it

- The worker API cancels the started work items if it is interrupted

- ManagedExecutors shut down immediately if they are interrupted while stopping

- We no longer log exceptions caused by interruption to the console

- Interrupting our caches no longer leaks file locks

Make environment variable mutation work on Java9+

Mutating the environment requires reflection on the

java.util package, so we need to open that package

up to Gradle. Since Gradle is not modularized, this

means opening it up to the whole classpath. This is

less than desirable, but the only way to restore the

behavior we had on Java 8 and below.

We should start limiting access to environment variables

going forward so users don't depend on arbitrary values.

However, this change allows us to unblock users who are

currently either not using Java 9 or running Gradle in

no-daemon mode, both of which are terrible solutions.

Switch to GradleSystem.getenv()

This commit replaces all System.getenv() with GradleSystem.getenv() in Gradle code base.

Add Gradle.getenv() api

Make OutputEventRenderer asynchronous

Don't emit a warning that the “daemon will cancel the build”.

This makes no sense to users.

Close System.in after build

Rename to EndOnClosedInputStream

Attempt to fix https://github.com/gradle/gradle-private/issues/975

This PR adds a guard for PipedInputStream to make sure it's safe to

read after being closed.

Address review feedback

- use DaemonServerConfiguration to pass information further down

- remove BuildAndStop command

- change naming to use singleUse

+review REVIEW-6567