Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

setup a cleaner build environment #665

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

setup a cleaner build environment #665

wants to merge 4 commits into from

Commits on Feb 1, 2022

  1. Force locale to C.UTF-8 

    Multiple packages have run into build failures on our OBS. On closer
inspection, it was concluded to be issues with the locale settings.

These issues were never seen on the Debian buildd infrastructure because
there the locale setting LC_ALL=C.UTF-8 is in use.

This change keeps us closer to what is used in Debian buildds.

openSUSE#665

Signed-off-by: Emanuele Aina <[email protected]>
    @em-
    em- authored and Ritesh Raj Sarraf committed Feb 1, 2022
    Configuration menu
    Copy the full SHA
    84a6abf View commit details
    Browse the repository at this point in the history

  2. Force the local timezone to be UTC 

    To make the build environment a bit more reproducible, ensure the local
timezone is set to UTC when building packages.

Signed-off-by: Emanuele Aina <[email protected]>
    @em-
    em- authored and Ritesh Raj Sarraf committed Feb 1, 2022
    Configuration menu
    Copy the full SHA
    ee3fae9 View commit details
    Browse the repository at this point in the history

  3. Unset $TERM to prevent unexpectedly colorized output from breaking bu… 

    …ilds

Our OBS workers run under a `screen` session, which gives them an actual
`$TERM` variable.

Some packages like `color.js` then enable colored output, which confuses
some other packages like `node-grunt-legacy-log` which use it in their
testsuite and do not expect colored output.

To avoid that, unset `TERM`.

Signed-off-by: Emanuele Aina <[email protected]>
    @em-
    em- authored and Ritesh Raj Sarraf committed Feb 1, 2022
    Configuration menu
    Copy the full SHA
    7253fc5 View commit details
    Browse the repository at this point in the history

  4. init_buildsystem: Move hostname ip entry to the end of /etc/hosts 

    spymemcached package is FTBFS because a test expects a reverse lookup of
127.0.0.1 to return localhost, but current /etc/hosts configuration
returns the hostname instead.

Let's move this `hostname/ip` entry to the end of /etc/hosts so the
reverse lookup of 127.0.0.1 returns localhost, which makes sense to be
the default case usually.

Signed-off-by: Ariel D'Alessandro <[email protected]>
    @adalessandro
    adalessandro authored and Ritesh Raj Sarraf committed Feb 1, 2022
    Configuration menu
    Copy the full SHA
    a68beef View commit details
    Browse the repository at this point in the history