LukeParke/vercel
1
0
Fork 0
mirror of https://github.com/LukeHagar/vercel.git synced 2025-12-10 04:22:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

[cli] Don't use tree-kill for shutting down startDevServer() process in vc dev (#7830)

Browse Source 
Follow-up to #7824 since using `tree-kill` here seems to make a CLI test hang forever.
This commit is contained in:
Nathan Rajlich
2022-05-19 18:16:09 -07:00
committed by GitHub
parent e829ce47c3
commit 0d1dc23d0d
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
packages/cli/src/util/dev/server.ts
View File

@@ -1040,7 +1040,12 @@ export default class DevServer {
const { debug } = this.output; const { debug } = this.output;
debug(`Killing builder dev server with PID ${pid}`); debug(`Killing builder dev server with PID ${pid}`);
this.devServerPids.delete(pid); this.devServerPids.delete(pid);
await treeKill(pid); try {
process.kill(pid, 'SIGTERM');
debug(`Killed builder dev server with PID ${pid}`);
} catch (err) {
debug(`Failed to kill builder dev server with PID ${pid}: ${err}`);
}
} }
async send404( async send404(