Skip to content

Commit e312d98

Browse files
EdgarTwiggEdgarTwigg
authored andcommitted
The workaround worked on clean builds, but tried to copy-over-existing on not-clean-builds. Fixed now.
1 parent 006c4d6 commit e312d98

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
# Webtools releases
22

33
## [Unreleased]
4+
### Fixed
5+
- The workaround worked on clean builds, but tried to copy-over-existing on not-clean-builds. Fixed now.
46

57
## [1.2.1] - 2025-08-07
68
### Fixed

src/main/java/com/diffplug/webtools/node/SetupCleanupNode.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
import java.io.IOException;
2525
import java.io.Serializable;
2626
import java.nio.file.Files;
27+
import java.nio.file.StandardCopyOption;
2728
import java.util.Collections;
2829

2930
class SetupCleanupNode implements Serializable {
@@ -61,7 +62,8 @@ protected void doStart(SetupCleanupNode key) throws TaskRunnerException, Install
6162
// copy npm.cmd as a windows workaround
6263
try {
6364
Files.copy(key.installDir.toPath().resolve("node/node_modules/npm/bin/npm.cmd"),
64-
key.installDir.toPath().resolve("node/npm.cmd"));
65+
key.installDir.toPath().resolve("node/npm.cmd"),
66+
StandardCopyOption.REPLACE_EXISTING);
6567
} catch (IOException e) {
6668
throw new RuntimeException(e);
6769
}

0 commit comments

Comments
 (0)