Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock
Please unblock package virtuoso-opensource
Hi Release-Team!
We, the Qt/KDE team, would like you to check if the following changes
are within what you are expecting for getting an unblock.
The most important stuff was solving RC #704521 [2], but while looking at it I
found other issues. The complete list of changes in the init script is:
* I removed "set -e" from the script because (according to a footnote in
policy [1]) lsb/init-functions is incompatible with that option.
Furthermore, even the script itself did not deal with that option
correctly, e.g., a non-0 exit code from start-stop-daemon would have
terminated the script without proper logging. This also allowed for
simplification where the $errcode was used previously.
* The RC bug about stopping the daemon [2] is fixed in the stop_server
function by using start-stop-daemon with a proper --retry argument,
instead of killproc. The script has two modes of operation (one to use
root, one with a dedicated user for the daemon), hence there are two
branches in stop_server - I did not simplify this to keep the diff smaller.
* The script used to INT the server due to [3], but according to
upstream documentation [4] TERM should be used in rc.d scripts, so
that's what it does now.
* I removed reload_server and the reload directive because virtuoso does
not support reloading (it used to print an error there, and then exit 0).
* I removed force-stop since (a) it's not mandated or even recommended
by anything (b) stop itself will already use KILL if TERM does not work
and (c) it essentially just duplicated the retry-functionality of
start-stop-daemon, but with way too long delays (60s).
* The changes for the restart directive ensure that the script handles
failure to stop the server appropriately. They also remove the 60s-delay
between stop and start (which is no longer needed now that stop_daemon
properly waits for virtuoso to quit).
[1] http://www.debian.org/doc/debian-policy/footnotes.html#f81
[2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=704521
[3] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=632060
[4] http://docs.openlinksw.com/virtuoso/signalsandexitcodes.html
Diffstats:
changelog| 15
control |5 -
virtuoso-opensource-6.1.init | 131
++-
3 files changed, 49 insertions(+), 102 deletions(-)
Kind regards,
Ralf Jung
Debian Qt/KDE team
unblock virtuoso-opensource/6.1.4+dfsg1-7
-- System Information:
Debian Release: 7.0
APT prefers testing
APT policy: (990, 'testing'), (100, 'unstable')
Architecture: amd64 (x86_64)
Foreign Architectures: i386
Kernel: Linux 3.8.6 (SMP w/4 CPU cores)
diff -Nru virtuoso-opensource-6.1.4+dfsg1/debian/changelog virtuoso-opensource-6.1.4+dfsg1/debian/changelog
--- virtuoso-opensource-6.1.4+dfsg1/debian/changelog 2013-02-25 09:49:36.0 -0300
+++ virtuoso-opensource-6.1.4+dfsg1/debian/changelog 2013-04-06 10:29:22.0 -0300
@@ -1,3 +1,18 @@
+virtuoso-opensource (6.1.4+dfsg1-7) UNRELEASED; urgency=low
+
+ [ Lisandro Damián Nicanor Pérez Meyer ]
+ * Add Sune and myself to Uploaders.
+
+ [ Ralf Jung ]
+ * init script: Use start-stop-daemon (Closes: 704521)
+ * init script: Do not use "set -e", that's incompatible with
+lsb/init-scripts
+ * init script: Stop attemtping to restart when stopping failed
+ * Change maintainer to Debian Krap team
+ * Remove obsolete DM-Upload-Allowed
+
+ -- Debian Krap Maintainers Thu, 04 Apr 2013 17:04:31 +0200
+
virtuoso-opensource (6.1.4+dfsg1-6) unstable; urgency=low
* Add safer-timeout.patch, avoids random FTBFS'es. These random FTBFS'es
diff -Nru virtuoso-opensource-6.1.4+dfsg1/debian/control virtuoso-opensource-6.1.4+dfsg1/debian/control
--- virtuoso-opensource-6.1.4+dfsg1/debian/control 2013-02-02 01:57:32.0 -0300
+++ virtuoso-opensource-6.1.4+dfsg1/debian/control 2013-04-06 10:29:22.0 -0300
@@ -1,8 +1,9 @@
Source: virtuoso-opensource
Section: database
Priority: optional
-Maintainer: José Manuel Santamaría Lema
-DM-Upload-Allowed: yes
+Maintainer: Debian Krap Maintainers
+Uploaders: Lisandro Damian Nicanor Pérez Meyer ,
+ Sune Vuorela
Standards-Version: 3.9.3
Homepage: http://virtuoso.openlinksw.com/wiki/main/Main/
Vcs-Browser: https://alioth.debian.org/plugins/scmgit/cgi-bin/gitweb.cgi?p=pkg-virtuoso/pkg-virtuoso.git
diff -Nru virtuoso-opensource-6.1.4+dfsg1/debian/virtuoso-opensource-6.1.init virtuoso-opensource-6.1.4+dfsg1/debian/virtuoso-opensource-6.1.init
--- virtuoso-opensource-6.1.4+dfsg1/debian/virtuoso-opensource-6.1.init 2013-01-21 22:45:48.0 -0300
+++ virtuoso-opensource-6.1.4+dfsg1/debian/virtuoso-opensource-6.1.init 2013-04-06 10:28:46.0 -0300
@@ -54,11 +54,6 @@
# at /etc/default/$NAME
DAEMON_OPTS="" # Additional options given to the server
-DIETIME=60 # Time to