Commit ea6b459
committed
Expose the real mysqld exit code to process-compose, so it can report any errors. As a side-effect, removes $MYSQL_PID. Fixes #2518.
The old code used shell '&' to background mysqld, then captured its pid
with MYSQL_PID=$!, in doing so obscuring the mysqld exit code.
I can't see any sane way of both propagating the exit code *and* setting
MYSQL_PID. I also can't see any point in MYSQL_PID, given that
process-compose is a TUI and the pid is readily available in
$MYSQL_PID_FILE. So to fix the bug, I've made mysqld no longer
backgrounded (is_daemon becomes 'false').1 parent dc50eb5 commit ea6b459
1 file changed
+2
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
6 | | - | |
| 5 | + | |
| 6 | + | |
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| |||
0 commit comments