Skip to content

Commit

Permalink
Merge pull request #40 from RoboStack/fix-install-location
Browse files Browse the repository at this point in the history
Fix binary install location
  • Loading branch information
Tobias-Fischer authored Feb 4, 2023
2 parents 3b26d3f + eee8179 commit a1e74dc
Showing 1 changed file with 16 additions and 1 deletion.
17 changes: 16 additions & 1 deletion vinca/templates/build_ament_python.sh.in
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,19 @@
# DO NOT EDIT!

pushd $SRC_DIR/$PKG_NAME/src/work
$PYTHON -m pip install . --no-deps -vvv

# If there is a setup.cfg that contains install-scripts then we should not set it here
grep -q "install[-_]scripts" setup.cfg
if [ $? -eq 0 ]; then
INSTALL_SCRIPTS_ARG=""
else
# this is not quite what we want as PKG_NAME contains ros-humble- ..
# INSTALL_SCRIPTS_ARG="--install-scripts=$PREFIX/lib/$PKG_NAME"
# therefore for now throw an error as packages should have a setup.cfg
echo "This package is missing a setup.cfg, stopping."
exit 1
fi

# Old: This installs the binaries in the wrong location - see https://github.com/RoboStack/ros-humble/issues/41
# $PYTHON -m pip install . --no-deps -vvv
$PYTHON setup.py install --prefix="$PREFIX" --install-lib="$SP_DIR" $INSTALL_SCRIPTS_ARG

0 comments on commit a1e74dc

Please sign in to comment.