Merge ~ogayot/ubiquity:mktemplates-set-x into ubiquity:main

Proposed by Olivier Gayot
Status: Merged
Merged at revision: ca90ecf5c7164d04f4b9f505be83008b314fe82a
Proposed branch: ~ogayot/ubiquity:mktemplates-set-x
Merge into: ubiquity:main
Diff against target: 21 lines (+2/-2)
1 file modified
d-i/source/localechooser/mktemplates.base (+2/-2)
Reviewer Review Type Date Requested Status
Steve Langasek Approve
Review via email: mp+436424@code.staging.launchpad.net

Commit message

mktemplates.base: add more logging to ease diagnosis for failed builds

Description of the change

After uploading 23.04.3, I observed build failures on amd64 when executing mktemplates.base.

The build logs do not tell exactly what command failed to execute, but so far I saw it fail when processing bg.po and hu.po:

./mktemplates.base debian/templates.base-in debian/templates.base
warning: locale kab_DZ.UTF-8 not listed in /usr/share/i18n/SUPPORTED
am.po ar.po ast.po be.po bg.po make[3]: *** [Makefile:11: debian/templates.base] Error 1

./mktemplates.base debian/templates.base-in debian/templates.base
warning: locale kab_DZ.UTF-8 not listed in /usr/share/i18n/SUPPORTED
am.po ar.po ast.po be.po bg.po bn.po bo.po bs.po ca.po cs.po ................................... done.
cy.po da.po .................................. done.
de.po .................................... done.
dz.po el.po eo.po es.po .................................... done.
et.po eu.po fa.po fi.po fr.po ga.po gl.po ............................................ done.
gu.po he.po hi.po hr.po hu.po ................................... done.
id.po make[3]: *** [Makefile:11: debian/templates.base] Error 1

This patch adds set -x so that the shell tells which command failed exactly. Also it removes the redirection of errors to /dev/null so we know what went wrong if the msgcat command fails.

Failed build logs:
https://launchpadlibrarian.net/648124470/buildlog_ubuntu-lunar-amd64.ubiquity_23.04.3_BUILDING.txt.gz
https://launchpadlibrarian.net/648213368/buildlog_ubuntu-lunar-amd64.ubiquity_23.04.3_BUILDING.txt.gz

To post a comment you must log in.
Revision history for this message
Steve Langasek (vorlon) :
review: Approve

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches