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

Only link against bfshell_plugin.so and enable pi for SDE 9.7.0 build #859

Merged
merged 3 commits into from Dec 8, 2021

Conversation

Yi-Tseng
Copy link
Contributor

@Yi-Tseng Yi-Tseng commented Dec 8, 2021

"barefoot-bin/lib/bfshell_plugin*", can include some unexpected files when building with older version of SDEs which cause the build failed.

ERROR: /root/.cache/bazel/_bazel_root/1b07b1db06ec9ee22c71eac6f0bd4549/external/local_barefoot_bin/BUILD:13:11: in srcs attribute of cc_library rule @local_barefoot_bin//:bfsde: source file '@local_barefoot_bin//:barefoot-bin/lib/bfshell_plugin_bf_rt.la' is misplaced here (expected .cc, .cpp, .cxx, .c++, .C, .cu, .cl, .c, .h, .hh, .hpp, .ipp, .hxx, .h++, .inc, .inl, .tlh, .tli, .H, .tcc, .S, .s, .asm, .a, .lib, .pic.a, .lo, .lo.lib, .pic.lo, .so, .dylib, .dll, .o, .obj or .pic.o). Since this rule was created by the macro 'cc_library', the error might have been caused by the macro implementation

@codecov
Copy link

codecov bot commented Dec 8, 2021

Codecov Report

Merging #859 (f3a7dac) into main (7059d58) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #859   +/-   ##
=======================================
  Coverage   78.56%   78.56%           
=======================================
  Files         334      334           
  Lines       30057    30057           
=======================================
  Hits        23613    23613           
  Misses       6444     6444           

@Yi-Tseng Yi-Tseng added this to the 2021-12 Release milestone Dec 8, 2021
@Yi-Tseng Yi-Tseng added Infra Things related to CI/CD, build, and tests P1 labels Dec 8, 2021
@Yi-Tseng
Copy link
Contributor Author

Yi-Tseng commented Dec 8, 2021

Testing now with Stratum CI....

pudelkoM
pudelkoM previously approved these changes Dec 8, 2021
Copy link
Member

@pudelkoM pudelkoM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Do you have the errors somewhere? Would be good for future reference.

@Yi-Tseng
Copy link
Contributor Author

Yi-Tseng commented Dec 8, 2021

Do you have the errors somewhere? Would be good for future reference.

Yes, you can find the error here:
https://jenkins.stratumproject.org/view/Stratum%20Nightly%20Builds/job/stratum-bfrt-9.5.0/190/console

ERROR: /root/.cache/bazel/_bazel_root/1b07b1db06ec9ee22c71eac6f0bd4549/external/local_barefoot_bin/BUILD:13:11: in srcs attribute of cc_library rule @local_barefoot_bin//:bfsde: source file '@local_barefoot_bin//:barefoot-bin/lib/bfshell_plugin_bf_rt.la' is misplaced here (expected .cc, .cpp, .cxx, .c++, .C, .cu, .cl, .c, .h, .hh, .hpp, .ipp, .hxx, .h++, .inc, .inl, .tlh, .tli, .H, .tcc, .S, .s, .asm, .a, .lib, .pic.a, .lo, .lo.lib, .pic.lo, .so, .dylib, .dll, .o, .obj or .pic.o). Since this rule was created by the macro 'cc_library', the error might have been caused by the macro implementation

@Yi-Tseng
Copy link
Contributor Author

Yi-Tseng commented Dec 8, 2021

I also need to update the build flag for SDE 9.7.0 to include pi library for stratum-bf

@Yi-Tseng
Copy link
Contributor Author

Yi-Tseng commented Dec 8, 2021

All build pass, should be good to go
https://jenkins.stratumproject.org/view/Stratum%20Nightly%20Builds/

@pudelkoM pudelkoM changed the title Fix Stratum build for older SDE Only link against bfshell_plugin.so and enable pi for SDE 9.7.0 build Dec 8, 2021
@pudelkoM pudelkoM merged commit 2832b4e into main Dec 8, 2021
@pudelkoM pudelkoM deleted the fix-sde-build branch December 8, 2021 19:52
bocon13 added a commit that referenced this pull request Dec 10, 2021
- Partially revert #859
- Don't include PI in 9.7.0 build
- Skip stratum_bf for SDE 9.7.0 in release script
bocon13 added a commit that referenced this pull request Dec 10, 2021
- Partially revert #859
- Don't include PI in 9.7.0 build
- Skip stratum_bf for SDE 9.7.0 in release script
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Infra Things related to CI/CD, build, and tests P1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants