View Issue Details

IDProjectCategoryView StatusLast Update
0005699Talerauditorpublic2019-12-20 18:11
ReporterLUG Assigned ToMarcello Stanisci Resolution deadline
ImplementerReviewer 
Release signed byDeployed by 
VerifierCommunicator 
Data protection authority notification requiredNo  
CSIRT notification requiredNo  
Estimated effort (hours)Actual effort (hours)0 
Fast-trackNo
Work packagenone
PrioritynormalSeveritytrivialReproducibilityalways
Status closedResolutionfixed 
PlatformLinuxOSUbuntuOS Version16.04
Product Versiongit (master) 
Target Version0.6Fixed in Version0.6 
Summary0005699: locale-update make recipe fails
DescriptionSince commit c288ba14fd475a03ef148769ffb4146669a34de9 the execution of make fails for me at the locale-update recipe with the following error:
> msgmerge -U -m --previous locale/en/LC_MESSAGES/messages.po locale/messages.pot
> msgmerge: error while opening "locale/en/LC_MESSAGES/messages.po" for reading: No such file or directory

And indeed I don't have a taler_auditor/locale/en/ directory, only taler_auditor/locale/babel.map and taler_auditor/locale/messages.pot are present.

On a more general note: Do you think it'd make sense to add a simple CI/CD job that mirrors the setup instructions from the README and just checks for the overall exit code? In that way we could make sure that trivial errors are caught automatically and also provide a setup reference for devs that we know at least builds.
Steps To Reproduce$ git clone https://git.taler.net/auditor.git taler_auditor

$ cd taler_auditor

$ git submodule init
Submodule 'static/web-common' (git://taler.net/web-common) registered for path 'static/web-common'

$ git submodule update --remote
Cloning into 'static/web-common'...
remote: Counting objects: 796, done.
remote: Compressing objects: 100% (792/792), done.
remote: Total 796 (delta 499), reused 0 (delta 0)
Receiving objects: 100% (796/796), 223.13 KiB | 0 bytes/s, done.
Resolving deltas: 100% (499/499), done.
Checking connectivity... done.
Submodule path 'static/web-common': checked out '68cb09ce78e261dbcddc69b174b8766f96794971'

$ make
env PYTHONPATH="." pybabel extract -F locale/babel.map -o locale/messages.pot .
extracting messages from index.html.j2 (encoding="utf-8", lstrip_blocks="True", trim_blocks="True")
writing PO template file to locale/messages.pot
msgmerge -U -m --previous locale/en/LC_MESSAGES/messages.po locale/messages.pot
msgmerge: error while opening "locale/en/LC_MESSAGES/messages.po" for reading: No such file or directory
Makefile:22: recipe for target 'locale-update' failed
make: *** [locale-update] Error 1
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2019-04-22 13:01

manager   ~0014324

I've fixed the specific issue. And yes, we should add a CI job also for the auditor page. Marcello, could you please do this?

Marcello Stanisci

2019-04-22 13:44

reporter   ~0014326

Last edited: 2019-04-22 13:45

92a9466..875fdb9 (deployment.git) makes it. The CI job is executed by the {test,demo}-{blue,green} users.

Will close this after the next successful build though.

Marcello Stanisci

2019-04-22 16:49

reporter   ~0014331

Auditor gets built by Buildbot now, closing.

Issue History

Date ModifiedUsernameFieldChange
2019-04-22 12:39LUGNew Issue
2019-04-22 12:39LUGStatusnew => assigned
2019-04-22 12:39LUGAssigned To => Christian Grothoff
2019-04-22 13:01Christian GrothoffNote Added: 0014324
2019-04-22 13:01Christian GrothoffAssigned ToChristian Grothoff => Marcello Stanisci
2019-04-22 13:02Christian GrothoffSeverityminor => trivial
2019-04-22 13:02Christian GrothoffProduct Version => git (master)
2019-04-22 13:02Christian GrothoffTarget Version => 0.6
2019-04-22 13:44Marcello StanisciNote Added: 0014326
2019-04-22 13:45Marcello StanisciNote Edited: 0014326
2019-04-22 16:49Marcello StanisciNote Added: 0014331
2019-04-22 16:49Marcello StanisciStatusassigned => informing
2019-04-22 16:49Marcello StanisciResolutionopen => fixed
2019-12-20 18:11Christian GrothoffFixed in Version => 0.6
2019-12-20 18:11Christian GrothoffStatusinforming => closed