diasurgical/devilutionX

Install GetText on CI runners to generate translations for test builds

Open

#3801 opened on Dec 19, 2021

View on GitHub
 (1 comment) (2 reactions) (0 assignees)C++ (7,255 stars) (743 forks)batch import
build systemgood first issue

Description

In most cases this shouldn't hurt performance much. But I think it might for the (already slow) MSVC build so maybe exclude that.

For most systems it should simply be adding apt gettext to the setup process.

Contributor guide

Tech stack
cppshellgithub actions
Domain
ci cddevops
Issue type
devops
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
1
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
basic CI knowledgefamiliarity with Linux command line
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
90
Research direction
The issue asks to add 'gettext' installation to CI runners for test builds, but exclude the MSVC build. Look for CI configuration files like .github/workflows/*.yml or similar. The solution is to add a step to install gettext via apt get. Ensure the change is conditional to skip MSVC. After adding, verify tests pass.