Variables in /etc/default/tomcat7 overridden in /etc/init.d/tomcat7

Bug #1354193 reported by Jon Stumpf
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tomcat7 (Ubuntu)
Expired
Medium
Unassigned

Bug Description

Many variables defined in /etc/default/tomcat7 are overridden in /etc/init.d/tomcat7. In short, they cannot be set in /etc/default/tomcat7. These are: TOMCAT7_USER, TOMCAT7_GROUP, TOMCAT7_SECURITY, JVM_TMP, and JAVA_HOME.

If empty, JVM_TMP was set to "/etc/tomcat7-$NAME-tmp". With NAME="tomcat7", this was redundant and I fixed it to match what is documented in /etc/default/tomcat7.

In the (temporary) logic to set JAVA_HOME, there was an error in enumerating the java-7-openjdk's. The error caused the enumerated list to only be the last directory discovered. I fixed the error to match the logic for the java-6-openjdk's.

Lastly, the "auto-generate" message for the policies is hardcoded. I added logic to list both the logical path and, if different, the physical path to the policy.d directory.

Revision history for this message
Jon Stumpf (jon-stumpf) wrote :
description: updated
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "tomcat7.diff" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Revision history for this message
Robie Basak (racb) wrote :

Thank you for taking the time to report this bug and helping to make Ubuntu better.

Ubuntu is in sync with Debian on tomcat7, and it sounds like this is a bug which would equally apply to Debian. So the next course of action here is to check if the bug affects Debian, and if it does then to file a bug there if there isn't already one filed. When Debian fixes the bug, Ubuntu can sync.

tags: added: needs-upstream-report
Changed in tomcat7 (Ubuntu):
importance: Undecided → Medium
Revision history for this message
Emmanuel Bourg (ebourg) wrote :

The variables in the init file are defined before loading /etc/default/tomcat7, I fail to see why the values were ignored in your case, but I don't think your patch will fix it.

Changed in tomcat7 (Ubuntu):
status: New → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for tomcat7 (Ubuntu) because there has been no activity for 60 days.]

Changed in tomcat7 (Ubuntu):
status: Incomplete → Expired
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.