OK, I'll modify the readme to say that there are classpath issues  
with sci, and I'll file a bug to fix it.

Thanks!

jim

On Sep 16, 2006, at 5:58 AM, P T Withington wrote:

> I approve of the build changes, but not the change to the readme.   
> If sci is broken because of classpath issues, it needs to be  
> fixed.  (It looks to me like this is another script where we tried  
> to use this extdirs thing, which does not work.)  sci is still a  
> useful tool for running the compiler debugging scripts which use  
> jython to interactively poke at the compiler.
>
> If you don't want to fix the classpath issue in sci here, please  
> file another bug to track it.
>
> On 2006-09-16, at 01:36 EDT, Jim Grandy wrote:
>
>> Change change.KIwizooTp.txt by [EMAIL PROTECTED] /Volumes/ 
>> data/Users/jgrandyw/dev/svn/openlaszlo/trunk/ on 2006-09-15  
>> 22:20:49 PDT
>>
>> Summary: Replace use of 'sci' shell script in ref guide  
>> compilation with a simple ant script in the right place.
>>
>> Technical Reviewer: ptw (pending)
>> QA Reviewer: ben (pending)
>> Doc Reviewer:
>>
>> Bugs Fixed: LPP-2697
>>
>> Details: While diagnosing a classpath problem, Ben and I noticed  
>> that the
>> "api-xml" ant task in docs/src was invoking a shell script that then
>> invoked java. This practice has caused classpath problems in the  
>> past,
>> and the simple solution is to modify the ant task to invoke java
>> directly. In addition, the shell script used a Jython program  
>> stored in
>> a different part of the source tree -- WEB-INF/lps/server/sc -- so we
>> chose to reimplement the task in that directory's build.xml to  
>> improve
>> modularity.
>>
>> The Jython script is not very long, and heavily leverages
>> the JavaScript compiler that is now written in Java. Ideally we would
>> rewrite the jsdoc2xml.py script in Java as well -- especially  
>> since it
>> will need to be largely rewritten in Legals to handle the new
>> class/interface syntax -- but I'm submitting these changes now in  
>> trunk
>> to get them in. (LPP-2698 is a placeholder for this leftover work.)
>>
>> Tests:Ran 'ant reference' in docs/src
>>
>> Files:
>> M      WEB-INF/lps/server/sc/README.TXT
>> M      WEB-INF/lps/server/build.xml
>> M      docs/src/build.xml
>>
>> <patch.jgrandyw.2cKl.tgz>
>


_______________________________________________
Laszlo-dev mailing list
[email protected]
http://www.openlaszlo.org/mailman/listinfo/laszlo-dev

Reply via email to