Build: another hack for translations (fix 1323).

Build: another hack for translations (fix 1323).

[John Mandereau, 2010-10-17 16:52:03]

Besides the niptick, it looks good to me.  I tested by building both offline and
online targets.

http://codereview.appspot.com/2520041/diff/1/python/auxiliar/postprocess_html.py
File python/auxiliar/postprocess_html.py (right):

http://codereview.appspot.com/2520041/diff/1/python/auxiliar/postprocess_html...
python/auxiliar/postprocess_html.py:357: extra_depth = '../'
'../' is is not needed for translations, as output web pages are at the same
depth as the pages in English, and may cause errors in case server configuration
doesn't redirect /.. to /. I suggest to simply remove all extra_depth stuff.

[Graham Percival (old account), 2010-10-17 18:28:50]

On 2010/10/17 16:52:03, John Mandereau wrote:
> python/auxiliar/postprocess_html.py:357: extra_depth = '../'
> '../' is is not needed for translations, as output web pages are at the same
> depth as the pages in English, and may cause errors in case server
configuration
> doesn't redirect /.. to /. I suggest to simply remove all extra_depth stuff.

Are you sure about that?  After disabling the 
(make the "return to doc index" work) stuff and building the online docs, I see
this:

**** notation/index.es.html
<p class="toc_uplink"><a href="../../..//Documentation/web/manuals.es.html"
         title="Documentation Index">&lt;&lt; Volver al índice de la
documentación</a></p>

**** notation/index.html
<p class="toc_uplink"><a href="../..//Documentation/web/manuals.html"
         title="Documentation Index">&lt;&lt; Back to Documentation
Index</a></p>


so it seems to me that we need the extra_depth for the old_link, at least.  Now,
I'm willing to believe that we don't need it for the devel_link and manual_link
(i.e. the new links)... those are much more awkward to test, so I haven't done
that yet.

oh wait -- the big-html manuals are different:
p class="toc_uplink"><a href="../../Documentation/web/manuals.es.html"
         title="Documentation Index">&lt;&lt; Volver al índice de la
documentación</a></p>

<p class="toc_uplink"><a href="../Documentation/web/manuals"
         title="Documentation Index">&lt;&lt; Back to Documentation
Index</a></p>


hmm... I'll try making a new patch, which only replaces for the
Documentation/web/manuals part.

[Graham Percival (old account), 2010-10-17 21:14:11]

Here's another version.  Note that the "extra depth" thing is now added to the
*non*-translated version.

I have no clue why.  I've spent 3 hours bashing against the build process, and
that's enough for me.
  CG 5.3 Debugging website and docs locally
has what I know about trying to solve this stuff.

Along the way, I've found that firefox is profoundly unhappy with "..//..". 
That may have prompted me to add the extra ../ to translations initially.  Or
maybe there's still something that's messed up.

This could really use a fresh pair of eyes to look at it.  The current patch
works for me, given the instructions in CG 5.3.  That's all I can say right now.

[John Mandereau, 2010-10-17 22:40:05]

On 2010/10/17 21:14:11, Graham Percival wrote:
> Here's another version.  Note that the "extra depth" thing is now added to the
> *non*-translated version.
> 
> I have no clue why.  I've spent 3 hours bashing against the build process, and
> that's enough for me.
>   CG 5.3 Debugging website and docs locally
> has what I know about trying to solve this stuff.
> 
> Along the way, I've found that firefox is profoundly unhappy with "..//..". 
> That may have prompted me to add the extra ../ to translations initially.  Or
> maybe there's still something that's messed up.

This new patch isn't good, not because of the double-slash, but because it
breaks offline docs "Back to..." link.

The discrepancy between English docs and translations comes from a bug in
Texi2HTML init file, as the relative URL is naively taken using depth. I'll look
at it after a few hours of sleeping.

Best,
John

[Graham Percival, 2010-10-17 22:45:25]

On Sun, Oct 17, 2010 at 10:40:05PM +0000, john.mandereau@gmail.com wrote:
> On 2010/10/17 21:14:11, Graham Percival wrote:
> >Along the way, I've found that firefox is profoundly unhappy with
> "..//..".
> >That may have prompted me to add the extra ../ to translations
> initially.  Or
> >maybe there's still something that's messed up.
> 
> This new patch isn't good, not because of the double-slash, but because
> it breaks offline docs "Back to..." link.

Eh?  The patch only does stuff for the online target.  That's what
line 349 is supposed to do:
            if target == 'online':

If the patch breaks offline docs, then something is /really/
messed up.  Did my 819365f311a832c5cc57f086016069efdbee91fa break
the offline docs?

Cheers,
- Graham