lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Doc: Correct and extend infos about LilyDev setup (issue 561360043 b


From: lemzwerg
Subject: Re: Doc: Correct and extend infos about LilyDev setup (issue 561360043 by address@hidden)
Date: Thu, 30 Jan 2020 02:12:13 -0800

Next round of nits :-)


https://codereview.appspot.com/561360043/diff/575590043/Documentation/contributor/quick-start.itexi
File Documentation/contributor/quick-start.itexi (left):

https://codereview.appspot.com/561360043/diff/575590043/Documentation/contributor/quick-start.itexi#oldcode103
Documentation/contributor/quick-start.itexi:103: specific option choose
@q{Linux 2.6/3.x/4.x (64-bit)}.
Is it really '64 bit' in one case and '64-bit' in the other case?

https://codereview.appspot.com/561360043/diff/575590043/Documentation/contributor/quick-start.itexi
File Documentation/contributor/quick-start.itexi (right):

https://codereview.appspot.com/561360043/diff/575590043/Documentation/contributor/quick-start.itexi#newcode46
Documentation/contributor/quick-start.itexi:46: create it from the
sources located in the @code{/mkosi} subdirectory
s/@code/@file/.

https://codereview.appspot.com/561360043/diff/575590043/Documentation/contributor/quick-start.itexi#newcode97
Documentation/contributor/quick-start.itexi:97: On Linux, run the
following command in the directory where you've
s/you've/you have/

https://codereview.appspot.com/561360043/diff/575590043/Documentation/contributor/quick-start.itexi#newcode108
Documentation/contributor/quick-start.itexi:108: As VirtualBox does not
support the raw format, you'll have to
The GNU documentation guidelines recommend to not use the future tense
if possible.

  s/you'll have/you have/

Please change similar occurrences of future tense, too.

https://codereview.appspot.com/561360043/diff/575590043/Documentation/contributor/quick-start.itexi#newcode118
Documentation/contributor/quick-start.itexi:118: @warning{You will need
a fair amount of disk space (around 30 GB) to
30@tie{}GB

https://codereview.appspot.com/561360043/diff/575590043/Documentation/contributor/quick-start.itexi#newcode176
Documentation/contributor/quick-start.itexi:176: layout, like e.g.
@q{lilzpond} on a German keyboard.}
There is an eternal battle whether there is a comma or not after 'e.g.'
(in the LilyPond documentation, we *do* use a comma).  However, I would
reformulate this to

  ... like @q{lilzpond} on a German keyboard, for example.

https://codereview.appspot.com/561360043/diff/575590043/Documentation/contributor/quick-start.itexi#newcode202
Documentation/contributor/quick-start.itexi:202: rights temporarily.  It
will show you a warning message on it's first
s/it's/its/

https://codereview.appspot.com/561360043/diff/575590043/Documentation/contributor/quick-start.itexi#newcode217
Documentation/contributor/quick-start.itexi:217: To set up your system
language(charset, localized messages etc.),
s/To set up your system language(charset, localized messages etc.),/
  To set up your system language (charset, localized messages, etc.),/

https://codereview.appspot.com/561360043/diff/575590043/Documentation/contributor/quick-start.itexi#newcode229
Documentation/contributor/quick-start.itexi:229: Finally, you should run
a setup script.  If you're on the command line
The GNU documentation guidelines recommend to not use contractions, so

  s/you're/you are/

https://codereview.appspot.com/561360043/diff/575590043/Documentation/contributor/quick-start.itexi#newcode231
Documentation/contributor/quick-start.itexi:231: which will set up git
and download all the repositories needed to build
s/which/that/

https://codereview.appspot.com/561360043/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]