Skip to content
This repository has been archived by the owner on Jul 18, 2023. It is now read-only.

Commit

Permalink
fix: use kill-process-by-name instead of fkill
Browse files Browse the repository at this point in the history
Resolves Error with builder version v3.9.1  #700
  • Loading branch information
shrihankp committed May 13, 2023
1 parent 8b519bc commit 375c863
Show file tree
Hide file tree
Showing 3 changed files with 36 additions and 494 deletions.
7 changes: 2 additions & 5 deletions index.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ const { WebSocketServer } = require('ws');
const open_ = require('open');
const pf = require('portfinder');

const fkill = require('fkill');
const killProcess = require('kill-process-by-name');

const {
updateFiles,
Expand Down Expand Up @@ -100,10 +100,7 @@ const cleanExit = async (svr) => {
log('Killing any dangling processes...', false);

try {
await fkill(['adb', 'java', 'aapt2'], {
forceAfterTimeout: 5000,
silent: true
});
['adb', 'java', 'aapt2'].forEach((p) => killProcess(p));
log('Done.', true, false);
} catch (error) {
log('Failed.', true, false);
Expand Down
Loading

0 comments on commit 375c863

Please sign in to comment.