Skip to content

Commit d9a26cc

Browse files
committed
fix(smol): use whichBinSync for ninja on all platforms
No need for platform conditional - whichBinSync works everywhere and is more explicit.
1 parent d317dd2 commit d9a26cc

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

packages/node-smol-builder/scripts/build.mjs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1508,8 +1508,8 @@ async function main() {
15081508
logger.log('::group::Compiling Node.js with Ninja (this will take a while...)')
15091509

15101510
try {
1511-
// Resolve full path to ninja on Windows since we use shell: false.
1512-
const ninjaCommand = WIN32 ? whichBinSync('ninja') : 'ninja'
1511+
// Resolve full path to ninja since we use shell: false.
1512+
const ninjaCommand = whichBinSync('ninja')
15131513
await exec(ninjaCommand, ['-C', 'out/Release', `-j${CPU_COUNT}`], {
15141514
cwd: NODE_DIR,
15151515
env: process.env,

0 commit comments

Comments
 (0)