problem extending lombok about lombok

[lombokissues]

👤 voidstar   🕗 Aug 09, 2009 at 17:01 UTC

What steps will reproduce the problem?

1. unzip
2. ant compile

What is the expected output? What do you see instead?
ray@ featuritis:~/projects/lombok_extra>ant compile
Buildfile: build.xml

compile:
[mkdir] Created dir: /Users/ray/projects/lombok_extra/build/lombok
[javac] Compiling 1 source file to /Users/ray/projects/lombok_extra/build/lombok
[javac] Compiling 1 source file to /Users/ray/projects/lombok_extra/build/lombok
[javac] /Users/ray/projects/lombok_extra/src/lombok/javac/handlers/HandleGet.java:81:
createGetterForField(lombok.AccessLevel,lombok.javac.JavacAST.Node,lombok.javac.JavacAST.No
de,com.sun.tools.javac.util.JCDiagnostic.DiagnosticPosition,boolean) in
lombok.javac.handlers.HandleGet cannot be applied to
(lombok.AccessLevel,lombok.javac.JavacAST.Node,lombok.javac.JavacAST.Node,java.lang.Object,
boolean)
[javac] return this.createGetterForField(level, fieldNode, annotationNode,
annotationNode.get(), true);
[javac] ^
[javac] 1 error

BUILD FAILED
/Users/ray/projects/lombok_extra/build.xml:70: Compile failed; see the compiler error output
for details.

What version of the product are you using? On what operating system?
v0.8.3

Please provide any additional information below.

[lombokissues]

👤 voidstar   🕗 Aug 09, 2009 at 17:01 UTC

🔗 lombok_extra.zip

[lombokissues]

👤 reinierz   🕗 Aug 10, 2009 at 18:14 UTC

Congratulations; you found a javac bug.

We'll have to rewrite how AST's generics works, which is non-trivial. Don't hold your breath :(

lombok.javac.JavacAST.Node should be castable to a lombok.core.AST.Node but it isn't. It works for
lombok itself because as long as all relevant classes are compiled in the same run, it does work - it fails when
part of the classes are already in .class form.

[lombokissues]

👤 voidstar   🕗 Aug 11, 2009 at 01:05 UTC

Thanks for looking into it.

I can make it compile by adding just lombok.javac.JavacAST.java (and EclipseAST.java for the eclipse handler).
Not quite as bad as having to fork the whole project, although it would be nice to depend only on the lombok jar.

This workaround will save me having to hold my breath though, so thanks for the push in the right direction :)

[lombokissues]

👤 reinierz   🕗 Oct 16, 2009 at 08:43 UTC

Fixed in commit b5c8b72 - should be out in v0.9.

[lombokissues]

👤 reinierz   🕗 Oct 16, 2009 at 08:49 UTC

[lombokissues]

End of migration