There's another kind of duplicate class definition warning:

I've been using this ant script to rebuild the libraries:

$ cd jade/client
$ ant build-libraries -Drelease.id=1.0

To see the error right away, you can run this (from jade/client/):
$ lzc -c components/window/dialog/library.lzx

Compiling: components/window/dialog/library.lzx to 
components/window/dialog/library.gz
Compilation errors occurred:
../../jade/client/components/wtstylishbutton/library.lzo:24:42: 
duplicate class definitions for tooltip; also defined at 
incubator/tooltip/tooltip.lzx:28

components/wtstylishbutton/library.lzo (gunzipped) reads:
<!-- This is a binary library.  Not meant for human consumption. -->
<!-- DO NOT EDIT THIS FILE.  Edit the source and recompile with `-c` -->
<library includes="wtstylishbutton.lzx">
<include href='../../../../lps/components/incubator/tooltip/tooltip.lzx' />
<include href='../../../../lps/components/base/basebutton.lzx' />
...
<interface name='tooltip' extends='node'>
...

Is there a way to get a look at the partial compiler output e.g. 
components/wtstylishbutton/library.lzo?  It's not clear to me where in 
components/wtstylishbutton/library.lzx the include tooltip is added 
again - I grepped components/wtstylishbutton/ and these direct includes 
with no luck - it's difficult to recurse through the includes to find it 
and know how to address the problem:

<library>
     <!--// DEPENDENCIES 
//////////////////////////////////////////////////// -->
     <include href="../../mtext"/>
     <include href="../../panel"/>
     <include href="../../wtstylishbutton"/>
     <include href="../../mcheckbox"/>
...


P T Withington wrote:
> [UPDATE^2
> 
> This update eliminates the duplicate class error and makes the <switch> tag 
> just issue a warning.  When it sees a <switch>, it processes the otherwise 
> clause and keeps going.]
> 
> Change 20091118-ptw-8 by [email protected] on 2009-11-18 16:53:36 EST
>     in /Users/ptw/OpenLaszlo/trunk-2
>     for http://svn.openlaszlo.org/openlaszlo/trunk
> 
> Summary: Binary compiler tweaks
> 
> Bugs Fixed:
> LPP-8624 <script>when ($runtimeflag) {...}</script> crashes library compiler
> LPP-8603 forward reference from instance-class in .lzo file gives error in 
> swf8 and dhtml runtimes
> LPP-8623 Library compiler should complain about <when/> tags
> 
> Technical Reviewer: max, hminsky (pending)
> QA Reviewer: fellenberger, mdemmon (pending)
> 
> Details:
> 
>     LzNode: Learn how to print nodes that are instances of anonymous
>     classes.
> 
>     ClassModel: Give anonymous classes a useful tagname for
>     debugging.  Remove erroneous test that prevented forward
>     references to classes in binary libraries.  Only output tagname
>     for debugging.
> 
>     Parser: For now, <switch> is not permitted in a binary library.
> 
>     ToplevelCompiler: Permit binary compiling an autoinclude.
> 
>     CodeGenerator, JavascriptGenerator:  Match modularization of
>     GenericVisitor, permit compile-time conditionals in user code.
> 
>     Compiler, Transformer:  Work in progress, just check-pointing.
> 
>     GenericVisitor: Add missing visitor, fix a few brain-ohs.
> 
> Tests:
>     Test cases from bug reports.  smokecheck.  QA Reviewers will
>     verify webtop SDK works.
> 
> Files:
> M      WEB-INF/lps/lfc/core/LzNode.lzs
> M      WEB-INF/lps/server/src/org/openlaszlo/sc/JavascriptGenerator.java
> A      WEB-INF/lps/server/src/org/openlaszlo/sc/Transformer.java
> M      WEB-INF/lps/server/src/org/openlaszlo/sc/Compiler.java
> M      WEB-INF/lps/server/src/org/openlaszlo/sc/CodeGenerator.java
> M      WEB-INF/lps/server/src/org/openlaszlo/sc/GenericVisitor.java
> M      WEB-INF/lps/server/src/org/openlaszlo/compiler/ToplevelCompiler.java
> M      WEB-INF/lps/server/src/org/openlaszlo/compiler/ClassModel.java
> M      WEB-INF/lps/server/src/org/openlaszlo/compiler/Parser.java
> 
> 
> Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20091118-ptw-8.tar

-- 
Regards,
Max Carlson
OpenLaszlo.org
_______________________________________________
Laszlo-reviews mailing list
[email protected]
http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews

Reply via email to