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

WIP: [capability] Add GitHub CI config #29

Open
wants to merge 3 commits into
base: kinetic
Choose a base branch
from

Conversation

130s
Copy link

@130s 130s commented Feb 11, 2022

In a support for #24 with as less effort as possible, I think having CI do some work will be helpful.

If CI fails and I can find time to address, I can also add such changes in this PR if that's ok.

@130s
Copy link
Author

130s commented Feb 11, 2022

Not yet sure why CI hasn't been triggered on this repo for this PR, but on my fork it is triggered https://github.com/130s/srv_tools/actions/runs/1831022995 and failed. Obviously some code changes seem to be needed

E.g. https://github.com/130s/srv_tools/runs/5160898301?check_suite_focus=true

  Starting  >>> plot_tools                          
  
  Warnings   << plot_tools:install /root/target_ws/logs/plot_tools/build.install.000.log
  package init file 'scripts/plot_tools/__init__.py' not found (or not a regular file)
  package init file 'scripts/plot_tools/__init__.py' not found (or not a regular file)
  cd /root/target_ws/build/plot_tools; catkin build --get-env plot_tools | catkin env -si  /usr/bin/make install; cd -
  
  
  Finished  <<< plot_tools                           [ 2.1 seconds ]
  Starting  >>> pointcloud_tools                    
  
  Errors     << pointcloud_tools:cmake /root/target_ws/logs/pointcloud_tools/build.cmake.000.log
  ** WARNING ** io features related to openni will be disabled
  ** WARNING ** io features related to openni2 will be disabled
  ** WARNING ** io features related to pcap will be disabled
  ** WARNING ** io features related to png will be disabled
  ** WARNING ** io features related to libusb-1.0 will be disabled
  ** WARNING ** visualization features related to openni will be disabled
  ** WARNING ** visualization features related to openni2 will be disabled
  ** WARNING ** apps features related to openni will be disabled
  CMake Error at /usr/share/cmake-3.16/Modules/FindPackageHandleStandardArgs.cmake:146 (message):
    Could NOT find Boost (missing: signals) (found version "1.71.0")
  Call Stack (most recent call first):
    /usr/share/cmake-3.16/Modules/FindPackageHandleStandardArgs.cmake:393 (_FPHSA_FAILURE_MESSAGE)
    /usr/share/cmake-3.16/Modules/FindBoost.cmake:2179 (find_package_handle_standard_args)
    CMakeLists.txt:8 (find_package)

If I can find time I'd like to contribute adding patches but if anyone can do that I'm happy to take PR against my forked branch so that I can merge your changes into my PR (and you retain your commit authorship).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants