jetty does not restart properly (test for already running is inverted for restart action)

[johan.gant]

Binary package hint: solr-jetty

1. Description: Ubuntu 10.04.1 LTS
Release: 10.04

$ dpkg -s solr-jetty
Package: solr-jetty
Status: install ok half-configured
Priority: optional
Section: web
Installed-Size: 60
Maintainer: Ubuntu MOTU Developers <email address hidden>
Architecture: all
Source: solr
Version: 1.2.0+ds2-5ubuntu1
Depends: solr-common (>= 1.2.0+ds2-5ubuntu1), jetty (>= 5.1)
Conflicts: solr-tomcat5.5
Description: enterprise search server based on Lucene - Jetty integration
 Solr is an open source enterprise search server based on the Lucene
 Java search library, with XML/HTTP and JSON APIs, hit highlighting,
 faceted search, caching, replication, and a web administration
 interface. It runs in a Java servlet container such as Tomcat.

ProblemType: Package
DistroRelease: Ubuntu 10.04
Package: solr-jetty 1.2.0+ds2-5ubuntu1
ProcVersionSignature: Ubuntu 2.6.32-24.43-generic 2.6.32.15+drm33.5
Uname: Linux 2.6.32-24-generic i686
Architecture: i386
Date: Thu Sep 30 09:47:22 2010
ErrorMessage: subprocess installed post-installation script returned error exit status 1
PackageArchitecture: all
SourcePackage: solr
Title: package solr-jetty 1.2.0+ds2-5ubuntu1 failed to install/upgrade: subprocess installed post-installation script returned error exit status 1

[Thierry Carrez (ttx)]

I reproduced it. If jetty is laredy running, solr-jetty postinst tries to:
  invoke-rc.d --quiet jetty restart
But that returns 1 (jetty is already running), so the postinst fails.

The workaround is to stop jetty (sudo service jetty stop) before installing solr-jetty.

[Thierry Carrez (ttx)]

The issue is in fact in jetty. restart action does not test properly if jetty is already running (should be if ! start-stop-daemon).

[Niels Thykier (niels-thykier)]

Hey,

I noticed we fixed this on the Debian side 6.1.24-6, so Maverick should be unaffected.

~Niels

[Thierry Carrez (ttx)]

jetty (6.1.24-6) unstable; urgency=medium

  * Removed Depends on JREs for library packages, no longer required
    by the policy.
  * Added missing depends on JREs for the jetty package.
  * Made init script ignore weird exit statuses from logrotate.
    (Closes: 589681, LP: #607202)
  * Fixed broken restart command in jetty init script. Was missing a
    negation.
 -- Benjamin Drung <email address hidden> Mon, 05 Jul 2010 22:28:14 +0200

[Tobias G. Pfeiffer (tgpfeiffer)]

I have this problem on lucid, where I installed the latest available jetty version (6.1.22-1ubuntu1) from the repositories. Would be great to have the fix backported to that version as well.