ecj based bootstrap broken with current repository

[Lillian Angel]

Hi,

Matthias Klose wrote:
> Lillian Angel schrieb:
>> Hi,
>>
>> Matthias Klose wrote:
>>> An ecj based bootstrap is currently broken with the current repository:
>>>
>>> [...]
>>> ecj-poured IcedTea is served: openjdk-ecj/build/linux-amd64
>>> [...]
>>> touch stamps/ports.stamp
>>> [...]
>>> if test "yes" == "yes" ; \
>>>         then \
>>>           g++ -g -O2 -I/usr/include/xulrunner/java
>>> -I/usr/include/xulrunner/plugin -I/usr/include/nspr
>>> -I/usr/include/xulrunner
>>> -I/usr/include/xulrunner/xpcom -I/usr/include/xulrunner/string   \
>>>             -L/usr/lib/xulrunner -lxpcom -lplds4 -lplc4 -lnspr4
>>> -lpthread -ldl
>>>  -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include   -lglib-2.0   \
>>>             -DPNG_NO_MMX_CODE -pthread -I/usr/include/gtk-2.0
>>> -I/usr/lib/gtk-2.0/include -I/usr/include/atk-1.0 -I/usr/include/cairo
>>> -I/usr/include/pango-1.0 -I/usr/include/glib-2.0
>>> -I/usr/lib/glib-2.0/include
>>> -I/usr/include/freetype2 -I/usr/include/libpng12   -pthread -lgtk-x11-2.0
>>> -lgdk-x11-2.0 -latk-1.0 -lpangocairo-1.0 -lpango-1.0 -lcairo
>>> -lgthread-2.0 -lrt
>>> -lgdk_pixbuf-2.0 -lm -lgobject-2.0 -lgmodule-2.0 -ldl -lglib-2.0   \
>>>             -DPACKAGE_VERSION="\"1.5\"" \
>>>             -fpic -shared -o gcjwebplugin.so gcjwebplugin.cc ; \
>>>         fi
>>> find tools -name '*.java' | sort > tools-source-files.txt
>>> mkdir -p lib/tools
>>> /usr/bin/ecj -nowarn  -g -d lib/tools \
>>>           -source 1.6 \
>>>           -sourcepath tools \
>>>           @tools-source-files.txt
>>> Annotation processing got disabled, since it requires a 1.6 compliant JVM
>>> ----------
>>> 1. ERROR in tools/javax/jnlp/BasicService.java (at line 1)
>>>
>>>
>>>         ^
>>> The type java.lang.Object cannot be resolved. It is indirectly
>>> referenced from
>>> required .class files
>>>
>>>
>>>
>>> JAVAC is unconditionally set to ecj in the Makefile, shouldn't the
>>> just built
>>> bootstrap/jdk1.6.0/bin/javac be used?
>> I was notified of this problem a couple of days ago by twisti. I haven't
>> found a fix yet, it seems as though JAVAC is set in configure(.ac
>> FIND_JAVAC) to /usr/bin/ecj for the first part of the build, but left
>> unchanged throughout the second part of the build.
> 
> the tools.jar is always built after the bootstrap jdk, so it should be safe to
> use the just built compiler. I didn't check with an icedtea-against-icedtea
> build. makes the ecj bootstrap work again.

Yes, I have a fix I am testing now. The patch you attached does not 
work with icedtea-against-icedtea.

I will commit soon.

Thx
Lillian