Skip to content

Commit

Permalink
Merge pull request #501 from k-okada/skip_circular_dependency
Browse files Browse the repository at this point in the history
Skip circular dependency when building ROS-O package
  • Loading branch information
k-okada authored Jan 9, 2025
2 parents 16cb8fa + d9581b1 commit 46b7fb2
Show file tree
Hide file tree
Showing 3 changed files with 36 additions and 16 deletions.
20 changes: 19 additions & 1 deletion .github/workflows/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,10 @@ jobs:
TEST_PKGS: "pr2eus_moveit pr2eus_tutorials"


container: ${{ matrix.CONTAINER }}
container:
image: ${{ matrix.CONTAINER }}
volumes:
- /tmp/node20:/__e/node20

steps:
- name: Install latest git ( use sudo for ros-ubuntu )
Expand All @@ -96,6 +99,21 @@ jobs:
git config --global --add safe.directory $GITHUB_WORKSPACE
fi
- name: Try to replace `node` with an glibc 2.17
shell: bash
run: |
if [ "${{ matrix.CONTAINER }}" = "jskrobotics/ros-ubuntu:14.04" ]; then
export USER=$(whoami)
sudo chmod 777 -R /__e/node20
sudo chown -R $USER /__e/node20
fi
ls -lar /__e/node20 &&
sudo apt-get install -y curl &&
curl -Lo /tmp/node.tar.gz https://unofficial-builds.nodejs.org/download/release/v20.17.0/node-v20.17.0-linux-x64-glibc-217.tar.gz &&
cd /__e/node20 &&
tar -x --strip-components=1 -f /tmp/node.tar.gz &&
ls -lar /__e/node20/bin/
- name: Checkout
uses: actions/[email protected]

Expand Down
30 changes: 16 additions & 14 deletions pr2eus_tutorials/package.xml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<package>
<package format="3">
<name>pr2eus_tutorials</name>
<version>0.3.15</version>
<description>pr2eus_tutorials</description>
Expand All @@ -16,21 +16,23 @@

<!-- Dependencies needed to compile this package. -->
<build_depend>pr2eus</build_depend>
<build_depend>jsk_pcl_ros</build_depend>

<!-- Dependencies needed after this package is compiled. -->
<run_depend>app_manager</run_depend>
<run_depend>eusurdf</run_depend>
<run_depend>pr2_gazebo</run_depend>
<run_depend>pr2eus</run_depend>
<run_depend>roseus_tutorials</run_depend>
<run_depend>image_view2</run_depend>
<run_depend>jsk_pcl_ros</run_depend>
<run_depend>jsk_interactive_marker</run_depend>
<run_depend>jsk_recognition_msgs</run_depend>
<run_depend>jsk_rviz_plugins</run_depend>
<run_depend>jsk_pr2_startup</run_depend>
<run_depend>jsk_maps</run_depend>
<exec_depend>app_manager</exec_depend>
<exec_depend>eusurdf</exec_depend>
<exec_depend>pr2_gazebo</exec_depend>
<exec_depend>pr2eus</exec_depend>
<exec_depend>roseus_tutorials</exec_depend>
<exec_depend>image_view2</exec_depend>
<exec_depend>jsk_pcl_ros</exec_depend>
<exec_depend>jsk_interactive_marker</exec_depend>
<exec_depend>jsk_recognition_msgs</exec_depend>
<exec_depend>jsk_rviz_plugins</exec_depend>
<!-- jsk_pr2_startup is provided from jsk_robot, this is circular dependency -->
<exec_depend condition="$ROS_DISTRO != debian">jsk_pr2_startup</exec_depend>
<!-- jsk_maps is provided from jsk_demos, this is circular dependency -->
<exec_depend condition="$ROS_DISTRO != debian">jsk_maps</exec_depend>
<exec_depend>jsk_pcl_ros</exec_depend>

<!-- Dependencies needed only for running tests. -->
<test_depend>rostest</test_depend>
Expand Down

0 comments on commit 46b7fb2

Please sign in to comment.