Added include dir to installation of rcl_logging_log4cxx.

[rasmusan]

Fixes issue #5. This enables compiling ROS2 with log4cxx (by setting RCL_LOGGING_IMPLEMENTATION to rcl_logging_log4cxx).

[nburek]

@thomas-moulard - please run the following CI job:

• Gist file: https://gist.github.com/nburek/695a3f1f884d5bd4bb5f9778abdf5feb
• Additional BUILD flags: --packages-up-to rcl_logging_log4cxx rcl
• Additional TEST flags: --packages-up-to rcl
• • CI Job: ci_launcher

[thomas-moulard]

• Linux
• Linux-aarch64
• macOS
• Windows

[tfoote]

This looks like an unrelated flake8 regressions in the idl generator on all platforms.

Error Message
'rosidl_parser.definition.SIGNED_INTEGER_TYPES' imported but unused:
from rosidl_parser.definition import SIGNED_INTEGER_TYPES
Standard Output
Checked files:
* ./rosidl_generator_c/__init__.py

Possibly related to ros2/rosidl#371 @jacobperron

[nburek]

Looks like the regression may have been fixed here too: ros2/rosidl#373

[thomas-moulard]

• Linux
• Linux-aarch64
• macOS
• Windows

[clalancette]

The change looks good to me with green CI.

[tfoote]

The CI failed some tests. This is now hitting a LaunchTestService import error on all platforms and rmws that looks unrelated to this change.

../../../../build/rcl/test/test/test_services__rmw_connext_cpp_.py:6: in <module>
    from launch_testing import LaunchTestService
E   ImportError: cannot import name 'LaunchTestService'

There's a recent change related here: ros2/rcl#405

[clalancette]

Oh, I see what happened. We merged ros2/rcl#405, which means that ros2/rcl#425 needs a rebase now. As @dirk-thomas suggested, this is only installing stuff that wasn't there before, so I'm going to merge this anyway.