Need reviewer - minor top level make changes
Kelly O'Hair
Kelly.Ohair at Sun.COM
Wed Dec 23 18:11:32 UTC 2009
Andrew John Hughes wrote:
> 2009/11/19 Kelly O'Hair <Kelly.Ohair at sun.com>:
>> Need reviewer. Some very minor top level make file changes.
>>
>> 6727046: Add message when docs are skipped in control build
>> 6864011: typo? in top level Makefile: DAYE_STAMP
>>
>> http://cr.openjdk.java.net/~ohair/openjdk7/top-make-fixes/webrev/
>>
>> -kto
>>
>>
>
> This is a bit more than just adding a message. It also adds:
>
> + # No DOCS build when JDK_UPDATE_VERSION set
> + ifdef JDK_UPDATE_VERSION
> + GENERATE_DOCS=false
> + endif
>
Sorry about that, I assumed I was making a correction to a long standing
problem. When we build jdk update releases, the docs are not regenerated.
The variable JDK_UPDATE_VERSION indicates a jdk update build, I
just assumed that's what it was being used for.
> JDK_UPDATE_VERSION has to be set for IcedTea to deal with broken
> plugins which expect this (such as
> http://www.java.com/en/download/help/testvm.xml). I don't think it
> follows that turning on a version setting forces documentation off.
> Can we make this an #ifndef OPENJDK block?
Strange use of JDK_UPDATE_VERSION if you ask me.
Why not just 'make GENERATE_DOCS=true' with the IcedTea builds?
Or am I missing the point?
What exactly is it that JDK_UPDATE_VERSION provides for IcedTea builds?
-kto
More information about the build-dev
mailing list