ament_lint_auto() does not use latest file for linting.

[dawonn-haval]

I'm trying to upgrade some packages to use ament_cmake_auto and ament_lint_auto but it seems that ament_lint_auto only lints the first version of the source file, and does not update between builds. Here is my CMakeLists.txt.

1. I clean my build environment.
2. I then run colcon build & colcon test
3. I have no linting errors
4. I insert some linter errors into the source file (such as whitespace at EOL)
5. run colcon build & colcon test
6. I have no linting errors
7. I clean my build environment.
8. I then run colcon build & colcon test
9. I have linting errors

Works as expected if I replace find_package(ament_lint_auto REQUIRED) with

  find_package(ament_cmake_gtest REQUIRED)
  find_package(ament_lint_auto REQUIRED)
  find_package(ament_cmake_cppcheck REQUIRED)
  find_package(ament_cmake_cpplint REQUIRED)
  find_package(ament_cmake_xmllint REQUIRED)

[dirk-thomas]

I tried this with a workspace containing the head of ROS 2 packages. I added a trailing space in a .cpp file in the ament_index_cpp package and the colcon test invocation (from step 5) shows the linter failure as a test failure as expected. Please try to reproduce the very same change in ament_index_cpp.

[zflat]

What is in your package.xml file? I had a similar sounding issue.

I had:

<test_depend>ament_cpplint</test_depend>

I needed:

<test_depend>ament_cmake_cpplint</test_depend>