Compare commits

...

5 Commits

Author SHA1 Message Date
Daz DeBoer
cd579d970f Build outputs 2022-08-18 17:19:11 -06:00
Daz DeBoer
f0ec7fb780 Remove 'followSymbolicLinks: false' flag for Globber
It appears that when restoring the Gradle User Home directory, certain empty
directories are being identified by Node as broken symlinks. This results in
`Error: ENOENT: No such file or directory` when attempting to resolve a Glob pattern.

By using the default behaviour of Globber (followSymbolicLinks && ignoreBrokenLinks),
these bad files no longer cause errors when saving the Gradle User Home to cache.

Fixes #408
2022-08-18 17:10:03 -06:00
Daz DeBoer
bf2a15ee94 Fix cache-read-only for scheduled jobs
The fix for #383 introduced a bug that caused the post-action to fail when
attemping to resolve `cache-read-only` for scheduled jobs.
This has been fixed with an explicit (rather than implicit) check for 'event.repository != null'.

Fixes #409
2022-08-18 12:35:47 -06:00
Daz DeBoer
c85b5e1be7 Run full-check when workflow-samples change 2022-08-17 14:53:21 -06:00
Daz DeBoer
330980897f Update to republished CCUD plugin version 2022-08-17 14:03:35 -06:00
9 changed files with 12 additions and 18 deletions

View File

@@ -1,6 +1,6 @@
plugins { plugins {
id "com.gradle.enterprise" version "3.11.1" id "com.gradle.enterprise" version "3.11.1"
id "com.gradle.common-custom-user-data-gradle-plugin" version "1.8.0" id "com.gradle.common-custom-user-data-gradle-plugin" version "1.8"
} }
gradleEnterprise { gradleEnterprise {

View File

@@ -1,6 +1,6 @@
plugins { plugins {
id("com.gradle.enterprise") version "3.11.1" id("com.gradle.enterprise") version "3.11.1"
id("com.gradle.common-custom-user-data-gradle-plugin") version "1.8.0" id("com.gradle.common-custom-user-data-gradle-plugin") version "1.8"
} }
gradleEnterprise { gradleEnterprise {

View File

@@ -9,7 +9,7 @@ on:
branches: branches:
- main - main
paths: paths:
- '.github/workflows/**' - '.github/**'
- 'dist/**' - 'dist/**'
jobs: jobs:

View File

@@ -18,7 +18,7 @@ inputs:
When 'true', existing entries will be read from the cache but no entries will be written. When 'true', existing entries will be read from the cache but no entries will be written.
By default this value is 'false' for workflows on the GitHub default branch and 'true' for workflows on other branches. By default this value is 'false' for workflows on the GitHub default branch and 'true' for workflows on other branches.
required: false required: false
default: ${{ github.event.repository && github.ref_name != github.event.repository.default_branch }} default: ${{ github.event.repository != null && github.ref_name != github.event.repository.default_branch }}
cache-write-only: cache-write-only:
description: | description: |

6
dist/main/index.js vendored
View File

@@ -65941,8 +65941,7 @@ class AbstractEntryExtractor {
const artifactType = cacheEntryDefinition.artifactType; const artifactType = cacheEntryDefinition.artifactType;
const pattern = cacheEntryDefinition.pattern; const pattern = cacheEntryDefinition.pattern;
const globber = yield glob.create(pattern, { const globber = yield glob.create(pattern, {
implicitDescendants: false, implicitDescendants: false
followSymbolicLinks: false
}); });
const matchingFiles = yield globber.glob(); const matchingFiles = yield globber.glob();
if (matchingFiles.length === 0) { if (matchingFiles.length === 0) {
@@ -66045,8 +66044,7 @@ class GradleHomeEntryExtractor extends AbstractEntryExtractor {
return __awaiter(this, void 0, void 0, function* () { return __awaiter(this, void 0, void 0, function* () {
const wrapperZips = path_1.default.resolve(this.gradleUserHome, 'wrapper/dists/*/*/*.zip'); const wrapperZips = path_1.default.resolve(this.gradleUserHome, 'wrapper/dists/*/*/*.zip');
const globber = yield glob.create(wrapperZips, { const globber = yield glob.create(wrapperZips, {
implicitDescendants: false, implicitDescendants: false
followSymbolicLinks: false
}); });
for (const wrapperZip of yield globber.glob()) { for (const wrapperZip of yield globber.glob()) {
(0, cache_utils_1.cacheDebug)(`Deleting wrapper zip: ${wrapperZip}`); (0, cache_utils_1.cacheDebug)(`Deleting wrapper zip: ${wrapperZip}`);

File diff suppressed because one or more lines are too long

6
dist/post/index.js vendored
View File

@@ -64992,8 +64992,7 @@ class AbstractEntryExtractor {
const artifactType = cacheEntryDefinition.artifactType; const artifactType = cacheEntryDefinition.artifactType;
const pattern = cacheEntryDefinition.pattern; const pattern = cacheEntryDefinition.pattern;
const globber = yield glob.create(pattern, { const globber = yield glob.create(pattern, {
implicitDescendants: false, implicitDescendants: false
followSymbolicLinks: false
}); });
const matchingFiles = yield globber.glob(); const matchingFiles = yield globber.glob();
if (matchingFiles.length === 0) { if (matchingFiles.length === 0) {
@@ -65096,8 +65095,7 @@ class GradleHomeEntryExtractor extends AbstractEntryExtractor {
return __awaiter(this, void 0, void 0, function* () { return __awaiter(this, void 0, void 0, function* () {
const wrapperZips = path_1.default.resolve(this.gradleUserHome, 'wrapper/dists/*/*/*.zip'); const wrapperZips = path_1.default.resolve(this.gradleUserHome, 'wrapper/dists/*/*/*.zip');
const globber = yield glob.create(wrapperZips, { const globber = yield glob.create(wrapperZips, {
implicitDescendants: false, implicitDescendants: false
followSymbolicLinks: false
}); });
for (const wrapperZip of yield globber.glob()) { for (const wrapperZip of yield globber.glob()) {
(0, cache_utils_1.cacheDebug)(`Deleting wrapper zip: ${wrapperZip}`); (0, cache_utils_1.cacheDebug)(`Deleting wrapper zip: ${wrapperZip}`);

File diff suppressed because one or more lines are too long

View File

@@ -150,8 +150,7 @@ abstract class AbstractEntryExtractor {
// Find all matching files for this cache entry definition // Find all matching files for this cache entry definition
const globber = await glob.create(pattern, { const globber = await glob.create(pattern, {
implicitDescendants: false, implicitDescendants: false
followSymbolicLinks: false
}) })
const matchingFiles = await globber.glob() const matchingFiles = await globber.glob()
@@ -308,8 +307,7 @@ export class GradleHomeEntryExtractor extends AbstractEntryExtractor {
private async deleteWrapperZips(): Promise<void> { private async deleteWrapperZips(): Promise<void> {
const wrapperZips = path.resolve(this.gradleUserHome, 'wrapper/dists/*/*/*.zip') const wrapperZips = path.resolve(this.gradleUserHome, 'wrapper/dists/*/*/*.zip')
const globber = await glob.create(wrapperZips, { const globber = await glob.create(wrapperZips, {
implicitDescendants: false, implicitDescendants: false
followSymbolicLinks: false
}) })
for (const wrapperZip of await globber.glob()) { for (const wrapperZip of await globber.glob()) {