AaronFiore/docker-finance
1
0
Fork 0
forked from EvergreenCrypto/docker-finance
Code Activity

client: src: lib_docker: run: only allocate TTY when needed

Browse Source 
Fixes running with client (host) cron.
This commit is contained in:
Aaron Fiore
2025-03-20 18:39:38 -07:00
parent dff09cb017
commit 050d95b60b
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
client/src/docker/lib/internal/lib_docker.bash
View File

@@ -394,9 +394,14 @@ function lib_docker::__run()
lib_utils::print_debug "Creating network" lib_utils::print_debug "Creating network"
docker network create "$global_network" &>/dev/null docker network create "$global_network" &>/dev/null
# NOTE: runs bash as interactive so .bashrc doesn't `return` (sources/aliases are needed)
lib_utils::print_debug "Spawning container with command '$*'" lib_utils::print_debug "Spawning container with command '$*'"
lib_docker::__docker_compose run -it --rm --entrypoint='/bin/bash -i -c' docker-finance "$@"
# Terminal requirements (e.g., for cron, don't allocate TTY)
local _tty
test -t 1 && _tty="t"
# NOTE: runs bash as interactive so .bashrc doesn't `return` (sources/aliases are needed)
lib_docker::__docker_compose run -i"${_tty}" --rm --entrypoint='/bin/bash -i -c' docker-finance "$@"
local -r _return=$? local -r _return=$?
lib_utils::print_debug "Removing network" lib_utils::print_debug "Removing network"