langtools in b20

[Rémi Forax]

Jonathan Gibbons a écrit :
>
> On Sep 23, 2007, at 4:31 PM, Rémi Forax wrote:
>
>> Jonathan Gibbons a écrit :
>>> Hi Remi,
>>>
>>> I'd like to help but I'll need a bit more to go on.
>>>
>>> What are the issues you are seeing with the copyright headers?
>> I think copyright headers are not up to date if you compare with
>> the last update of the openjdk/compiler workspace.
>> I don't know if it's a problem or not.
>
> This is in KSL, right?
no,
headers of files from 
https://openjdk.dev.java.net/svn/openjdk/compiler/trunk
seems more accurate than headers of files from 
https://openjdk.dev.java.net/svn/openjdk/jdk/trunk/langtools
>
>>
>>>
>>> I assume you can edit the files in the javax.lang.model package OK;
>> yes
>
>>> what problems are you having when you try and compile the workspace?
>> javax.model.* is a part of JDK so when i compile a patched version 
>> javac/tree/JCTree.java,
>> that adds two new trees, the build process use the package 
>> javax.model.element of the boot jdk and not
>> the one that you can find in the source folder.
>
> The build process should be putting all the necessary files on the 
> command line.  Can you provide some script output to show what you are 
> doing and how it is going wrong for you.
ok, it's compile now, i've made "ant clean" and it's working
(almost because it seems i forget that    "byte data[]" is a valid field 
declaration,
 but that is another story)

So i think there is a dependency problem with in the build.xml,
here is the use case:
1) compile the whole repository
    "ant"
2) patch the source file (here com.sun.source.tree.TreeVisitor)
3) compile using ant
    -> some files in /build/bootstrap/classes/ are not recompiled
        so it fails
3bis) use "ant clean; ant"
         it compiles

>
> -- Jon
>
>
it's 2 am here, so i have to go to sleep.

cheers,
Rémi