This is the mail archive of the
kawa@sourceware.org
mailing list for the Kawa project.
Re: please try out the Kawa binary pre-release
- From: Per Bothner <per at bothner dot com>
- To: Sudarshan S Chawathe <chaw at eip10 dot org>
- Cc: Kawa mailing list <kawa at sourceware dot org>
- Date: Thu, 3 Nov 2016 10:45:43 -0700
- Subject: Re: please try out the Kawa binary pre-release
- Authentication-results: sourceware.org; auth=none
- References: <7753.1478163506@vereq.eip10.org>
On 11/03/2016 01:58 AM, Sudarshan S Chawathe wrote:
I noticed a minor problem: The automatic CLASSPATH-setting code in the
'kawa' shell script does not do the intended (I think) thing when that
shell script is invoked as "./kawa" (leading to an error "Could not find
or load main class kawa.repl). I believe a small change to the sed
command in the kawa script does the trick:
kawadir=`echo "$thisdir" | sed -e 's|/bin\(/.\)*$||'`
Perhaps it is safe to replace the '*' with a '?', but the above will
cover the case of multiple trailing './' substrings if they somehow
appear.
Thanks! I checked your fix into Subversion.
With one minor change: I escaped the period:
kawadir=`echo "$thisdir" | sed -e 's|/bin\(/\.\)*$||'`
I am including a context diff below for completeness sake.
For future reference: Most people and projects these days prefer
unified (-u) diffs. Also note your patch was reversed.
--
--Per Bothner
per@bothner.com http://per.bothner.com/