This patch fixes
https://bugzilla.redhat.com/show_bug.cgi?id=569694
569694 - yumdownloader not killed by ABRT
Please review.
--
vda
diff -x '*.po' -d -urpN abrt.9/src/Daemon/abrt-debuginfo-install
abrt.a/src/Daemon/abrt-debuginfo-install
--- abrt.9/src/Daemon/abrt-debuginfo-install 2010-05-12 14:27:08.000000000 +0200
+++ abrt.a/src/Daemon/abrt-debuginfo-install 2010-05-14 16:17:49.030188584 +0200
@@ -249,8 +249,11 @@ download_packages() {
cmd="yumdownloader $yum_repo_opts --quiet $pkg"
$debug && echo "Running: $cmd" >&2
# eval is needed to strip away ''s in $yum_repo_opts
- eval $cmd >>yumdownloader.OUT 2>&1
+ eval $cmd >>yumdownloader.OUT 2>&1 &
+ CHILD_PID=$!
+ wait
err=$?
+ CHILD_PID=""
echo "exitcode:$err" >>yumdownloader.OUT
echo >>yumdownloader.OUT
test $err = 0 || echo "Download of $pkg failed!"
@@ -319,6 +322,11 @@ cd "$tempdir" || exit 2
abort_if_low_on_disk_space 1024
+# A hook to stop yumdownloader, in case we are terminated by kill -TERM etc.
+CHILD_PID=
+trap 'test x"$CHILD_PID" != x"" && kill --
"$CHILD_PID"' EXIT
+
+
$debug && echo "Downloading rpms to $tempdir"