Re: search-box.ihtml not tracked by check-translation script?

2020-12-18 Thread Federico Bruni

Maybe you're right: a symlink would be better in this case.
I've just pushed other two commits. Let's see if the build works now.


Il giorno ven, dic 18 2020 at 22:39:42 +0100, Jonas Hahnfeld 
 ha scritto:
That would obviously also work, but the last update in 
Documentation/cs

seems to be
commit e593c62a665b8d3a8e095da059595301585c2163
Author: Pavel Fric 
Date:   Thu Aug 30 18:46:50 2012 +0200

Doc-cs. Update of fundamental.itely.

I had already planned to go over the state of translated manuals and
the website early next year, IMHI heavily outdated pages are worse 
than

English content.

Jonas

Am Freitag, dem 18.12.2020 um 22:32 +0100 schrieb Federico Bruni:

 Ok, thank you Jonas!

 There's a third option I like more:

 3) mandatory translation of search-box.ihtml (it's already in the 
list

 of files to be translated in the CG manual)
 I may contact the cs translator to get it translated and/or ask for
 help on lilypond-user.

 Il giorno ven, dic 18 2020 at 22:25:48 +0100, Jonas Hahnfeld
  ha scritto:
 > There's no Documentation/cs/search-box.ihtml, so IIRC lilypond-
 > texi2html.init searches for the fallback in Documentation/. Now we
 > could of course adapt this check, but this feels odd from a 
language

 > perspective. Two more ideas:
 > 1) Adapt the check-translation script
 > 2) Also add a symlink from Documentation/search-box.ihtml to
 > en/search-
 > box.ihtml.
 >
 > WDYT?
 > Jonas
 >
 > Am Freitag, dem 18.12.2020 um 21:05 + schrieb Kevin Barry:
 > >  Hi Federico,
 > >
 > >  I don't know anything about this part of the code, but the 
relevant

 > > errors
 > >  from the doc build pipeline seem to be:
 > >  no such file: search-box.ihtml: No such file or directory at
 > >  
/builds/lilypond/lilypond/Documentation/lilypond-texi2html.init line

 > >  836.
 > >  
/builds/lilypond/lilypond/build/.././Documentation/GNUmakefile:246:

 > >  recipe for target 'out-www/cs/web-big-page.html' failed
 > >
 > >  Does that point you in the right direction?
 > >
 > >  Kevin
 > >
 > >  On Fri, 18 Dec 2020 at 20:27, Federico Bruni 


 > > wrote:
 > >  >
 > >  > I've just submitted a MR:
 > >  > https://gitlab.com/lilypond/lilypond/-/merge_requests/545
 > >  >
 > >  > but it doesn't pass the doc build pipeline.
 > >  >
 > >  > It worked for me on my computer, even though only english 
docs

 > > were
 > >  > built.
 > >  > Basically I just added 'en' to the LY_LANGUAGES variable 
before

 > > the
 > >  > other languages are appended.
 > >  >
 > >  > Any advice is welcomed! :-)
 > >  >
 > >  >
 > >  >
 > >  > Il giorno mer, dic 9 2020 at 00:38:05 +0100, Federico Bruni
 > >  >  ha scritto:
 > >  > > Any reason why search-box.ihtml is in Documentation/ 
instead of

 > >  > > Documentation/en?
 > >  > > It seems it's causing troubles to the check-translation 
script:

 > >  > >
 > >  > > $ cd Documentation
 > >  > > $ make ISOLANG=fr check-translation | grep 'diff --git'
 > >  > > warning: fr/search-box.ihtml: 128 fatal: path
 > >  > > 'Documentation/en/search-box.ihtml' does not exist in 
'HEAD'

 > >  > >
 > >  > >
 > >  > >
 > >  >
 > >  >
 > >  >
 > >
 >










Re: search-box.ihtml not tracked by check-translation script?

2020-12-18 Thread Jonas Hahnfeld via Discussions on LilyPond development
That would obviously also work, but the last update in Documentation/cs
seems to be
commit e593c62a665b8d3a8e095da059595301585c2163
Author: Pavel Fric 
Date:   Thu Aug 30 18:46:50 2012 +0200

Doc-cs. Update of fundamental.itely.

I had already planned to go over the state of translated manuals and
the website early next year, IMHI heavily outdated pages are worse than
English content.

Jonas

