Repository navigation
Future of ament_build_type_gradle #9
Description
As I understand it, ament_tools is deprecated having been replaced by colcon as the build tool for ROS distros newer than Bouncy. Further, ament_build_type_gradle depends on ament_tools. I'm opening this issue to start a discussion about how to proceed for later ROS releases (e.g. Dashing).
If I ignore ament_build_type_gradle, I'm able to build ROS 2 Dashing with Java support, except some runtime environment information is missing for gradle packages (e.g. CLASSPATH, AMENT_PREFIX_PATH) and the packages are not registered with the ament index. My main question is where should these things be handled?
Should we keep the logic in ament_build_type_gradle and refactor so it doesn't use ament_tools?
Or perhaps we can move all of the functionality into ament_gradle_plugin?
It seems like what we want is a new package ament_index_java (similar to the C++ and Python packages) that provides helpers for registering/querying entries in the ament index. Then this new Java API can be called from ament_gradle_plugin or directly by the gradle package.
I'm new to gradle, so I'm still doing some reading and experimenting.
Activity
esteve commentedon Oct 9, 2019
@jacobperron
ament_build_type_gradledepends onament_toolsjust because at the time ROS 2 was distributed only as a .repos file withament_toolsbeing one of the repos. The dependency was there only to forceament_toolsto be built before buildingament_build_type_gradle.In any case, there should be no need for
ament_build_type_gradlenow that https://github.com/colcon/colcon-ros-gradle is part of thecolcondistribution. A good start would be to change the CI scripts to usecolconand fix whatever break.jacobperron commentedon Oct 9, 2019
@esteve Thanks for the reply 🙂
From trying to run examples, it appears we're missing some logic for setting the environment (
CLASSPATHandAMENT_PREFIX_PATH) and registering with the ament index.This logic from
ament_build_type_gradleneeds to go somewhere:https://github.com/esteve/ament_java/blob/e6e23fbf54c2027999afa234474a4126175617cd/ament_build_type_gradle/ament_build_type_gradle/__init__.py#L151-L176
https://github.com/esteve/ament_java/blob/e6e23fbf54c2027999afa234474a4126175617cd/ament_build_type_gradle/ament_build_type_gradle/__init__.py#L210-L225
Even with
colcon-ros-gradleinstalled, I am not able to run examples without manually setting up the environment. Though, I'm not sure thatcolcon-ros-gradleshould be responsible for this, since I think we should be able to build an ament gradle package without colcon (e.g.gradle build).I'll do this too.
nuclearsandwich commentedon Oct 16, 2019
I think that we'll likely want a support package suite analogous to ament_cmake, that contains common code for handling the installation of package.xml files and ament index registration. We don't have one in Python because the setuptools format doesn't allow us to depend on third-party libraries during the setup itself. Keeping the code central allows us to update it in only one place which is far preferable to the approach necessary in Python.
jacobperron commentedon Feb 5, 2020
I've added a support package
ament_index_javathat supports both ament index queries as well as registering new resources (like packages). Currently, registering a package only installs a marker file, though it might make sense to expand on this to also install a package.xml and/or provide shell rules for configuring your environment.talregev commentedon Feb 15, 2020
Can you try to compile with colcon?
https://github.com/colcon/colcon-gradle
https://github.com/colcon/colcon-ros-gradle
I added them to pr
I add a repo change that you could compile your ament java with colcon.
I try it on windows with vs2019 and dashing. It compile.
https://github.com/ros2-java/ament_java/pull/13/files
Please try and let me know.