Re: [Sugar-devel] metacity is not reaped, bug #4958

2016-04-20 Thread Manash Raja
Hi James, Thanks, :) I updated the logic flow and also included a Popen.communicate() call to remove the waiting defunct process. Please check. https://github.com/sugarlabs/sugar/pull/682 I was able to reproduce the error and after the patch there was only one process for metacity with Sl+

Re: [Sugar-devel] metacity is not reaped, bug #4958

2016-04-20 Thread James Cameron
Thanks, good start. I've updated the bug with a reproducer, and reproduced twice, once on Ubuntu 16.04 and once on Fedora 23 SoaS. I've tried your patch; small logic error causes two instances of metacity to start, and the second one (correctly) exits. The process is still not reaped. Your

[Sugar-devel] metacity is not reaped, bug #4958

2016-04-19 Thread Manash Raja
Hi, I was looking at this bug: https://bugs.sugarlabs.org/ticket/4958 How can I create the bug on my machine to test? Even though I created a patch, can you please verify: https://github.com/ManashRaja/sugar/commit/d1d3c852adb6a9a2c10942ed6a96f2c4bb7ed61b Thanks Manash