RFR: 8023216: Feedback on README-builds.html

[Anthony Petrov]

Hi Erik,

The fix looks fine to me.

--
best regards,
Anthony

On 08/19/13 16:43, Erik Joelsson wrote:
> And again, here we go:
>
> http://cr.openjdk.java.net/~erikj/8023216/webrev.root.01/
>
> /Erik
>
> On 2013-08-19 11:05, Erik Joelsson wrote:
>> Thanks for the feedback!
>>
>> I took most of it and made into a change. Also removed some obsolete
>> parts that I stumbled over.
>>
>> Warn is supposed to be quiet, but at least hotspot is still being
>> quite noisy.
>>
>> /Erik
>>
>> On 2013-08-06 01:18, Dan Smith wrote:
>>> I'm building for the first time under the new infrastructure (late to
>>> the party, I know). I appreciate the comprehensive readme page.
>>> Here's a list of minor points of feedback:
>>>
>>> "The set of repositories and what they contain": need to add nashorn
>>>
>>> "warn — Default and very quiet": I presume this is still an
>>> in-progress goal? I'm seeing almost 3000 lines of text in a fresh
>>> build (including over 1000 from 'images')
>>>
>>> ---
>>>
>>> Some copy editing:
>>>
>>> "each repository is it's own independent repository": it's -> its
>>>
>>> "Insure that GNU make": insure -> ensure
>>>
>>> "rather thane excludes": thane -> than
>>>
>>> "Other depends on pieces of code": other depends -> others depend
>>>
>>> "for all classes that contains constants": contains -> contain
>>>
>>> "we have missed to add support": missed -> neglected