Skip to content
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.

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

Poetry #707

Draft
wants to merge 35 commits into
base: master
Choose a base branch
from
Draft
Changes from 1 commit
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
d14c0d6
Remove unneeded semi-colon
MatthijsBurgh Jan 31, 2024
99c734d
Bump desired pip version to 23
MatthijsBurgh Jan 31, 2024
a47c284
Use bash array correctly
MatthijsBurgh Jan 31, 2024
9d4a239
(bootstrap) no escaping needed in echo's
MatthijsBurgh Feb 1, 2024
d826e82
(check-pip-pkg-installed-version.py) small update
MatthijsBurgh Feb 5, 2024
782b172
Fix typos
MatthijsBurgh Feb 5, 2024
0ebdfd1
(tue-env) add include system sites arg for venv
MatthijsBurgh Feb 5, 2024
a823986
(bootstrap) lower-/uppercase for local/global vars
MatthijsBurgh Feb 5, 2024
b32defa
(bootstrap) add arg to include system args in venv
MatthijsBurgh Feb 5, 2024
1be0954
(tue-make) run colcon on system interpreter
MatthijsBurgh Feb 5, 2024
641bb43
(tue-get) run scripts with system interpreter
MatthijsBurgh Feb 5, 2024
ef656c0
(tue-get) consistent case styling
MatthijsBurgh Feb 5, 2024
fd933a5
(tue-get) implement pip install site
MatthijsBurgh Feb 5, 2024
f8124c7
(tue-get) PyYaml>=6
MatthijsBurgh Feb 5, 2024
d3e9c17
Extend .git-blame-ignore-revs
MatthijsBurgh Feb 5, 2024
1a66fd2
(bootstrap) consistent case styling
MatthijsBurgh Feb 5, 2024
23c70b6
Extend .git-blame-ignore-revs
MatthijsBurgh Feb 5, 2024
1867072
Bump desired pip version to 24
MatthijsBurgh Feb 5, 2024
84019c7
targetdir -> target_dir
MatthijsBurgh Feb 5, 2024
a09c725
Fix typo
MatthijsBurgh Feb 5, 2024
8f86ad0
Use var for reused file name
MatthijsBurgh Feb 7, 2024
6c2a75a
targetdir -> target_dir
MatthijsBurgh Feb 7, 2024
5e916f7
Fix typo's and var/function names
MatthijsBurgh Feb 7, 2024
80bd342
(docs) markdownlint
MatthijsBurgh Feb 7, 2024
b56f9af
(docker) Remove equal sign, when no default value
MatthijsBurgh Feb 7, 2024
07f5712
Fix typo's
MatthijsBurgh Feb 7, 2024
79b39f2
catkin/colcon run outside the venv
MatthijsBurgh Feb 9, 2024
7a8cf31
Fix typo
MatthijsBurgh Feb 9, 2024
dae83ac
(tue-install-ros) small logic fix
MatthijsBurgh Feb 9, 2024
276a6b5
(tue-get) better target args handling
MatthijsBurgh Feb 9, 2024
4601fbc
(tue-get) split pip list for different site-packages
MatthijsBurgh Feb 9, 2024
749fc9b
First working integration of poetry into tue-env
MatthijsBurgh Feb 9, 2024
c575230
(tue-get) fix update of all targets
MatthijsBurgh Feb 12, 2024
626f1e6
(tue-get) add debug logging to target resolving
MatthijsBurgh Feb 12, 2024
d23a61c
(tue-get) print debug info of creating the state file
MatthijsBurgh Feb 12, 2024
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
Prev Previous commit
Next Next commit
targetdir -> target_dir
  • Loading branch information
MatthijsBurgh committed Feb 5, 2024
commit 84019c79390335765e76aecea4094b7af9898317
8 changes: 4 additions & 4 deletions installer/tue-install-impl.bash
Original file line number Diff line number Diff line change
@@ -459,8 +459,8 @@ function tue-install-git
"The problem will probably be solved by resourcing the setup"
fi

local targetdir version
targetdir=$(_git_url_to_repos_dir "${repo_pre}")
local target_dir version
target_dir=$(_git_url_to_repos_dir "${repo_pre}")

# The shift here is to ensure that all options are explicitly checked as they can be at most 2
# and are optional
@@ -476,8 +476,8 @@ function tue-install-git
do
case $i in
--target-dir=* )
targetdir="${i#*=}"
targetdir="${targetdir/#\~/$HOME}"
target_dir="${i#*=}"
target_dir="${target_dir/#\~/$HOME}"
;;
--version=* )
version="${i#*=}" ;;