Atlassian JIRA   History | Log In     View a printable version of the current page. Get help!  
Issue Details [XML]

Key: XRP-29
Type: Bug Bug
Status: Closed Closed
Resolution: Applied
Priority: Trivial Trivial
Reporter: John Shott
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
xReporter

export CLASSPATH in xreporter-cocoon shell script

Created: 08/Mar/05 03:41 PM   Updated: 29/Apr/05 03:39 PM
Component/s: xReporter-Cocoon
Affects Version/s: 1.2.1
Fix Version/s: 1.2.1

File Attachments: 1. Text File xreporter-cocoon-sh.patch (0.7 kb)

Environment: Solaris 8 and 9 ... and probably other Unix machines that don't
support "export CLASSPATH=...." single-line export env variables.


 Description   
I believe that there is a minor problem with the xreporter-cocoon
shell script (the one that resides in xreporter-cocoon/bin/xreporter-cocoon.

It makes use of the contstruct:

export CLASSPATH=...

While many versions of /bin/sh on Linux seem to support this
set-and-export at the same time syntax, I believe that the default version of /bin/sh that comes on Solaris 8/9 does not understand this construct and results in an error when trying to run 'xreporter-cocoon' or 'xreporter-cocoon deploy'.

The equivalent shell script for xreporter (xreporter/bin/xreporter)
uses the safer set-first-export-later syntax:

CLASSPATH=...
export CLASSPATH;

I'd suggest that xreporter-cocoon/bin/xreporter-cocoon be modified very slightly to match the syntax used in the xreporter/bin/xreporter script which will, I beleve, be compatible
with all versions of /bin/sh.

I've attached an almost trivial patch file to incorporate that changed.

Thanks for your consideration,

John Shott

 All   Comments   Change History      Sort Order:
Comment by Marc Portier [09/Mar/05 02:50 PM]