On 10/2/20 2:28 PM, Michal Prívozník wrote:
On 10/2/20 6:42 PM, Daniel Henrique Barboza wrote:
On 10/2/20 1:07 PM, Michal Prívozník wrote:
On 10/1/20 4:35 PM, Daniel Henrique Barboza wrote:
When VIR_EXEC_DAEMON is true and cmd->pidfile exists, the parent
[...]
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1882093
Signed-off-by: Daniel Henrique Barboza <[email protected]>
---
src/util/vircommand.c | 19 ++++++++++---------
1 file changed, 10 insertions(+), 9 deletions(-)
Reviewed-by: Michal Privoznik <[email protected]>
And congratz onto gaining commit rights :-)
Thanks!
By the way, we have a 'do not push stuff on Friday' rule to avoid breaking
things during weekends, right? Guess I'll wait to push this one next Monday.
I'm not aware of any such rule. In fact, I've pushed couple of patches today
already. I say, go ahead and push them! It doesn't break the build and if there
is a subtle bug with a commit, I'm sure it can wait over the weekend anyway
(I'm not picking on your commit in particular, just any commit - the sooner we
test it the better).
Roger that. The deed is done :)
Thanks,
DHB
Michal