Am Freitag, dem 18.12.2020 um 22:32 +0100 schrieb Federico Bruni:
> Ok, thank you Jonas!
> 
> There's a third option I like more:
> 
> 3) mandatory translation of search-box.ihtml (it's already in the list 
> of files to be translated in the CG manual)
> I may contact the cs translator to get it translated and/or ask for 
> help on lilypond-user.
> 
> Il giorno ven, dic 18 2020 at 22:25:48 +0100, Jonas Hahnfeld 
>  ha scritto:
> > There's no Documentation/cs/search-box.ihtml, so IIRC lilypond-
> > texi2html.init searches for the fallback in Documentation/. Now we
> > could of course adapt this check, but this feels odd from a language
> > perspective. Two more ideas:
> > 1) Adapt the check-translation script
> > 2) Also add a symlink from Documentation/search-box.ihtml to 
> > en/search-
> > box.ihtml.
> > 
> > WDYT?
> > Jonas
> > 
> > Am Freitag, dem 18.12.2020 um 21:05 + schrieb Kevin Barry:
> > >  Hi Federico,
> > > 
> > >  I don't know anything about this part of the code, but the relevant 
> > > errors
> > >  from the doc build pipeline seem to be:
> > >  no such file: search-box.ihtml: No such file or directory at
> > >  /builds/lilypond/lilypond/Documentation/lilypond-texi2html.init line
> > >  836.
> > >  /builds/lilypond/lilypond/build/.././Documentation/GNUmakefile:246:
> > >  recipe for target 'out-www/cs/web-big-page.html' failed
> > > 
> > >  Does that point you in the right direction?
> > > 
> > >  Kevin
> > > 
> > >  On Fri, 18 Dec 2020 at 20:27, Federico Bruni  
> > > wrote:
> > >  >
> > >  > I've just submitted a MR:
> > >  > https://gitlab.com/lilypond/lilypond/-/merge_requests/545
> > >  >
> > >  > but it doesn't pass the doc build pipeline.
> > >  >
> > >  > It worked for me on my computer, even though only english docs 
> > > were
> > >  > built.
> > >  > Basically I just added 'en' to the LY_LANGUAGES variable before 
> > > the
> > >  > other languages are appended.
> > >  >
> > >  > Any advice is welcomed! :-)
> > >  >
> > >  >
> > >  >
> > >  > Il giorno mer, dic 9 2020 at 00:38:05 +0100, Federico Bruni
> > >  >  ha scritto:
> > >  > > Any reason why search-box.ihtml is in Documentation/ instead of
> > >  > > Documentation/en?
> > >  > > It seems it's causing troubles to the check-translation script:
> > >  > >
> > >  > > $ cd Documentation
> > >  > > $ make ISOLANG=fr check-translation | grep 'diff --git'
> > >  > > warning: fr/search-box.ihtml: 128 fatal: path
> > >  > > 'Documentation/en/search-box.ihtml' does not exist in 'HEAD'
> > >  > >
> > >  > >
> > >  > >
> > >  >
> > >  >
> > >  >
> > > 
> > 
> 
> 



signature.asc
Description: This is a digitally signed message part


Re: search-box.ihtml not tracked by check-translation script?

2020-12-18 Thread Federico Bruni

Ok, thank you Jonas!

There's a third option I like more:

3) mandatory translation of search-box.ihtml (it's already in the list 
of files to be translated in the CG manual)
I may contact the cs translator to get it translated and/or ask for 
help on lilypond-user.


Il giorno ven, dic 18 2020 at 22:25:48 +0100, Jonas Hahnfeld 
 ha scritto:

There's no Documentation/cs/search-box.ihtml, so IIRC lilypond-
texi2html.init searches for the fallback in Documentation/. Now we
could of course adapt this check, but this feels odd from a language
perspective. Two more ideas:
1) Adapt the check-translation script
2) Also add a symlink from Documentation/search-box.ihtml to 
en/search-

box.ihtml.

WDYT?
Jonas

Am Freitag, dem 18.12.2020 um 21:05 + schrieb Kevin Barry:

 Hi Federico,

 I don't know anything about this part of the code, but the relevant 
errors

 from the doc build pipeline seem to be:
 no such file: search-box.ihtml: No such file or directory at
 /builds/lilypond/lilypond/Documentation/lilypond-texi2html.init line
 836.
 /builds/lilypond/lilypond/build/.././Documentation/GNUmakefile:246:
 recipe for target 'out-www/cs/web-big-page.html' failed

 Does that point you in the right direction?

 Kevin

 On Fri, 18 Dec 2020 at 20:27, Federico Bruni  
wrote:

 >
 > I've just submitted a MR:
 > https://gitlab.com/lilypond/lilypond/-/merge_requests/545
 >
 > but it doesn't pass the doc build pipeline.
 >
 > It worked for me on my computer, even though only english docs 
were

 > built.
 > Basically I just added 'en' to the LY_LANGUAGES variable before 
the

 > other languages are appended.
 >
 > Any advice is welcomed! :-)
 >
 >
 >
 > Il giorno mer, dic 9 2020 at 00:38:05 +0100, Federico Bruni
 >  ha scritto:
 > > Any reason why search-box.ihtml is in Documentation/ instead of
 > > Documentation/en?
 > > It seems it's causing troubles to the check-translation script:
 > >
 > > $ cd Documentation
 > > $ make ISOLANG=fr check-translation | grep 'diff --git'
 > > warning: fr/search-box.ihtml: 128 fatal: path
 > > 'Documentation/en/search-box.ihtml' does not exist in 'HEAD'
 > >
 > >
 > >
 >
 >
 >









Re: search-box.ihtml not tracked by check-translation script?

2020-12-18 Thread Jonas Hahnfeld
There's no Documentation/cs/search-box.ihtml, so IIRC lilypond-
texi2html.init searches for the fallback in Documentation/. Now we
could of course adapt this check, but this feels odd from a language
perspective. Two more ideas:
1) Adapt the check-translation script
2) Also add a symlink from Documentation/search-box.ihtml to en/search-
box.ihtml.

