【deepin_V23_beta2】【一般】【有空】【发散测试】【DDE】【启动器】启动器中卸载软件，通知“卸载失败”，而实际成功卸载 #4412
Comments
SuperEffie
changed the title
black-desk
added a commit
to black-desk/dde-application-manager
that referenced
this issue
May 19, 2023
linuxdeepin/developer-center#4412 QProcess::execute will use waitid to get exit code of that process it starts. Register SIG_CHILD and wait all dead childs will break this mechanism, as QProcess will find out that there is no child to wait, and report this situation as a child process exit by crash. So I change the method of launch application to use a double fork, to start process and detach. Just as the way `QProcess` get this job done in its `startDetached` method. As a quick fix, I change the signature of `launch` method as no one use its return value currently. If this is need someday, we should use a pipe to let double forked child process report the return value and errno of its `exec` call. Signed-off-by: black-desk <me@black-desk.cn>
black-desk
added a commit
to black-desk/dde-application-manager
that referenced
this issue
May 19, 2023
linuxdeepin/developer-center#4412 QProcess::execute will use waitid to get exit code of that process it starts. Register SIG_CHILD and wait all dead childs will break this mechanism, as QProcess will find out that there is no child to wait, and report this situation as a child process exit by crash. So I change the method of launch application to use a double fork, to start process and detach. Just as the way `QProcess` get this job done in its `startDetached` method. As a quick fix, I change the signature of `launch` method as no one use its return value currently. If this is need someday, we should use a pipe to let double forked child process report the return value and errno of its `exec` call. Signed-off-by: black-desk <me@black-desk.cn>
black-desk
added a commit
to black-desk/dde-application-manager
that referenced
this issue
May 30, 2023
linuxdeepin/developer-center#4412 QProcess::execute will use waitid to get exit code of that process it starts. Register SIG_CHILD and wait all dead childs will break this mechanism, as QProcess will find out that there is no child to wait, and report this situation as a child process exit by crash. So I change the method of launch application to use a double fork, to start process and detach. Just as the way `QProcess` get this job done in its `startDetached` method. As a quick fix, I change the signature of `launch` method as no one use its return value currently. If this is need someday, we should use a pipe to let double forked child process report the return value and errno of its `exec` call. Signed-off-by: black-desk <me@black-desk.cn>
black-desk
added a commit
to linuxdeepin/dde-application-manager
that referenced
this issue
May 31, 2023
linuxdeepin/developer-center#4412 QProcess::execute will use waitid to get exit code of that process it starts. Register SIG_CHILD and wait all dead childs will break this mechanism, as QProcess will find out that there is no child to wait, and report this situation as a child process exit by crash. So I change the method of launch application to use a double fork, to start process and detach. Just as the way `QProcess` get this job done in its `startDetached` method. As a quick fix, I change the signature of `launch` method as no one use its return value currently. If this is need someday, we should use a pipe to let double forked child process report the return value and errno of its `exec` call. Signed-off-by: black-desk <me@black-desk.cn>
|
@SuperEffie 请在最新版本中核实是否已经修复合入，如没有合入请将修复优先级调整为1级，已经修复请关闭问题单。 |
|
【验证环境】 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
SUMMARY | 问题概要
No response
SOFTWARE/OS VERSIONS | 软件/系统版本信息
PRE-REQUIREMENT | 预置条件
No response
STEPS TO REPRODUCE | 复现步骤
OBSERVED RESULT | 观察到的结果
横幅中显示“卸载失败”，而实际已卸载成功
EXPECTED RESULT | 期望的结果
通知横幅中显示“卸载成功”
ADDITIONAL INFORMATION | 额外补充
No response
The text was updated successfully, but these errors were encountered: