Skip to content

fix(deps-dev): Ensure user-scoped pipx installs and reliable version checks for pipx packages. #1320

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
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

fix(deps-dev): Ensure user-scoped pipx installs and reliable version checks for pipx packages. #1320

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
e6817d8
remove sudo when installing pipx packages
Bill-hbrhbr Sep 23, 2025
1701f05
Apply suggestions from code review
Bill-hbrhbr Sep 23, 2025
896a9c4
line break for lint
Bill-hbrhbr Sep 23, 2025
12fc072
Preserve all env vars when running pipx install as user
Bill-hbrhbr Sep 23, 2025
24adfec
Add pipx-packages to trigger various build CI pipelines
Bill-hbrhbr Sep 23, 2025
84c01bf
Merge branch 'main' into sudo-install-lib
Bill-hbrhbr Sep 23, 2025
22ec79e
fix issue where the installed binary is not immediately available for…
Bill-hbrhbr Sep 25, 2025
59de3b0
Find installed pipx binary in the more reliable way
Bill-hbrhbr Sep 25, 2025
611596a
Merge branch 'main' into sudo-install-lib
Bill-hbrhbr Sep 25, 2025
ac5ec93
fix bug
Bill-hbrhbr Sep 25, 2025
d547a7c
Merge branch 'main' into sudo-install-lib
Bill-hbrhbr Sep 27, 2025
6bea8cf
Merge branch 'main' into sudo-install-lib
Bill-hbrhbr Sep 28, 2025
7523533
Merge branch 'main' into sudo-install-lib
Bill-hbrhbr Oct 1, 2025
4640164
Merge branch 'main' into sudo-install-lib
Bill-hbrhbr Oct 2, 2025
7a371fc
Merge branch 'main' into sudo-install-lib
Bill-hbrhbr Oct 10, 2025
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
6 changes: 6 additions & 0 deletions components/core/tools/scripts/lib_install/pipx-packages/install-all.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,12 @@ set -o errexit
set -o nounset
set -o pipefail

is_sudo_from_non_root=$(( $EUID == 0 && ${SUDO_UID:-0} != 0 ))
if (( is_sudo_from_non_root )); then
echo "Installing pipx packages to the user environment (sudo lifted)."
exec sudo --set-home --user="$SUDO_USER" "$0" "$@"
fi

script_dir="$(cd "$(dirname "${BASH_SOURCE[0]}")" &>/dev/null && pwd)"

"${script_dir}/install-cmake.sh"
Expand Down
Loading