Gradle

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merge Hasher and FileSnapshotter into FileHasher

We were using these only to get to the hash of files and stuff anyway. This simplifies the code somewhat, especially tests, because they can now use DefaultFileHasher instead of having to mock several methods.

+review REVIEW-6303

  1. … 18 more files in changeset.
Indicate that test running with isolated home dir may leak file handle

Add test case for ProjectBuilder to verify proper behavior

Add test fixture for testing serialization

+review REVIEW-6315

Fix test

+review REVIEW-6303

Add test for serializing exception in model field

- more specific test for GRADLE-3307

+review REVIEW-6315

Add special file collection snapshotter for Java classpaths

This snapshotter keeps the order of root files, but sorts elements of any root file tree. This way changes in only the order of the roots is considered a difference, while a change in only the order of non-root elements inside a single root does not considered a difference.

+review REVIEW-6303

Javadoc

+review REVIEW-6303

Test super-class instead

+review REVIEW-6303

Extract classpath snapshot normalization strategy

+review REVIEW-6303

Introduce FileCollectionSnapshotterRegistry

This will allow us to register special file collection snapshotters.

+review REVIEW-6303

  1. … 13 more files in changeset.
Merge branch 'release'

    • -9
    • +0
    /subprojects/docs/src/docs/release/notes.md
Update to build scan version 1.2

Use `buildFinished` hook for buildScanUserData

+review REVIEW-6320

Update to build scan version 1.2

Use `buildFinished` hook for buildScanUserData

    • -40
    • +26
    /gradle/buildScanUserData.gradle
Explicitly flush System.err on Groovy compiler error

System.err is an "auto-flushing" PrintWriter unless it is replaced.

In practice it's a LinePerThreadBufferingOutputStream here, so

definitely not "auto-flushing".

This caused test flakiness and potentially confusing user experience

because of mixed output.

+review REVIEW-6314

Remove unused code

The previous commit made this code unused.

+review REVIEW-6303

Simplify code: OutputFilesSnapshotter

This class doesn't need to be a bona fide FileCollectionSnapshotter - all it did anyway was to delegate those methods to a real FileCollectionSnapshotter. Untie this.

+review REVIEW-6303

Replace usage of Guava in TAPI integration test

- fix "NoClassDefFoundError: com/google/common/base/Throwables"

+review REVIEW-6309

Reorganize test code

+review REVIEW-6303

Revert "Don't assume exception is a RuntimeException"

This reverts commit 94fcd10f291c24bcc223f957c7de43068b0441a4.

+review REVIEW-6309

Add Gradle Script Kotlin v0.3.3 to the release notes

    • -0
    • +8
    /subprojects/docs/src/docs/release/notes.md
Remove outdated comment

Inline single-use interface method for FileCollectionSnapshotter

+review REVIEW-6303

Simplify FileCollectionSnapshotter interface

We don't need a method on the interface just to return a constant.

+review REVIEW-6303

Merge branch 'release'

Make tests that close the TAPI (via reset()) require an isolated daemon

+review REVIEW-6310

Ignore non-queryable devices when finding own IP addresses

This was the probably cause of a flaky test which failed with

```

Caused by: java.lang.RuntimeException: Could not determine the IP addresses for network interface vethdf3543b

at org.gradle.internal.remote.internal.inet.InetAddresses.analyzeNetworkInterface(InetAddresses.java:95)

at org.gradle.internal.remote.internal.inet.InetAddresses.analyzeNetworkInterfaces(InetAddresses.java:50)

at org.gradle.internal.remote.internal.inet.InetAddresses.<init>(InetAddresses.java:40)

at org.gradle.internal.remote.internal.inet.InetAddressFactory.init(InetAddressFactory.java:100)

at org.gradle.internal.remote.internal.inet.InetAddressFactory.getLocalBindingAddress(InetAddressFactory.java:85)

... 58 more

Caused by: java.net.SocketException: No such device

at org.gradle.internal.remote.internal.inet.InetAddresses.analyzeNetworkInterface(InetAddresses.java:57)

... 62 more

```

The test was `UserGuideSamplesIntegrationTest#scalaCrossCompilation`

+review REVIEW-6311

Merge branch 'release'

    • -0
    • +9
    /subprojects/docs/src/docs/release/notes.md
Make tests that close the TAPI require an isolated daemon

`DefaultGradleConnector.close()` sends stop signals to associated

daemons. Thus it might stop a daemon still used by another running test.

+review REVIEW-6310

Use default classloader for serializing exception placeholders

+review REVIEW-6309