Bug 982787 - broken comment syntax in catalina.sh for tomcat 7
Summary: broken comment syntax in catalina.sh for tomcat 7
Keywords:
Status: CLOSED EOL
Alias: None
Product: JBoss Enterprise Web Server 2
Classification: JBoss
Component: tomcat7
Version: 2.1.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ER04
: 2.1.0
Assignee: Jean-frederic Clere
QA Contact: Libor Fuka
URL:
Whiteboard:
Depends On:
Blocks: 988396
TreeView+ depends on / blocked
 
Reported: 2013-07-09 21:52 UTC by Aaron Ogburn
Modified: 2019-06-13 12:09 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Previously in JBoss Enterprise Web Server, starting Tomcat 7 using the <filename>catalina.sh</filename> file resulted in a <literal>command not found</literal> error due to the broken comment syntax in <filename>catalina.sh</filename>. This is now fixed in JBoss Enterprise Server 2.1. Starting Tomcat with <filename>catalina.sh</filename> works as expected.
Clone Of:
: 988396 (view as bug list)
Environment:
Last Closed: 2019-06-13 12:09:36 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Patch that fixes typos in tomcat7.conf (1.10 KB, patch)
2013-07-22 17:54 UTC, David Knox
no flags Details | Diff

Description Aaron Ogburn 2013-07-09 21:52:21 UTC
Description of problem:

It looks like as a result of JBEWS-178, some bad comments were added to EWS 2.0.1/tomcat 7's catalina.sh:

+#JBPAPP-3749/6036/9724
+# Cannot resolve user database reference
+# the real problem is a cnfe that is avoided by configuring
+# the -Djavax.sql.DataSource.Factory. This fixes the rpm install.
+# The zip install is fixed in catalina.sh
+# JAVA_OPTS="${JAVA_OPTS} -Djavax.sql.DataSource.Factory=org.apache.commons.dbcp.BasicDataSourceFactory"
+


Tomcat starts regardless, but we get unneeded complaints commands not found:

$ ./catalina.sh start
./catalina.sh: line 267: +#JBPAPP-3749/6036/9724: No such file or directory
./catalina.sh: line 268: +#: command not found
./catalina.sh: line 269: +#: command not found
./catalina.sh: line 270: +#: command not found
./catalina.sh: line 271: +#: command not found
./catalina.sh: line 272: +#: command not found
./catalina.sh: line 273: +: command not found


Just need to drop the +s from those comment lines



Version-Release number of selected component (if applicable):


How reproducible:

Always


Steps to Reproduce:
1.Run ./catalina.sh start
2.
3.

Actual results:

"command not found" complaints


Expected results:

No such complaints


Additional info:

Comment 3 David Knox 2013-07-22 17:54:03 UTC
Created attachment 776990 [details]
Patch that fixes typos in tomcat7.conf

The problem affects tomcat7 only.

Comment 5 Libor Fuka 2014-06-27 10:37:07 UTC
There is still + in catalina.sh (line nr# 271):
rhel5-i386, rhle5-x86_64 zips - tomcat7/catalina.sh

Comment 6 Libor Fuka 2014-06-27 10:39:09 UTC
please fix it in ER4, if its possible

Comment 7 David Knox 2014-06-30 15:07:28 UTC
regression in 7.0.40-11_patch_03

Comment 8 Libor Fuka 2014-07-17 15:40:50 UTC
VERIFIED on ER4

Comment 9 Mandar Joshi 2014-08-11 13:42:15 UTC
Added Doc Text


Note You need to log in before you can comment on or make changes to this bug.