Re: [Tails-dev] language_statistics.sh script broken?

[intrigeri]

Hi,

Thank you both for the debugging!

Cyril Brulebois:
> It might be worth supporting an option (or environment variable) in that
> script to avoid cleaning tmp/pot when it's called with this extra
> setting from the language_statistics.sh script?

Absolutely. Done: https://labs.riseup.net/code/issues/15165

Cyril, do you want to review my branch? If not, I'll ask the RM for
3.5 (anonym) to do it.

Cheers,
-- 
intrigeri
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.

Re: [Tails-dev] language_statistics.sh script broken?

[Cyril Brulebois]

Hi Muri & list,

Muri Nicanor  (2017-12-11):
> thanks! if i comment out the line the script works with the current
> commit, but if i go back to --before="December 1" (8d60cc4596) in only
> get the first part of the output of the script (## All the website) but
> for the core pages it breaks with:
> > msgcat: error while opening
> "wiki/src/./misc/unsafe_browser_warning.de.po" for reading: No such file
> or directory

Yeah, it seems you might need to adjust a few files to let you compute
stats for the past. See this commit, which brought the list of core
files in line with recent changes:
| commit 791437c88902e65b5425036cc119a3baf2d059f2
| Author: sajolida 
| Date:   Thu Nov 16 18:13:54 2017 +
| 
| Update core pages and .htaccess to moved and removed files for the new 
download page

That includes getting rid of “./misc/unsafe_browser_warning” in this
file: wiki/src/contribute/l10n_tricks/core_po_files.txt

I'm attaching the full diff (trap commented out in addition to the
change mentioned above) that lets me compute stats for the commit you
mentioned (8d60cc4596).

Cheers,
-- 
Cyril Brulebois -- Debian Consultant @ DEBAMAX -- https://debamax.com/
diff --git a/refresh-translations b/refresh-translations
index 740e9fe671..df781b038e 100755
--- a/refresh-translations
+++ b/refresh-translations
@@ -135,7 +135,7 @@ intltool_merge_xml () {
 ### Main
 
 # Schedule clean up
-trap "rm -fr tmp/pot po/*.new po/*.orig" EXIT
+# trap "rm -fr tmp/pot po/*.new po/*.orig" EXIT
 
 FORCE=no
 while [ -n "${@:-}" ]; do
diff --git a/wiki/src/contribute/l10n_tricks/core_po_files.txt b/wiki/src/contribute/l10n_tricks/core_po_files.txt
index b1000eb354..915d0b6144 100644
--- a/wiki/src/contribute/l10n_tricks/core_po_files.txt
+++ b/wiki/src/contribute/l10n_tricks/core_po_files.txt
@@ -86,7 +86,6 @@
 ./install/win
 ./install/win/usb
 ./install/win/usb/overview
-./misc/unsafe_browser_warning
 ./sidebar
 ./support
 ./upgrade


signature.asc
Description: PGP signature
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.

Re: [Tails-dev] language_statistics.sh script broken?

[Muri Nicanor]

Hi Cyril and list,

On 12/10/2017 07:42 PM, Cyril Brulebois wrote:
> This script seems to rely on ./refresh-translations, which used to leave
> files under tmp/pot. Starting with the commit below, files are cleaned
> up in every situation, including clean exit; this explains why attempts
> with recent dates fail, while going back earlier in time makes this
> succeed again.
> 
> | commit d199d512d5cf64782a159176cf99412d332d40ac
> | Author: anonym 
> | Date:   Thu Sep 7 18:58:48 2017 +0200
> | 
> | Do some clean up.
> | 
> | If the script fails at various points, these files would linger.
> 
> If you want to get a one-time report, commenting out this line seems
> sufficient:
> | trap "rm -fr tmp/pot po/*.new po/*.orig" EXIT

thanks! if i comment out the line the script works with the current
commit, but if i go back to --before="December 1" (8d60cc4596) in only
get the first part of the output of the script (## All the website) but
for the core pages it breaks with:
> msgcat: error while opening
"wiki/src/./misc/unsafe_browser_warning.de.po" for reading: No such file
or directory

cheers,
muri



signature.asc
Description: OpenPGP digital signature
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.

Re: [Tails-dev] language_statistics.sh script broken?

[Cyril Brulebois]

Hi Muri,

Muri Nicanor  (2017-12-10):
> when trying to run the
> ./wiki/src/contribute/l10n_tricks/language_statistics.sh script to get
> the statistics for the monthly report, it fails with:
> > xgettext: error while opening "../tmp/pot/60-tor-ready.sh.pot" for
> > reading: No such file or directory
> > ERROR: xgettext failed to generate PO template file. Please consult
> >   error message above if there is any.
> 
> this happens when i checkout --before="December 1" or --before="November
> 1", but apparently it worked for the last report- so does anyone have an
> idea? (i'm running buster, but i've also tried in a jessie chroot...)

This script seems to rely on ./refresh-translations, which used to leave
files under tmp/pot. Starting with the commit below, files are cleaned
up in every situation, including clean exit; this explains why attempts
with recent dates fail, while going back earlier in time makes this
succeed again.

| commit d199d512d5cf64782a159176cf99412d332d40ac
| Author: anonym 
| Date:   Thu Sep 7 18:58:48 2017 +0200
| 
| Do some clean up.
| 
| If the script fails at various points, these files would linger.

If you want to get a one-time report, commenting out this line seems
sufficient:
| trap "rm -fr tmp/pot po/*.new po/*.orig" EXIT

It might be worth supporting an option (or environment variable) in that
script to avoid cleaning tmp/pot when it's called with this extra
setting from the language_statistics.sh script?


Cheers,
-- 
Cyril Brulebois -- Debian Consultant @ DEBAMAX -- https://debamax.com/


signature.asc
Description: PGP signature
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.

[Tails-dev] language_statistics.sh script broken?

[Muri Nicanor]

hi,

when trying to run the
./wiki/src/contribute/l10n_tricks/language_statistics.sh script to get
the statistics for the monthly report, it fails with:
> xgettext: error while opening "../tmp/pot/60-tor-ready.sh.pot" for
> reading: No such file or directory
> ERROR: xgettext failed to generate PO template file. Please consult
>   error message above if there is any.

this happens when i checkout --before="December 1" or --before="November
1", but apparently it worked for the last report- so does anyone have an
idea? (i'm running buster, but i've also tried in a jessie chroot...)

cheers,
muri



signature.asc
Description: OpenPGP digital signature
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.