Skip to content

Fix interpreter selection in the ansible-cmdb wrapper #4

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

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

Fix interpreter selection in the ansible-cmdb wrapper #4

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
0a3ca01
Look for python3/python2 binaries, not just 'python', in wrapper
gdevenyi Aug 5, 2026
227fa47
Prefer the interpreter next to the wrapper
gdevenyi Aug 5, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 15 additions & 1 deletion src/ansible-cmdb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,21 @@ dbg () {

# Find suitable python binary
find_py_bin () {
which -a python | while read -r TRY_PY_BIN
# An interpreter sitting next to this script wins. When installed into a
# virtualenv, that is the interpreter the package was installed for, and
# it is the only one that can import ansiblecmdb. Searching $PATH first
# would pick the system python whenever the venv isn't activated.
SELF_DIR=$(dirname "$0")
for SIBLING in "$SELF_DIR/python3" "$SELF_DIR/python"
do
if [ -x "$SIBLING" ]; then
dbg "Using interpreter next to wrapper: $SIBLING"
echo "$SIBLING"
return 0
fi
done

which -a python3 python2 python | while read -r TRY_PY_BIN
do
dbg "Trying python bin: $TRY_PY_BIN"

Expand Down