this post was submitted on 10 Jun 2024
24 points (100.0% liked)

Python

6331 readers
305 users here now

Welcome to the Python community on the programming.dev Lemmy instance!

πŸ“… Events

PastNovember 2023

October 2023

July 2023

August 2023

September 2023

🐍 Python project:
πŸ’“ Python Community:
✨ Python Ecosystem:
🌌 Fediverse
Communities
Projects
Feeds

founded 1 year ago
MODERATORS
[email protected]
[email protected]
[email protected]
24
What Am I Doing Wrong? (lazysoci.al)
submitted 4 months ago by [email protected] to c/[email protected]
16 comments fedilink hide all child comments
 

If I run this

#!/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

all 17 comments
sorted by: hot top controversial new old
[–] friend_of_satan 14 points 4 months ago* (last edited 4 months ago) (1 children)

Python is not found, so $ARCH gets assigned to "", and you didn't double quote your variables in the comparison, so the code parses as [ == "aarch64" which is a syntax error.

Also, maybe uname -m could work instead of that Python script.

[–] [email protected] 7 points 4 months ago

This was super insightful. Thank you so much.

[–] [email protected] 7 points 4 months ago (1 children)

You should use python3 anyway not python. The latter is sometimes Python 3, sometimes Python 2 and sometimes doesn't exist. python3 works reliably, assuming you have it installed.

(And assuming you aren't using the official Windows Python installer, but that doesn't seem like the case here!)

[–] [email protected] 3 points 4 months ago

I ended up adding Python3 as it moaned when I just added Python. Thank you so much for your advice though. I'm grateful.

[–] [email protected] 7 points 4 months ago (1 children)

What is your base image? It has no python installed.

[–] [email protected] 2 points 4 months ago

Rust:1.78-bookworm.

Thank you, manually included it.

[–] thenextguy 6 points 4 months ago (1 children)

Python is not on the Path for docker. The error message "python: command not found" is then passed to the [ command (also called test) which says too many arguments.

Add the path /use/bin/ to your python command. Or figure out why it isn't on the docker path.

[–] [email protected] 3 points 4 months ago

Thank you so much.

[–] [email protected] 5 points 4 months ago* (last edited 4 months ago) (2 children)

In addition to what everybody said:

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

[–] [email protected] 2 points 4 months ago

Thank you so much. Especially for the Shell Check. That's an invaluable resource.

[–] [email protected] 1 points 4 months ago (1 children)

Could you elaborate on the weird behaviour introduced by pipefail?

[–] [email protected] 2 points 4 months ago* (last edited 4 months ago) (1 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

[–] [email protected] 1 points 4 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.

[–] [email protected] 1 points 4 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

[–] [email protected] 4 points 4 months ago* (last edited 4 months ago) (1 children)

This is a docker/bash question, not a python question. Also reading the error message explains the problem.

[–] [email protected] 1 points 4 months ago

Apologies, I thought it was Python because I was trying to execute a Python script. And double sorry for not knowing how to interpret the error message.