RFR: 8023216: Feedback on README-builds.html

Erik Joelsson erik.joelsson at oracle.com
Wed Aug 21 12:53:41 UTC 2013


Thanks, updated webrev:

http://cr.openjdk.java.net/~erikj/8023216/webrev.root.02/

/Erik

On 2013-08-19 16:38, Dalibor Topic wrote:
> On 8/19/13 2:43 PM, Erik Joelsson wrote:
>> And again, here we go:
>>
>> http://cr.openjdk.java.net/~erikj/8023216/webrev.root.01/
> A few typos:
>
> javascript ->  JavaScript
> sourses ->  sources
>
> cheers,
> dalibor topic
>
>
>> /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
>
>



More information about the build-dev mailing list