Skip to content

Commit 4e5856a

Browse files
authored
fix: fallback to global mvn if mvnw not found (#193)
1 parent 359bf47 commit 4e5856a

File tree

1 file changed

+11
-16
lines changed

1 file changed

+11
-16
lines changed

src/providers/java_maven.js

Lines changed: 11 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -213,24 +213,19 @@ export default class Java_maven extends Base_java {
213213
try {
214214
this._invokeCommand(mvnw, ['--version'])
215215
} catch (error) {
216-
if (error.code === 'ENOENT') {
217-
useMvnw = false
218-
} else {
219-
throw new Error(`failed to check for mvnw`, {cause: error})
220-
}
216+
throw new Error(`failed to check for mvnw`, {cause: error})
221217
}
222-
mvn = useMvnw ? mvnw : mvn
218+
return mvnw
223219
}
224-
} else {
225-
// verify maven is accessible
226-
try {
227-
this._invokeCommand(mvn, ['--version'])
228-
} catch (error) {
229-
if (error.code === 'ENOENT') {
230-
throw new Error(`maven not accessible at "${mvn}"`)
231-
} else {
232-
throw new Error(`failed to check for maven`, {cause: error})
233-
}
220+
}
221+
// verify maven is accessible, if mvnw was not requested or not found
222+
try {
223+
this._invokeCommand(mvn, ['--version'])
224+
} catch (error) {
225+
if (error.code === 'ENOENT') {
226+
throw new Error((useMvnw ? 'mvnw not found and ' : '') + `maven not accessible at "${mvn}"`)
227+
} else {
228+
throw new Error(`failed to check for maven`, {cause: error})
234229
}
235230
}
236231
return mvn

0 commit comments

Comments
 (0)