[Tails-l10n] Build failed in Jenkins: check_PO_master #4238

See <https://jenkins.tails.boum.org/job/check_PO_master/4238/changes>

Changes:

[u] Add pre-commit hook which calls check_po.sh whenever we try to commit po

[u] Let's use a correct path, so that we can execute this from whereever within the repo.

[u] Let's execute check_po only if the file we are trying to commit is a po.

[sknauss] add unifiy po script to unifiy po headers.

[hefee] check_po update

[hefee] mark pre-commit as executable.

[hefee] Simplify the po check logic.

[hefee] run check_po with file list of changed files.

[hefee] list only changed po files.

[hefee] ignore deleted po files for check_po.

[hefee] update check_po

[u] control freakiness for whitespaces.

[u] Move this file to a section that is not publicly writable.

[u] Moved this file to bin instead.

[u] Add comment and function to rewrap the po file.

[u] This needs an email address according to spec.

[u] Add more comments

[u] Do not call check_po via contribute/l10n/ for security reasons, wiki/src might be writable publicly, so let's avoid that people call this file from there.

[u] Delete unused exit-code. It is not necessary.

[u] Rename variable to something more understandable.

[u] Let's use this check only on files that are part of wiki/, ie. our documentation. will-fix: #15403.

[u] I have no idea why I added that dot initially.

[u] We should add a whitespace check somewhere..

[u] Add set -u and set -e for consistency with our other bash scripts.

[hefee] Replace unify_po-headers bash script with a python one.

[hefee] fix typos.

[u] Fix typos

[hefee] Merge unify_po with check_po.

[u] wording

[hefee] Add description about check mode.

[hefee] Ignore __pycache__ directories.

[hefee] unify_po-headers: flake8

[hefee] unify_po-headers: move main code to main function.

[hefee] WIP: review from enrico

[hefee] unifyPo: Add tests.

[hefee] rename unifyPo.py -> unifyPo

[hefee] Add ignoring .pyc files.

[hefee] rename

[hefee] Align function names with PEP8.

[hefee] Add decicated Exception: NoLanguageError to raise issues with non langauge filenames.

[hefee] Nitpicking.

[hefee] simplify pre-commit-translation hook.

[hefee] Remove check for wrapped for 79 chars.

[hefee] Remove WARNING: WARNING: from commandline output.

[hefee] refactor msgcat method to not use msgcat and use polib directly instead.

[hefee] rename unifyPo -> lint_po

[hefee] Fail with a FileNotFounderror when po does not exist.

[hefee] default operation is to run against all files in cwd.

[hefee] Rephrase the usage of the script

[hefee] rename --modify -> --fix.

[intrigeri] Move lint_po to jenkins-tools.git.

[intrigeri] Lint PO files.

------------------------------------------
Started by an SCM change
[EnvInject] - Loading node environment variables.
Building remotely on isobuilder3 in workspace <https://jenkins.tails.boum.org/job/check_PO_master/ws/>
Wiping out workspace first.
Cloning the remote Git repository
Cloning repository gitolite@???:tails
> git init <https://jenkins.tails.boum.org/job/check_PO_master/ws/> # timeout=10
Fetching upstream changes from gitolite@???:tails
> git --version # timeout=10
> git -c core.askpass=true fetch --tags --progress gitolite@???:tails +refs/heads/*:refs/remotes/origin/*
> git config remote.origin.url gitolite@???:tails # timeout=10
> git config remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # timeout=10
> git config remote.origin.url gitolite@???:tails # timeout=10
Fetching upstream changes from gitolite@???:tails
> git -c core.askpass=true fetch --tags --progress gitolite@???:tails +refs/heads/*:refs/remotes/origin/*
> git rev-parse origin/master^{commit} # timeout=10
Checking out Revision 7095f81f8ce8fd3fdfab06cb96f2c8b3fd9806b2 (origin/master)
> git config core.sparsecheckout # timeout=10
> git checkout -f 7095f81f8ce8fd3fdfab06cb96f2c8b3fd9806b2
> git rev-list 44ef4fb7f3d6ed57d5603de9e3057887a14f330a # timeout=10

[check_PO_master] $ /bin/sh -xe /tmp/hudson3534751038926389879.sh
+ /var/lib/jenkins/tools/slaves/check_po
  File "/var/lib/jenkins/tools/slaves/check_po", line 83
    return f"Can't detect expect file suffix .XX.po for '{self.fname}'."
                                                                       ^
SyntaxError: invalid syntax
Build step 'Execute shell' marked build as failure