Amuled leaves zombie processes

Bug #65496 reported by Eduardo Durany Fernández
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
amule (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Amuled leaves zombie processes when "ExecOnCompletionCommand" is used.

Revision history for this message
Emilio Pozuelo Monfort (pochu) wrote :

Hello Roax

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

Changed in amule:
status: Unconfirmed → Needs Info
Revision history for this message
Eduardo Durany Fernández (edurany) wrote :

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

Revision history for this message
Emilio Pozuelo Monfort (pochu) wrote :

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

Changed in amule:
assignee: nobody → pochu
Changed in amule:
importance: Undecided → Low
Changed in amule:
status: Needs Info → In Progress
Revision history for this message
Emilio Pozuelo Monfort (pochu) wrote :

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

Revision history for this message
Emilio Pozuelo Monfort (pochu) wrote :

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

Changed in amule:
status: In Progress → Fix Committed
Revision history for this message
Emilio Pozuelo Monfort (pochu) wrote :

I forgot to mention the ChangeLog entry:

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

Revision history for this message
Eduardo Durany Fernández (edurany) wrote :

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

Changed in amule:
assignee: pochu → nobody
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

Changed in amule:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.