- 26 May, 2022 1 commit
-
- 25 May, 2022 1 commit
-
-
intrigeri authored
This error breaks CI with PO files generated by recent po4a. I've asked i18nspector upstream to fix this: https://github.com/jwilk/i18nspector/issues/11 I'll create an issue to revert this once this is fixed in a version of i18nspector that we can install everywhere relevant.
-
- 06 Jun, 2021 1 commit
-
-
intrigeri authored
We don't use them.
-
- 12 Apr, 2020 1 commit
-
-
intrigeri authored
-
- 19 Mar, 2020 4 commits
-
-
Cyril Brulebois authored
This wasn't tidied up in earlier commits to make sure the revert (in the previous commit) would apply cleanly, but there's of course no need to keep creating this temporary directory. [ Due to indentation changes, use -b/-w to double check this commit. ]
-
Cyril Brulebois authored
This reverts commit c957b5b8. With the new approach (implemented two commits ago), a cache directory is created for each i18nspector call. This means that we don't need to treat the first file specifically (which wasn't working, depending on which file that was). As a result, the code can be simplified, getting back to the previous “treat all files the same way” implementation.
-
Cyril Brulebois authored
Let's clean things up a little: The previous commit was an incremental evolution, creating individual cache directories below the previously global temporary directory. Even if that works fine, that's a little convoluted, and there's no reason for this indirection: each i18nspector call can get its own temporary directory anywhere, so simplify how the temporary directories are created.
-
Cyril Brulebois authored
Treating the first PO file as special was a nice try but some files don't trigger the creation of a parser, i.e. don't result in the creation of a rply cache directory and/or cache file. That's the case if the first file is wiki/src/getting_started.fr.po for example, which results in various failure modes, depending on who wins the race: - mkdir on the just-created rply directory; - trying to read from the just-created and still-empty cache file, which is not valid JSON-wise; - trying to read from a being-written cache file, leading to inconsistent data, meaning another JSON validity problem. Therefore, consider the XDG_CACHE_HOME environment variable set at the top-level to be the top-level directory, below which a specific subdirectory will be created for each i18nspector call. This should ensure nobody steps on someone else's toes. This is just an attempt at a minimal bug fix, follow-up commits will simplify that code drastically.
-
- 02 Jan, 2020 1 commit
-
-
intrigeri authored
When having "lint_po --cached" in one's pre-commit hook, any attempt to create a commit that does not modify any PO file fails with: Traceback (most recent call last): File "/home/intrigeri/.home/prv-user/.combined/cyber/amnesia/live/git/submodules/jenkins-tools/slaves/lint_po", line 347, in <module> main(logging.getLogger()) File "/home/intrigeri/.home/prv-user/.combined/cyber/amnesia/live/git/submodules/jenkins-tools/slaves/lint_po", line 327, in main chain = [(_check_po_file(next(iterator)),), StopIteration The po files you're trying to commit contain errors. Please fix them and try again. This is a regression introduced by c957b5b8, which b37a38ce fixed already, but only for the case when --cached is *not* passed. Note that in --cached mode, we don't display a warning: for details, see 0da94c64.
-
- 20 Dec, 2019 5 commits
-
-
Sandro Knauß authored
Because the plain yaml.load could be easily exploited. Please read https://msg.pyyaml.org/load for full details.
-
Sandro Knauß authored
-
Sandro Knauß authored
When there are no files to process on, we should stop the script directly. So other parts of the script can expect that there are files to operate on. With the last commit, this is needed, as we execute next(iterator), that fails if there is no element in the iterator.
-
Sandro Knauß authored
As we run check_po for ten files in parallel, we end up in a race condition about the creation of $XDG_CACHE_HOME/rply. That's why we check the first file before checking the rest (#17359#note-14).
-
Sandro Knauß authored
-
- 18 Dec, 2019 1 commit
-
-
Sandro Knauß authored
Without setting XDG_CACHE_HOME to a temporally directory, we end up using very old ~/.cache/rply.
-
- 18 Sep, 2019 1 commit
-
-
Sandro Knauß authored
-
- 12 Sep, 2019 5 commits
-
-
intrigeri authored
I understand this error happens because polib has no type hints: https://mypy.readthedocs.io/en/latest/running_mypy.html#missing-imports
-
intrigeri authored
-
intrigeri authored
The ":(" smiley unnecessarily conveys bad feelings. Last time I checked, programs have no feelings :) Besides, fix typography and make every line of the error message not longer than 80 chars.
-
intrigeri authored
Most commits we create don't modify any PO file. It's annoying and confusing to be told "WARNING: no file to process :(" every time this happens: one may wonder if they've done something wrong, which is generally not the case.
-
intrigeri authored
-
- 26 Jun, 2019 5 commits
-
-
intrigeri authored
Thanks in advance.
-
intrigeri authored
i18nspector's --language option takes "a 2- or 3-letter ISO 639 language code, possibly followed by underscore and a 2-letter ISO 3166 territory code" so we need to strip any "_<SCRIPT>" suffix when passing it the language we've detected from the PO file name. Otherwise, we get: i18nspector: error: invalid language
-
intrigeri authored
lang() can return e.g.: - fr: language = French, no territory nor script specified - fr_CA: language = French, territory = Canada - sr_Latn: language = Serbian, script = Latin
-
intrigeri authored
-
- 25 Jun, 2019 3 commits
- 29 Feb, 2016 3 commits
- 13 Feb, 2016 1 commit
-
-
intrigeri authored
i18nspector (0.22) unstable; urgency=low * Summary of tag changes: + Renamed: - unusual-plural-forms (from incorrect-plural-forms)
-
- 19 Dec, 2015 1 commit
-
-
intrigeri authored
-
- 30 Nov, 2015 1 commit
-
-
intrigeri authored
I don't think that patching ikiwiki to avoid this is worth the effort: we have licensing and copyright info at the repository level.
-
- 12 Jun, 2015 1 commit
-
-
intrigeri authored
It comes from the main Tails Git repository at commit 3d216e76e61514e25e01791ec88e7750c86a90b3.
-