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

ABI checker jobs are failing #3292

Closed
scpeters opened this issue Jan 26, 2023 · 4 comments
Closed

ABI checker jobs are failing #3292

scpeters opened this issue Jan 26, 2023 · 4 comments
Labels
bug Something isn't working

Comments

@scpeters
Copy link
Member

Environment

  • OS Version: Ubuntu 22.04
  • Source or binary build? CI build from source

Description

Steps to reproduce

  1. Open a pull request with a minor change, which triggers an ABI checker job
  2. The job will fail

Output

From the Abi checker log:

The GCC parameters:
  gcc -fdump-lang-raw -fkeep-inline-functions -c -x c++ -fpermissive -w  -std=c++11 "/tmp/twsUMx5ApM/dump1.h"  -I/usr/local/destination_branch/include/gazebo-11 -I/usr/local/destination_branch/include/gazebo-11/gazebo -I/usr/local/destination_branch/include/gazebo-11/gazebo/ode -I/usr/include/tbb -I/usr/include/simbody -I/usr/include/sdformat-9.10 -I/usr/include/bullet -I/usr/include/OGRE-2.1 -I/usr/include/OGRE -I/usr/include/x86_64-linux-gnu/qt5 -I/usr/include/ignition/transport8 -I/usr/include/ignition/msgs5 -I/usr/include/ignition/math6 -I/usr/include/ignition/fuel_tools4 -I/usr/include/ignition/common3 -I/usr/include/ignition/cmake2 -I/usr/include/OGRE/Paging

In file included from /usr/local/destination_branch/include/gazebo-11/gazebo/GIMPACT/gim_memory.h:34,
                 from /usr/local/destination_branch/include/gazebo-11/gazebo/GIMPACT/gim_radixsort.h:35,
                 from /usr/local/destination_branch/include/gazebo-11/gazebo/GIMPACT/gim_boxpruning.h:35,
                 from /usr/local/destination_branch/include/gazebo-11/gazebo/GIMPACT/gim_trimesh.h:33,
                 from /usr/local/destination_branch/include/gazebo-11/gazebo/GIMPACT/gimpact.h:59,
                 from /tmp/twsUMx5ApM/dump1.h:6:
/usr/local/destination_branch/include/gazebo-11/gazebo/GIMPACT/gim_math.h:34:10: fatal error: config.h: No such file or directory
   34 | #include "config.h"
      |          ^~~~~~~~~~
compilation terminated.
@scpeters scpeters added the bug Something isn't working label Jan 26, 2023
@scpeters
Copy link
Member Author

I guess we are installing the GIMPACT headers but not their config.h file?

@scpeters
Copy link
Member Author

I think I've found the underlying issue: gazebo-tooling/release-tools#890

@traversaro
Copy link
Collaborator

Recent builds have a green status, see for example: https://build.osrfoundation.org/job/gazebo-abichecker-any_to_any-ubuntu_auto-amd64/1306/ . However, the output is:

+ abi-compliance-checker -lang C++ -lib gazebo-classic -old pkg.xml -new devel.xml
Preparing, please wait ...
WARNING: version number #1 is not set (use --v1=NUM option)
Using GCC 9 (x86_64-linux-gnu, target: x86_64)
WARNING: May not work properly with GCC 4.8.[0-2], 6.* and higher due to bug #78040 in GCC. Please try other GCC versions with the help of --gcc-path=PATH option or create ABI dumps by ABI Dumper tool instead to avoid using GCC. Test selected GCC version first by -test and -gcc-path options.
cc1: warning: command line option ‘-std=c++11’ is valid for C++/ObjC++ but not for C
Checking header(s) X ...
ERROR: some errors occurred when compiling headers
ERROR: see log for details:
  /home/jenkins/workspace/gazebo-abichecker-any_to_any-ubuntu_auto-amd64/abi_checker/logs/gazebo-classic/X/log.txt

WARNING: version number #2 is not set (use --v2=NUM option)
cc1: warning: command line option ‘-std=c++11’ is valid for C++/ObjC++ but not for C
Checking header(s) Y ...
ERROR: some errors occurred when compiling headers
ERROR: see log for details:
  /home/jenkins/workspace/gazebo-abichecker-any_to_any-ubuntu_auto-amd64/abi_checker/logs/gazebo-classic/Y/log.txt

Comparing ABIs ...
Comparing APIs ...
Creating compatibility report ...
Binary compatibility: 100%
Source compatibility: 100%
Total binary compatibility problems: 0, warnings: 0
Total source compatibility problems: 0, warnings: 0
Report: compat_reports/gazebo-classic/X_to_Y/compat_report.html

I am not sure if ERROR: some errors occurred when compiling headers can be safely ignored. If yes, probably we can close the issue. @scpeters @j-rivero

@scpeters
Copy link
Member Author

I think we can close the issue since the build is passing again. I've copied the contents of one of the error logs from the jenkins workspace to the following gist in case anyone wants to follow up:

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

No branches or pull requests

2 participants