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

syncd build failure on broadcom. #1062

Closed
lguohan opened this issue Jun 10, 2022 · 1 comment · Fixed by #1063
Closed

syncd build failure on broadcom. #1062

lguohan opened this issue Jun 10, 2022 · 1 comment · Fixed by #1063

Comments

@lguohan
Copy link
Contributor

lguohan commented Jun 10, 2022

On master branch, in local test I have met with one below issue and not able to advance submodule sonic-sairedis. It causes the failure of syncd build for platform broadcom.

configure: libsai found
checking for swig3.0... /usr/bin/swig3.0
checking whether CXX supports -Wcast-align=strict... yes
checking whether CXX supports -Wno-cast-function-type... yes
checking SAI headers API version and library version check... failed
configure: error: "SAI headers API version and library version mismatch"
make[1]: *** [Makefile:405: config.status] Error 1
make[1]: Leaving directory '/sonic/src/sonic-sairedis'
make: *** [Makefile:594: ../aclocal.m4] Error 2

junhuazhai@e8ea45ebfbd4:/sonic/src/sonic-sairedis/syncd$ dpkg -l | grep saibcm
ii libsaibcm 7.1.0.0-1 amd64 Switch Abstraction Interface sdk based on Broadcom SAI
ii libsaibcm-dev 7.1.0.0-1 amd64 development files for Switch Abstraction Interface sdk

This issue is due to [[configure.ac] implement SAI API version check by #1000 The API sai_query. _api_version is new and has not been implemented in latest BRCM SAI. This PR only has the API implementation for vs SAI lib. Not sure whether the API has been included in other vendor SAI libs.

@lguohan
Copy link
Contributor Author

lguohan commented Jun 10, 2022

in the configure.ac, can you first check if the function exist or not, if not, then do not check.

lguohan added a commit that referenced this issue Jun 11, 2022
Fix #1062

Signed-off-by: Guohan Lu <lguohan@gmail.com>
pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this issue Nov 18, 2022
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 a pull request may close this issue.

1 participant