Amuled leaves zombie processes

[Eduardo Durany Fernández]

Amuled leaves zombie processes when "ExecOnCompletionCommand" is used.

[Emilio Pozuelo Monfort (pochu)]

Hello Roax

Could you provide us more info?
Which version of Ubuntu were you running? And which one of amule?

[Eduardo Durany Fernández (edurany)]

Ubuntu edgy up to date and amule-daemon 2.1.3-1.

Amule options:
ExecOnCompletion=1
ExecOnCompletionCommand=/usr/local/bin/avisarmail "%FILE"

The script is executed correctly but a ps shows this:
p2p 9130 0.0 0.0 0 0 ? Z 11:10 0:00 [avisarmail] <defunct>

This is the script:

#!/bin/sh
file=`basename "$1"`
asunto="aMule -- $file"
echo "El archivo $file ha sido descargado completamente." | mail -s "$asunto" roax@localhost
exit 0

[Emilio Pozuelo Monfort (pochu)]

Hi Roax

I don't know much about bash, but couldn't be the problem in your script? Maybe it isn't closing well because the script, and not because amule.

Could you confirm it, and tell us?

Thank you
Pochu

[Emilio Pozuelo Monfort (pochu)]

There is a patch which adds charset support to amule:
http://forum.amule.org/index.php?topic=12335

It hasn't been applied yet, 'couse it needs more testing, but it will probably be applied for 2.2.0 (Feisty+1).

Best regards
Emilio

[Emilio Pozuelo Monfort (pochu)]

Fixed in the amule source tree. This will be fixed in Ubuntu once we have 2.2.0

If you can try a recent (from March 11th) and tell if it works fine, it would be fine.

Best regards
Emilio

[Emilio Pozuelo Monfort (pochu)]

I forgot to mention the ChangeLog entry:

 * Added a termination process to wxExecute() calls, so that we no longer leave zombies.

[Eduardo Durany Fernández (edurany)]

Great news. I am very busy at the moment, so i will wait until 2.2.0 is released.
Thanks for your attention.

[Launchpad Janitor (janitor)]

This bug was fixed in the package amule - 2.2.0~svn20080218-0ubuntu1

---------------
amule (2.2.0~svn20080218-0ubuntu1) hardy; urgency=low

  * New svn snapshot from 20080218 tarball, renamed debian/ to
    debian-upstream/.
    FeatureFreeze exception: LP: #192156.
    The 2.2.0 isn't released yet because of build failures in MacOS X,
    but is stable enough and recommended by upstream.
    LP: #40238, #112745, #65496, #123695, #192781.

  * debian/patches/cas_configfile.c_good_default_paths.diff:
    - Updated for the new release.
  * debian/patches/wx-2.8.diff,
    debian/patches/configure_proper_libpng_detection.diff,
    debian/patches/for_upstream-manpage_typos.diff
    - Removed, fixed upstream.
  * debian/patches/nodes-url.diff:
    - Removed, as upstream uses now a different server list.
  * debian/patches/series:
    - Updated accordingly.
  * debian/control:
    - Require libcrypto++-dev >= 5.5.2-1, as aMule now checks for crypopp
      instead of crypto++, and 5.5.2-1 added the necessary symlinks.
      Otherwise aMule FTBFS.
    - Build-depend on libgeoip-dev to get geoip support.
    - Require debhelper >= 5.0.51 to have dh_icons.
    - Moved homepage to the Homepage field.
    - Updated Standards-Version to 3.7.3. No changes needed.
  * debian/mans/amulegui.1:
    - Removed, as it's shipped upstream now.
  * debian/amule-utils-gui.manpages:
    - Ship amulegui.*1 manpages from upstream, and stop shipping
      debian/mans/amulegui.1
  * debian/rules:
    - Stop manually installing amulegui.desktop and amulegui.xpm.
      It's now done upstream.
    - No need to symlink debian/amule.xpm to debian/amule-32.xpm and
      remove it in clean. Let's rename amule-32.xpm to amule.xpm instead.
    - Build with geoip support.
    - Call dh_icons and dh_desktop.
    - Explicitly install debian/amule.xpm, as upstream's amule.xpm isn't
      menu-policy compliant (32x32 vs 128x128).
  * debian/amule-32.xpm, debian/amule.xpm:
    - Renamed debian/amule-32.xpm as debian/amule.xpm.
  * debian/amulegui.desktop:
    - Removed, it's included upstream now.

 -- Emilio Pozuelo Monfort <email address hidden> Mon, 18 Feb 2008 19:43:28 +0100