Re: Build: another hack for translations (fix 1323). (issue2520041)

[Graham Percival]

On Sun, Oct 17, 2010 at 8:22 PM, John Mandereau
<address@hidden> wrote:
> Il giorno dom, 17/10/2010 alle 18.28 +0000, address@hidden
> ha scritto:
>> Are you sure about that?  After disabling the
>> (make the "return to doc index" work) stuff
>
> Do you mean your patch?

I mean, change line 349:
-             if target == 'online':
+             if 0:

> It seems so from the file paths in URLs you
> quote below.  I mean, your patch does a good job, except that given the
> path substitution you add, it is not necessary to add an extra "../" for
> translations, despite the discrepancy between English docs and
> translations in the output without your patch.

The extra "../" is present in the docs, in the output of
postprocess.html, if no attempt is made to rewrite the "return to doc
index".  I don't know if these exist in the docs previously (I suspect
this is the case, though), or if the extra "../" was added somewhere
else in postprocess.html, but it definitely exists.

I'm still working on the updated patch.

> in out-www/online-root/Documentation/learning/index.it.html, so you see
> there is an extra "../" (the doc root is at depth 2 on lilypond.org and
> this page is at depth 2 from doc root); BTW it should also be
> "manuals.it.html", not "development...".  I'm commenting this in on
> Rietveld.

No; if VERSION is unstable, it should point to development, not
manuals.  That's what the int(verstiontup[1] % 2) is doing.

Cheers,
- Graham