Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(15773)

Issue 6454114: Linux->GNU/Linux (Issue 2651) (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 8 months ago by PhilEHolmes
Modified:
11 years, 8 months ago
Reviewers:
Graham Percival, dak
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Updates all the instances of linux that I can find in the English docs to GNU/Linux.

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -17 lines) Patch
M Documentation/contributor/administration.itexi View 1 chunk +1 line, -1 line 0 comments Download
M Documentation/contributor/programming-work.itexi View 1 chunk +1 line, -1 line 0 comments Download
M Documentation/contributor/quick-start.itexi View 2 chunks +3 lines, -3 lines 0 comments Download
M Documentation/contributor/source-code.itexi View 1 chunk +1 line, -1 line 0 comments Download
M Documentation/learning/tweaks.itely View 1 chunk +2 lines, -2 lines 0 comments Download
M Documentation/usage/running.itely View 1 chunk +1 line, -1 line 0 comments Download
M Documentation/usage/suggestions.itely View 1 chunk +2 lines, -2 lines 0 comments Download
M Documentation/web/download.itexi View 1 chunk +1 line, -1 line 0 comments Download
M Documentation/web/introduction.itexi View 1 chunk +1 line, -1 line 0 comments Download
M flower/libc-extension.cc View 1 chunk +1 line, -1 line 1 comment Download
M scripts/build/create-weblinks-itexi.py View 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 2
PhilEHolmes
Please review.
11 years, 8 months ago (2012-08-07 13:56:25 UTC) #1
Graham Percival
11 years, 8 months ago (2012-08-07 14:03:20 UTC) #2
LGTM, one thing to fix then please push directly to staging.

http://codereview.appspot.com/6454114/diff/1/flower/libc-extension.cc
File flower/libc-extension.cc (right):

http://codereview.appspot.com/6454114/diff/1/flower/libc-extension.cc#newcode58
flower/libc-extension.cc:58: The prototype is not in accordance with the
GNU/Linux Programmer's
"Linux Programmer's manual" is a title of a book, so adding GNU/ is not
appropriate in this case.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b