Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ament_cppcheck fails to find file #177

Closed
esteve opened this issue Aug 9, 2019 · 0 comments · Fixed by #188
Closed

ament_cppcheck fails to find file #177

esteve opened this issue Aug 9, 2019 · 0 comments · Fixed by #188
Assignees
Labels
bug Something isn't working

Comments

@esteve
Copy link
Contributor

esteve commented Aug 9, 2019

Running cppcheck for the repository in https://github.com/ApexAI/velodyne_driver/tree/initial-checkin fails

I don't know what's going on, because the other packages pass just fine, but ament_cppcheck fails with only one package (velodyne_driver) with the following error:

Starting >>> lidar_utils
Starting >>> udp_driver
Finished <<< udp_driver [4.02s]                                           
Finished <<< lidar_utils [4.25s]          
Starting >>> velodyne_driver
--- stderr: velodyne_driver                   
Errors while running CTest
---
Finished <<< velodyne_driver [4.42s]    [ with test failures ]
Starting >>> velodyne_node
Finished <<< velodyne_node [3.58s]           

Summary: 4 packages finished [12.4s]
  1 package had stderr output: velodyne_driver
  1 package had test failures: velodyne_driver
../velodyne_driver_build/velodyne_driver/Testing/20190809-1420/Test.xml: 7 tests, 0 errors, 1 failure, 0 skipped
../velodyne_driver_build/velodyne_driver/test_results/velodyne_driver/cppcheck.xunit.xml: 1 test, 0 errors, 1 failure, 0 skipped
- velodyne_driver cppcheck.xunit.missing_result
  <<< failure message
    The test did not generate a result file:
    
    Traceback (most recent call last):
      File "/opt/ros/dashing/bin/ament_cppcheck", line 11, in <module>
        load_entry_point('ament-cppcheck==0.7.6', 'console_scripts', 'ament_cppcheck')()
      File "/opt/ros/dashing/lib/python3.6/site-packages/ament_cppcheck/main.py", line 164, in main
        report[filename].append(data)
    KeyError: '/home/esteve/velodyne_driver/velodyne_driver/include/velodyne_driver/vlp16_translator.hpp'
  >>>

Summary: 130 tests, 0 errors, 2 failures, 0 skipped

Is there anything in the directory structure of the repository that makes ament_cppcheck ?

t0ny-peng pushed a commit to t0ny-peng/ament_lint that referenced this issue Sep 4, 2019
t0ny-peng pushed a commit to t0ny-peng/ament_lint that referenced this issue Sep 4, 2019
Signed-off-by: Hao Peng <hao.peng@apex.ai>
@dirk-thomas dirk-thomas added the bug Something isn't working label Sep 5, 2019
@dirk-thomas dirk-thomas self-assigned this Sep 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants