need to source build dependencies for ament-python build type

[mikaelarguedas]

No description provided.

[dirk-thomas]

This seems to be already the case: see

ament_tools/ament_tools/build_types/ament_python.py

Line 152 in 711fa60

prefix = self._get_command_prefix('install', context)

Can you please provide a reproducible example.

[dirk-thomas]

Might it be related to the introspection of the setup file before the actual build is happening? Triggered by this line

ament_tools/ament_tools/build_types/ament_python.py

Line 470 in 711fa60

args = get_setup_arguments(os.path.join(context.source_space, 'setup.py'))

[mikaelarguedas]

TBH that was a long time ago, I don't really remember. I think that I needed to source manually rclpy otherwise I could not build the executables of ros2/examples#127 (because setup.py was calling some rclpy functions)

[clalancette]

@mikaelarguedas Since the upstream PR was merged, can this be closed?

[wjwwood]

@dirk-thomas when you get a chance, can this be merged? (@mikaelarguedas said to ask you)