WDYT?
Jonas

Am Freitag, dem 18.12.2020 um 21:05 + schrieb Kevin Barry:
> Hi Federico,
> 
> I don't know anything about this part of the code, but the relevant errors
> from the doc build pipeline seem to be:
> no such file: search-box.ihtml: No such file or directory at
> /builds/lilypond/lilypond/Documentation/lilypond-texi2html.init line
> 836.
> /builds/lilypond/lilypond/build/.././Documentation/GNUmakefile:246:
> recipe for target 'out-www/cs/web-big-page.html' failed
> 
> Does that point you in the right direction?
> 
> Kevin
> 
> On Fri, 18 Dec 2020 at 20:27, Federico Bruni  wrote:
> > 
> > I've just submitted a MR:
> > https://gitlab.com/lilypond/lilypond/-/merge_requests/545
> > 
> > but it doesn't pass the doc build pipeline.
> > 
> > It worked for me on my computer, even though only english docs were
> > built.
> > Basically I just added 'en' to the LY_LANGUAGES variable before the
> > other languages are appended.
> > 
> > Any advice is welcomed! :-)
> > 
> > 
> > 
> > Il giorno mer, dic 9 2020 at 00:38:05 +0100, Federico Bruni
> >  ha scritto:
> > > Any reason why search-box.ihtml is in Documentation/ instead of
> > > Documentation/en?
> > > It seems it's causing troubles to the check-translation script:
> > > 
> > > $ cd Documentation
> > > $ make ISOLANG=fr check-translation | grep 'diff --git'
> > > warning: fr/search-box.ihtml: 128 fatal: path
> > > 'Documentation/en/search-box.ihtml' does not exist in 'HEAD'
> > > 
> > > 
> > > 
> > 
> > 
> > 
> 



signature.asc
Description: This is a digitally signed message part


Re: search-box.ihtml not tracked by check-translation script?

2020-12-18 Thread Kevin Barry
Hi Federico,

I don't know anything about this part of the code, but the relevant errors
from the doc build pipeline seem to be:
no such file: search-box.ihtml: No such file or directory at
/builds/lilypond/lilypond/Documentation/lilypond-texi2html.init line
836.
/builds/lilypond/lilypond/build/.././Documentation/GNUmakefile:246:
recipe for target 'out-www/cs/web-big-page.html' failed

Does that point you in the right direction?

Kevin

On Fri, 18 Dec 2020 at 20:27, Federico Bruni  wrote:
>
> I've just submitted a MR:
> https://gitlab.com/lilypond/lilypond/-/merge_requests/545
>
> but it doesn't pass the doc build pipeline.
>
> It worked for me on my computer, even though only english docs were
> built.
> Basically I just added 'en' to the LY_LANGUAGES variable before the
> other languages are appended.
>
> Any advice is welcomed! :-)
>
>
>
> Il giorno mer, dic 9 2020 at 00:38:05 +0100, Federico Bruni
>  ha scritto:
> > Any reason why search-box.ihtml is in Documentation/ instead of
> > Documentation/en?
> > It seems it's causing troubles to the check-translation script:
> >
> > $ cd Documentation
> > $ make ISOLANG=fr check-translation | grep 'diff --git'
> > warning: fr/search-box.ihtml: 128 fatal: path
> > 'Documentation/en/search-box.ihtml' does not exist in 'HEAD'
> >
> >
> >
>
>
>



Re: search-box.ihtml not tracked by check-translation script?

2020-12-18 Thread Dan Eble
On Dec 18, 2020, at 15:26, Federico Bruni  wrote:
> 
> I've just submitted a MR:
> https://gitlab.com/lilypond/lilypond/-/merge_requests/545
> 
> but it doesn't pass the doc build pipeline.
> 
> It worked for me on my computer, even though only english docs were built.
> Basically I just added 'en' to the LY_LANGUAGES variable before the other 
> languages are appended.
> 
> Any advice is welcomed! :-)

I applied your patch over master and I hit the same problem as the CI when I 
ran make -j7 CPU_COUNT=7 doc.
— 
Dan




Re: search-box.ihtml not tracked by check-translation script?

2020-12-18 Thread Federico Bruni

I've just submitted a MR:
https://gitlab.com/lilypond/lilypond/-/merge_requests/545

but it doesn't pass the doc build pipeline.

It worked for me on my computer, even though only english docs were 
built.
Basically I just added 'en' to the LY_LANGUAGES variable before the 
other languages are appended.


Any advice is welcomed! :-)



Il giorno mer, dic 9 2020 at 00:38:05 +0100, Federico Bruni 
 ha scritto:
Any reason why search-box.ihtml is in Documentation/ instead of 
Documentation/en?

It seems it's causing troubles to the check-translation script:

$ cd Documentation
$ make ISOLANG=fr check-translation | grep 'diff --git'
warning: fr/search-box.ihtml: 128 fatal: path 
'Documentation/en/search-box.ihtml' does not exist in 'HEAD'