#!/bin/bash

ARCH=$(python fetch_architecture.py)
if [ $ARCH == "aarch64" -o $ARCH == "armv71" ] ; then
    export PATH=/opt/local/llvm/bin:${PATH}
    cd /app
    RUSTFLAGS="-C linker=lld" wasm-pack build --target web --release plume-front
else
    wasm-pack build --target web --release plume-front
fi

via the terminal, it works just fine. But when I run it via the Dockerfile,

COPY . . 
RUN cargo install wasm-pack 
RUN chmod a+x ./script/plume-front.sh 
RUN sleep 1 
RUN ./script/plume-front.sh

It says python: command not found and then [: too many arguments

you are viewing a single comment's thread
view the rest of the comments

[–] onlinepersona@programming.dev 5 points 10 months ago* (last edited 10 months ago) (5 children)

In addition to what everybody said:

use bash strict mode
use shellcheck

Those will help catch many problems before they happen or exactly when they happen.

P.S I would recommend against

set -o pipefail

It can introduce some very weird behaviour when it doesn't work.

Anti Commercial-AI license

[–] wasabi@discuss.tchncs.de 1 points 10 months ago (3 children)

Could you elaborate on the weird behaviour introduced by pipefail?

[–] onlinepersona@programming.dev 2 points 10 months ago* (last edited 10 months ago) (2 children)

This wiki explains one pitfall of nonstandard return codes in pipes. The other one that bit me is pipes that just stop existing (error 141).

Anti Commercial-AI license

[–] wasabi@discuss.tchncs.de 1 points 10 months ago (1 children)

Thanks. Bash should just be avoided except trivial uses. It has an impressive collection of footguns. I've written a lot of bash, even used it for CGI scripts to create web tools, but honestly it is time to move on. Ansible and Python cover most of the bash use cases.

[–] onlinepersona@programming.dev 1 points 10 months ago

I fully agree with you. Hopefully newer shells will be used for distros (nushell, xonsh, osh, ...). Backwards compatibility with bash is a huge pitfall that should be a disqualification criterion.

Anti Commercial-AI license

load more comments (1 replies)