Bug 175687 - Number each section in ReadMe.md and add more clarifications
Summary: Number each section in ReadMe.md and add more clarifications
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Perf Dashboard (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryosuke Niwa
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-08-17 14:24 PDT by Ryosuke Niwa
Modified: 2017-08-17 15:36 PDT (History)
6 users (show)

See Also:


Attachments
Updates ReadMe.md (7.47 KB, patch)
2017-08-17 14:27 PDT, Ryosuke Niwa
joepeck: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2017-08-17 14:24:04 PDT
Clarify ReadMe.md to address issues Akash encountered.
Comment 1 Ryosuke Niwa 2017-08-17 14:27:34 PDT
Created attachment 318420 [details]
Updates ReadMe.md
Comment 2 Joseph Pecoraro 2017-08-17 14:47:24 PDT
Comment on attachment 318420 [details]
Updates ReadMe.md

rs=me
Comment 3 Joseph Pecoraro 2017-08-17 14:48:55 PDT
Comment on attachment 318420 [details]
Updates ReadMe.md

View in context: https://bugs.webkit.org/attachment.cgi?id=318420&action=review

> Websites/perf.webkit.org/ReadMe.md:11
> +1. Install Server.app but do NOT launch/open it. (if you don't want to use Server.app, install PostgreSQL: http://www.postgresql.org/download/macosx/)

Why is the NOT so significant here. What goes wrong if it is launched?

> Websites/perf.webkit.org/ReadMe.md:98
> +### Instructions if you've accidentally launched Server.app

Are these changes that Server.app always makes and we are reverting them?
Comment 4 Ryosuke Niwa 2017-08-17 15:02:44 PDT
(In reply to Joseph Pecoraro from comment #3)
> Comment on attachment 318420 [details]
> Updates ReadMe.md
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=318420&action=review
> 
> > Websites/perf.webkit.org/ReadMe.md:11
> > +1. Install Server.app but do NOT launch/open it. (if you don't want to use Server.app, install PostgreSQL: http://www.postgresql.org/download/macosx/)
> 
> Why is the NOT so significant here. What goes wrong if it is launched?

It breaks the setup :(

> > Websites/perf.webkit.org/ReadMe.md:98
> > +### Instructions if you've accidentally launched Server.app
> 
> Are these changes that Server.app always makes and we are reverting them?

No. If you've launched Server.app, then you have to follow a different set of instructions :(
Comment 5 Ryosuke Niwa 2017-08-17 15:34:51 PDT
Landed in https://trac.webkit.org/changeset/220882.
Comment 6 Radar WebKit Bug Importer 2017-08-17 15:36:15 PDT
<rdar://problem/33951679>