Langtools resources

[Jonathan Gibbons]

Sounds like you have it under control. You will probably get integrated 
to the master before TL.

-- Jon

On 11/27/2012 08:38 AM, Erik Joelsson wrote:
> 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
>>