Adds version number to web page side bar (Issue 3367)

Adds version number to web page side bar (Issue 3367)

[PhilEHolmes, 2013-06-24 16:32:25]

Please review

[Graham Percival, 2013-06-27 22:55:42]

LGTM

[Mark Polesky, 2013-06-30 05:48:26]

https://codereview.appspot.com/10506043/diff/1/python/auxiliar/postprocess_ht...
File python/auxiliar/postprocess_html.py (right):

https://codereview.appspot.com/10506043/diff/1/python/auxiliar/postprocess_ht...
python/auxiliar/postprocess_html.py:60: sidebar_version = _doc ('
V%(package_version)s (%(branch_str)s).')
Personally I prefer a lower-case "v" -- it's somehow easier to read.

https://codereview.appspot.com/10506043/diff/1/python/auxiliar/postprocess_ht...
python/auxiliar/postprocess_html.py:367: 
omit whitespace

[Mark Polesky, 2013-06-30 06:25:36]

Also, I don't have time to compile right now, but I don't suppose it also prints
a version stamp in the older docs, like 2.12 and 2.14?  Otherwise it doesn't
really satisfy the original request on the tracker -
http://code.google.com/p/lilypond/issues/detail?id=3367 .  Personally, I think
the real way to solve this problem is by fixing the issue I raised here:
http://lists.gnu.org/archive/html/bug-lilypond/2013-06/msg00108.html .

- Mark

[mail_philholmes.net, 2013-06-30 10:03:55]

----- Original Message ----- 
From: <markpolesky@gmail.com>
To: <PhilEHolmes@googlemail.com>; <graham@percival-music.ca>
Cc: <reply@codereview-hr.appspotmail.com>; <lilypond-devel@gnu.org>
Sent: Sunday, June 30, 2013 7:25 AM
Subject: Re: Adds version number to web page side bar (Issue 3367) 
(issue10506043)


> Also, I don't have time to compile right now, but I don't suppose it
> also prints a version stamp in the older docs, like 2.12 and 2.14?


Obviously not.  They're history and are never updated.  However, not doing 
this will mean that 2.17 won't get a version number, nor 2.18, nor 2.19, 
etc.,etc.  Doing it means they will and is therefore an improvement.  Let's 
not stop an improvement simply because it's not the Ultimate Solution (TM).

> Otherwise it doesn't really satisfy the original request on the tracker
> - http://code.google.com/p/lilypond/issues/detail?id=3367 .  Personally,
> I think the real way to solve this problem is by fixing the issue I
> raised here:
> http://lists.gnu.org/archive/html/bug-lilypond/2013-06/msg00108.html .
>
> - Mark


--
Phil Holmes 

[PhilEHolmes, 2013-07-02 09:52:00]

Updated following comments

[PhilEHolmes, 2013-07-02 09:55:31]

Further upload to get issue number correct