[LUDOC-263] wrapped text lines in xml make review in gerrit easier.

• Status: Open
• Project: Lustre Documentation
• Type: Bug
• Priority: Major
• Reporter: Richard Henwood (Inactive)
• Assignee: Lustre Manual Triage
• Resolution: Unresolved
• Severity: 3

Comments

[Richard Henwood (Inactive)]

I've spent a bit of time looking into tools to wrap xml. The best I found is 'tidy'. It has a problem, however, which is that in xml mode, --new-pre-tags are ignored. What this means, is that <screen> and <programlisting> tags, which you typically would want to avoid being parsed and wrapped, get parsed and wrapped.

So you need to go and manually check the <screen> and <programlisting>s after you tidy. For an example of a 'tidy'ied file see here:
http://review.whamcloud.com/#/c/11938/

This was generated with the command below, and then <screen> tags were manually reviewed.

cat ./LustreTuning.xml | ~/tools/tidy/bin/tidy --new-pre-tags screen,programlisting --wrap-script-literals no -utf8 -xml -w 80 -i -c -asxml | vi -

[Richard Henwood (Inactive)]

oh, I've also noticed that spaces are remove after 'literal' tags - thanks to the tidy command above. One quick fix is:

:%s/\/literal>\([a-zA-Z]\)/\/literal> \1/gc

[Gerrit Updater]

Delete comment from wrong patch.

[Gerrit Updater]

Richard Henwood ([e-mail]) merged in patch http://review.whamcloud.com/12966/
Subject: LUDOC-263 lfsck: review and update LFSCK documentation.
Project: doc/manual
Branch: master
Current Patch Set:
Commit: 6c441812758e47b4698086173b583e3d6bd051b0

[Gerrit Updater]

Richard Henwood ([e-mail]) uploaded a new patch: http://review.whamcloud.com/17393
Subject: LUDOC-263 wrap: wrap lines for review convenience.
Project: doc/manual
Branch: master
Current Patch Set: 1
Commit: 9f3a6c3c6f46cea90247b0ab8844c22c9d639f5b

[Gerrit Updater]

Richard Henwood ([e-mail]) uploaded a new patch: http://review.whamcloud.com/17398
Subject: LUDOC-263 wrap: wrap lines for review convenience.
Project: doc/manual
Branch: master
Current Patch Set: 1
Commit: bb1fa93236e752a1c7f038292163f3a610711eca

[Gerrit Updater]

Richard Henwood ([e-mail]) uploaded a new patch: http://review.whamcloud.com/17410
Subject: LUDOC-263 wrap: wrap lines for review convenience.
Project: doc/manual
Branch: master
Current Patch Set: 1
Commit: ca8564f78226c3b5d68edd768ee96ca48401818a

[Gerrit Updater]

Richard Henwood ([e-mail]) merged in patch http://review.whamcloud.com/17410/
Subject: LUDOC-263 wrap: wrap lines for review convenience.
Project: doc/manual
Branch: master
Current Patch Set:
Commit: 516fe708f75b8f6fe1d9cc8a21be6888acae5b21

[Gerrit Updater]

Richard Henwood ([e-mail]) uploaded a new patch: http://review.whamcloud.com/17485
Subject: LUDOC-263 wrap: wrap lines for review convenience.
Project: doc/manual
Branch: master
Current Patch Set: 1
Commit: b774f9044a5455cdd2a0010ad435adf8f4c94407

[Gerrit Updater]

Richard Henwood ([e-mail]) merged in patch http://review.whamcloud.com/17485/
Subject: LUDOC-263 wrap: wrap lines for review convenience.
Project: doc/manual
Branch: master
Current Patch Set:
Commit: bda9d624b241ed373b0909d99a0361324ba5aaab

[Gerrit Updater]

Richard Henwood ([e-mail]) uploaded a new patch: http://review.whamcloud.com/17530
Subject: LUDOC-263 wrap: wrap lines for review convenience.
Project: doc/manual
Branch: master
Current Patch Set: 1
Commit: 6fc53417a399acd665a312327b805f485ba20088

[Gerrit Updater]

Richard Henwood ([e-mail]) merged in patch http://review.whamcloud.com/17398/
Subject: LUDOC-263 wrap: wrap lines for review convenience.
Project: doc/manual
Branch: master
Current Patch Set:
Commit: f6e8b0623e4e3188b3c91db22436695ebded08f8

[Gerrit Updater]

Richard Henwood ([e-mail]) merged in patch http://review.whamcloud.com/17393/
Subject: LUDOC-263 wrap: wrap lines for review convenience.
Project: doc/manual
Branch: master
Current Patch Set:
Commit: bfc3782667e73a23f28a7b9cac0df416e9b575b0

[Gerrit Updater]

Richard Henwood ([e-mail]) merged in patch http://review.whamcloud.com/17530/
Subject: LUDOC-263 wrap: wrap lines for review convenience.
Project: doc/manual
Branch: master
Current Patch Set:
Commit: 93cda492417285364795cb14506a4cc6259c6700