Your message dated Sat, 20 Apr 2013 17:32:45 +0000
with message-id <[email protected]>
and subject line Bug#689333: fixed in drupal7 7.22-1
has caused the Debian Bug report #689333,
regarding drupal7: cron.sh silently ignores error conditions - except for the 
last
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
689333: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=689333
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: drupal7
Version: 7.14-1~bpo60+1
Severity: normal
Tags: patch

When running a multisite installation, the provided
/usr/share/drupal7/scripts/cron.sh ignores failure conditions in all
but the last found sites - And for this last site, it just reports an
unhelpful "/USR/SBIN/CRON[8921]: (CRON) error (grandchild #8938 failed
with exit status 22)" message in the logs.

The attached patch fixes this condition, along with #659848 (CRON_KEY
persisting between sites). Yes, it will be more verbose in case of
error, but it will no longer ignore curl's --fail switch.

Thanks,

-- System Information:
Debian Release: 6.0.6
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable')
Architecture: i386 (i686)

Kernel: Linux 2.6.32-5-openvz-686 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages drupal7 depends on:
ii  apache2              2.2.16-6+squeeze8   Apache HTTP Server metapackage
ii  apache2-mpm-prefork  2.2.16-6+squeeze8   Apache HTTP Server - traditional n
ii  curl                 7.21.0-2.1+squeeze2 Get a file from an HTTP, HTTPS or 
ii  dbconfig-common      1.8.46+squeeze.0    common framework for packaging dat
ii  debconf [debconf-2.0 1.5.36.1            Debian configuration management sy
ii  mysql-client-5.1 [vi 5.1.63-0+squeeze1   MySQL database client binaries
ii  php5                 5.3.3-7+squeeze14   server-side, HTML-embedded scripti
ii  php5-gd              5.3.3-7+squeeze14   GD module for php5
ii  php5-mysql           5.3.3-7+squeeze14   MySQL module for php5
ii  php5-pgsql           5.3.3-7+squeeze14   PostgreSQL module for php5
ii  postfix [mail-transp 2.7.1-1+squeeze1    High-performance mail transport ag
ii  postgresql-client    8.4.13-0squeeze1    front-end programs for PostgreSQL 
ii  postgresql-client-8. 8.4.13-0squeeze1    front-end programs for PostgreSQL 
ii  wwwconfig-common     0.2.1               Debian web auto configuration

Versions of packages drupal7 recommends:
ii  mysql-server           5.1.63-0+squeeze1 MySQL database server (metapackage
ii  mysql-server-5.1 [mysq 5.1.63-0+squeeze1 MySQL database server binaries and
ii  postgresql             8.4.13-0squeeze1  object-relational SQL database (su

drupal7 suggests no packages.

-- debconf information:
  drupal7/pgsql/changeconf: false
  drupal7/dbconfig-remove:
  drupal7/pgsql/authmethod-user:
  drupal7/db/basepath:
  drupal7/upgrade-error: abort
* drupal7/dbconfig-install: false
  drupal7/purge: false
  drupal7/upgrade-backup: true
  drupal7/pgsql/manualconf:
  drupal7/database-type:
  drupal7/internal/reconfiguring: false
  drupal7/install-error: abort
  drupal7/remote/port:
  drupal7/remove-error: abort
  drupal7/mysql/admin-user: root
  drupal7/pgsql/admin-user: postgres
  drupal7/missing-db-package-error: abort
  drupal7/dbconfig-upgrade: true
  drupal7/remote/host:
  drupal7/db/app-user: drupal7
  drupal7/pgsql/authmethod-admin: ident
  drupal7/db/dbname: drupal7
  drupal7/pgsql/no-empty-passwords:
  drupal7/internal/skip-preseed: true
  drupal7/passwords-do-not-match:
  drupal7/dbconfig-reinstall: false
  drupal7/mysql/method: unix socket
  drupal7/pgsql/method: unix socket
  drupal7/remote/newhost:
--- cron.sh.orig        2012-10-01 11:13:06.000000000 -0500
+++ cron.sh     2012-10-01 11:05:20.000000000 -0500
@@ -2,6 +2,8 @@
 
 for site in /etc/drupal/7/sites/* ; do
        BASE_URL=""
+       CRON_KEY=""
+       FULL_URL=""
 
        if [ ! "`basename $site`" = "all" ]; then
                for file in $site/baseurl.php $site/settings.php; do
@@ -19,9 +21,17 @@
                fi
 
                if [ "X$CRON_KEY" = "X" ] ; then
-                       curl --fail --silent --compressed --location 
$BASE_URL/cron.php
+                       FULL_URL="$BASE_URL/cron.php"
                else
-                       curl --fail --silent --compressed --location 
$BASE_URL/cron.php?cron_key=$CRON_KEY
+                       FULL_URL="$BASE_URL/cron.php?cron_key=$CRON_KEY"
+               fi
+
+               if curl -S --fail --silent --compressed --location $FULL_URL ; 
then
+                       # Success!
+                       true
+               else
+                       echo "Error running the periodic maintenance for $site: 
CURL exit code $?"
+                       echo "Requested URL: $FULL_URL"
                fi
        fi
 done

--- End Message ---
--- Begin Message ---
Source: drupal7
Source-Version: 7.22-1

We believe that the bug you reported is fixed in the latest version of
drupal7, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Luigi Gangitano <[email protected]> (supplier of updated drupal7 package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Sat, 20 Apr 2013 18:29:57 +0200
Source: drupal7
Binary: drupal7
Architecture: source all
Version: 7.22-1
Distribution: unstable
Urgency: low
Maintainer: Luigi Gangitano <[email protected]>
Changed-By: Luigi Gangitano <[email protected]>
Description: 
 drupal7    - fully-featured content management framework
Closes: 659848 685006 685333 689333 703322
Changes: 
 drupal7 (7.22-1) unstable; urgency=low
 .
   [ Luigi Gangitano ]
   * New upstream release (Closes: #685006)
     - Remove patches integrated upstream
       + debian/patches/40_SA-CORE-2012-003
       + debian/patches/50_SA-CORE-2012-004
       + debian/patches/60_SA-CORE-2013-001
       + debian/patches/70_SA-CORE-2013-002
     - Fixes Test module failure with curl >= 5.4.4 (Closes: #703322)
 .
   * debian/control
     - Bumped Standard-Version to 3.9.4, no change needed
     - Added Gunnar Wolf <[email protected]> to Uploaders
 .
   * debian/cron.sh
     - Fixed CRON_KEY not resetting each loop iteration (Closes: #659848)
       (Thanks to Michael Wikberg <[email protected]>)
     - Fixed missing report of errors except last one (Closes: #689333)
       (Thanks to Gunnar Wolf <[email protected]>)
 .
   * debian/README.Debian
     - Fixed location of locally installed modules (Closes: #685333)
 .
   * debian/watch
     - Updated by Bart Martens <[email protected]>
Checksums-Sha1: 
 b4f4f28d7e627da941468dd9d333c3449155631c 1220 drupal7_7.22-1.dsc
 81c4d72b374dbcb338ec53049ca659a2d1ee9d87 3183014 drupal7_7.22.orig.tar.gz
 4058c4270b529e3ce3cff130358c42b66131433c 190829 drupal7_7.22-1.debian.tar.gz
 b83dfece3132c748243507ac73990f70705ba925 3232472 drupal7_7.22-1_all.deb
Checksums-Sha256: 
 faa7c9472abc00359317605ef0392187e330d2cc179e57a11c8928c681919e95 1220 
drupal7_7.22-1.dsc
 3f64ccfbb1c62bdef870207aa5de84ad11da1c8a94800fce5ad3a484b47cb29b 3183014 
drupal7_7.22.orig.tar.gz
 52f9a5eefe6fb1f7057f8ad475e801e5875c2e77fa7e67c02159745402b35913 190829 
drupal7_7.22-1.debian.tar.gz
 b2c0b689575e41536ee7c601baabd24fef3127a51113081da00766cc7cd5944f 3232472 
drupal7_7.22-1_all.deb
Files: 
 cb20b647aad4d881136030cbf6a3fbbc 1220 web extra drupal7_7.22-1.dsc
 068d7a77958fce6bb002659aa7ccaeb7 3183014 web extra drupal7_7.22.orig.tar.gz
 97bfd0762cd8b9c819aef71dba343b58 190829 web extra drupal7_7.22-1.debian.tar.gz
 4d930acb6d55d00307f894cfaa75a726 3232472 web extra drupal7_7.22-1_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.13 (Darwin)

iEYEARECAAYFAlFyz6MACgkQ8ZumGJJMDCaNSgCeP/TbS6cvPXcsC7QzUIb+7cv/
lFMAn0qy3FTQoakCZJqBn7u5y/eNIQiL
=NwaT
-----END PGP SIGNATURE-----

--- End Message ---

Reply via email to