search

anqixu / ueye_cam

A ROS nodelet and node that wraps the driver API for UEye cameras by IDS Imaging Development Systems GMBH.
Other
60 stars 102 forks source link

Ci build #104

Closed nullket closed 3 years ago

nullket commented 3 years ago

This will test if the package builds for every push to master and PR to master.

It is not meant to prepare or create the release. It is only meant for testing within this repo. Further details here https://github.com/anqixu/ueye_cam/issues/101

anqixu commented 3 years ago

I just realized that our master branch is automatically being CI-ed by ROS jenkin already:

https://build.ros.org/job/Kdev__ueye_cam__ubuntu_xenial_amd64/ https://build.ros.org/job/Mdev__ueye_cam__ubuntu_bionic_amd64/

Oh well :)

nullket commented 3 years ago

I already thought so, but as mentioned in the issue, I could not find out how all of this works. It should also be possible to check PRs with the rosbuild farm.

It still seems like we get cmake warnings (stating perfectly fine that the driver is not installed and will be downloaded) which then cause the build to be marked „unstable“ in Jenkins. Fixing this would again require some work and it is not a real issue as the driver is indeed really missing.

Anyways: We currently have a working solution for releases and ci testing of the main branch and PRs. If someday somebody finds the energy to integrate the PR tests and fixes the „unstable“ build markings on the ros buildfarm, we could completely move there. Until then we have something that works and might good enough :)