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.

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

fix: replaced ARCH with uname -m #100

Merged
merged 3 commits into from
Jan 5, 2024

Commits on Jan 4, 2024

  1. fix: replaced ARCH with uname -m

    In the previous commits the ARCH
    env variable was introduced to the Dockerfile
    and build_container.sh scripts. It was meant to
    help with configuration of linux headers for
    musl-tools, but did not work for some reason.
    Considering the build_container.sh was already
    using `uname -m` calls to get the current arch
    this commit also uses it, so there is no need
    for ARCH anymore.
    
    Signed-off-by: Egor Lazarchuk <[email protected]>
    ShadowCurse committed Jan 4, 2024
    Configuration menu
    Copy the full SHA
    23c434c View commit details
    Browse the repository at this point in the history

Commits on Jan 5, 2024

  1. fix(build_container): removed &&

    Removed && from the script because they
    were causing the script build to succeed
    even if errors were occurring.
    
    Signed-off-by: Egor Lazarchuk <[email protected]>
    ShadowCurse committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    d8fddb6 View commit details
    Browse the repository at this point in the history
  2. refactor(build_container): global ARCH variable

    There are several places that need current arch
    of the system to install/configure the container.
    This commit replaces separate calls of $(uname -m)
    with one global ARCH variable.
    
    Signed-off-by: Egor Lazarchuk <[email protected]>
    ShadowCurse committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    bcd5bc2 View commit details
    Browse the repository at this point in the history