TL make/modules/Makefile requires ALT_JDK_IMPORT_PATH

[Mandy Chung]

Thanks for the pointer to [1].   I will fix it (cr# 6916217).

Mandy

Mark Reinhold wrote:
> I tried building the modules target in the current TL forest with this
> command:
>
>   $ make JDK_BUILD_TARGETS=modules jdk_only
>
> but ran into the following error:
>
>   >>>Making gen-classlist @ Thu Jan  7 09:57:24 PST 2010 ...
>   /NOT-SET/re/jdk/1.7.0/promoted/latest/binaries/linux-i586/bin/java -XX:-PrintVMOptions -XX:+UnlockDiagnosticVMOptions -XX:-LogVMOutput -client -Xmx896m -Xms128m -XX:PermSize=32m -XX:MaxPermSize=160m -Xbootclasspath:/w/tl/build/classes \
>                   -Dclassanalyzer.debug \
>                   -jar /w/tl/build/btjars/classanalyzer.jar \
>                   -jdkhome /w/tl/build \
>                   -config modules.config \
>                   -config modules.group \
>                   -depconfig jdk7.depconfig \
>                   -depconfig optional.depconfig \
>                   -showdynamic \
>                   -output /w/tl/build/tmp/modules/classlist
>   make[4]: /NOT-SET/re/jdk/1.7.0/promoted/latest/binaries/linux-i586/bin/java: Command not found
>   make[4]: *** [gen-classlist] Error 127
>
> It looks like your use of JAVA_TOOLS_DIR in the gen-classlist target
> winds up requiring that ALT_JDK_IMPORT_PATH be set, but it shouldn't
> be necessary to set that variable when doing a full build [1].
>
> This isn't a critical issue -- the workaround is simple enough -- but it
> would be good to fix it at some point.
>
> - Mark
>
>
> [1] http://mail.openjdk.java.net/pipermail/build-dev/2009-October/002457.html
>