FS#50081 - [incron] incrond creates zombies

Attached to Project: Community Packages
Opened by jaro3 (jaro33) - Monday, 18 July 2016, 09:30 GMT
Last edited by Sergej Pupykin (sergej) - Tuesday, 31 October 2017, 11:59 GMT
Task Type Bug Report
Category Upstream Bugs
Status Closed
Assigned To Sergej Pupykin (sergej)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 2
Private No

Details

Description:

incron 0.5.12-1 leaves Zombies after executing shell scripts
already reported here:

https://bbs.archlinux.org/viewtopic.php?id=212456

https://github.com/ar-/incron/issues/22

Additional info:
* package version(s)
* config and/or log files etc.


Steps to reproduce:
This task depends upon

Closed by  Sergej Pupykin (sergej)
Tuesday, 31 October 2017, 11:59 GMT
Reason for closing:  Fixed
Comment by Sergej Pupykin (sergej) - Thursday, 17 November 2016, 14:14 GMT
Suggested patch affects behaviour if event handler execution takes much time. Let's upstream decide and fix it.
Comment by Ronan (ronjouch) - Monday, 23 October 2017, 20:30 GMT
Hi. https://github.com/ar-/incron/issues/22 got fixed; I've been testing it on my home machine for a few days and it seems okay.

Not sure what's the policy here at Arch for packaging unreleased versions, so sorry for the noise if the answer is "Yes but unrelease, will release when upstream releases".

Thanks for your work on maintaining packages :)
Comment by Eli Schwartz (eschwartz) - Monday, 23 October 2017, 20:33 GMT
Considering how active upstream seems to (not) be, backporting this patch seems pretty reasonable.
Comment by Ronan (ronjouch) - Tuesday, 31 October 2017, 02:12 GMT
@eschwartz re "backporting this patch seems pretty reasonable": cool :)

However, maybe wait a bit, there's discussion at https://github.com/ar-/incron/pull/23#issuecomment-340542365 that the current patch isn't entirely satisfying and a new one is brewing. I'll ping here once the new one is merged.
Comment by Ronan (ronjouch) - Tuesday, 31 October 2017, 02:13 GMT
Well that was fast, the new patch is already merged, https://github.com/ar-/incron/pull/42
Comment by Sergej Pupykin (sergej) - Tuesday, 31 October 2017, 09:14 GMT
Please try incron-0.5.12-2
Comment by Ronan (ronjouch) - Tuesday, 31 October 2017, 11:48 GMT
Installed, reloaded systemd daemons, restarted incrond; incron still works fine for me and no zombies. Thanks@!

Loading...