Langtools resources

[Erik Joelsson]

It's being reviewed now. My plan was to wait for my committer status in 
jdk8 to go through (which happens in a couple of hours) and start 
pushing these changes to jdk8/build tomorrow, in time for the merge with 
jdk8/jdk8. This might fail for a number of reasons though so if it's 
urgent, Tim could probably push that particular change today already to 
be sure it makes it.

I could also hold out on that particular change until your change gets 
into jdk8/build. How long would that take, give that build is integrated 
on Wednesdays?

/Erik

On 2012-11-27 17:25, Jonathan Gibbons wrote:
> Erik,
>
> How soon will your change arrive in TL and/or JDK 8?  Right now, until 
> we have the fix, javadoc is broken.
>
> -- Jon
>
> On 11/27/2012 08:14 AM, Erik Joelsson wrote:
>> This change has been noticed and .js is being added with the javadocs 
>> patch:
>>
>> 8003844: build-infra: docs target isn't working properly
>>
>> In that patch I'm also moving the definition of that variable (since 
>> it's needed for bootstrap javadoc generation too). This means that if 
>> you add .js it will cause a merge conflict for the integrator. I 
>> don't know if this is ok or not.
>>
>> /Erik
>>
>> On 2012-11-27 16:46, Jonathan Gibbons wrote:
>>> Build-infra folk,
>>>
>>> We have recently added a new file to javadoc, but the new file is 
>>> "ignored" in builds with the new makefiles, resulting in a broken 
>>> javadoc.
>>>
>>> We will need to add .js to the following list.
>>>
>>> langtools/makefiles/BuildLangtools.gmk: RESOURCE_SUFFIXES:=.gif .xml 
>>> .css javax.tools.JavaCompilerTool
>>>
>>> We'd like to make the change and push it up through TL, with your 
>>> approval.  Will that be OK?  If so, I'll create and post a webrev.
>>>
>>> -- Jon
>