ivabus/gui
1
0
Fork 0
mirror of https://github.com/ivabus/gui synced 2025-04-23 05:57:11 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

handle stderr from cli better (#630)

Browse source 
Co-authored-by: Neil <get.neilmolina@gmail.com>
This commit is contained in:
ABevier 2023-05-29 21:23:10 -04:00 committed by GitHub
parent c5d9b3bc46
commit 0d9ac5e30e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 18 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
modules/desktop/electron/libs/cli.ts
View file

@ -44,13 +44,24 @@ export async function installPackage(
child.stderr.on("data", (data) => {
try {
const msg = JSON.parse(data.toString().trim());
progressNotifier(msg);
} catch (err) {
//swallow it
}
data
.toString()
.split("\n")
.map((s: string) => s.trim())
.filter((s: string) => s.length > 0)
.forEach((line) => {
try {
const msg = JSON.parse(line.trim());
progressNotifier(msg);
} catch (err) {
log.error("handling cli notification line", line, err);
}
});
stderr += data.toString().trim();
stderr += data.toString();
} catch (err) {
log.error("error processing cli data", data.toString(), err);
}
});
child.on("exit", (code) => {
@ -83,7 +94,7 @@ function newInstallProgressNotifier(full_name: string, notifyMainWindow: MainWin
let currentPackageNumber = 0;
return function (msg: any) {
if (msg.status !== "downloading") {
if (msg.status !== "downloading" && msg.status !== "installing") {
log.info("cli:", msg);
}