BUILD for aarch64 : Suggested changes#100
BUILD for aarch64 : Suggested changes#100rahulthakre29 wants to merge 3 commits intoeclipse-score:mainfrom
Conversation
|
The created documentation from the pull request is available at: docu-html |
346bf99 to
79047c4
Compare
79047c4 to
b45bfdb
Compare
b45bfdb to
f8d4bb8
Compare
f8d4bb8 to
7c5e445
Compare
ffb641c to
9d7698f
Compare
9d7698f to
4446685
Compare
4446685 to
c8d2343
Compare
c8d2343 to
94efb57
Compare
BUILD
Outdated
| # Platform-aware library aliases for backward compatibility | ||
| alias( | ||
| name = "acl", | ||
| actual = "//third_party/acl:acl", | ||
| visibility = ["//visibility:public"], | ||
| ) | ||
|
|
||
| alias( | ||
| name = "libcap2", | ||
| actual = "//third_party/libcap2:libcap2", | ||
| visibility = ["//visibility:public"], | ||
| ) | ||
|
|
||
| alias( | ||
| name = "libcap2-dev", | ||
| actual = "//third_party/libcap2:libcap2", | ||
| visibility = ["//visibility:public"], | ||
| ) | ||
|
|
||
| alias( | ||
| name = "libseccomp2", | ||
| actual = "//third_party/libseccomp2:libseccomp2", | ||
| visibility = ["//visibility:public"], | ||
| ) | ||
|
|
||
| alias( | ||
| name = "libseccomp2-dev", | ||
| actual = "//third_party/libseccomp2:libseccomp2", | ||
| visibility = ["//visibility:public"], | ||
| ) | ||
|
|
||
| alias( | ||
| name = "valgrind", | ||
| actual = "//third_party/valgrind:valgrind", | ||
| visibility = ["//visibility:public"], | ||
| ) |
There was a problem hiding this comment.
There are already aliases and cc_library targets in //third-party/... Let's not have one more indirection
score/concurrency/BUILD
Outdated
| deps = select({ | ||
| "@platforms//os:qnx": [], | ||
| "//conditions:default": ["//third_party/valgrind"], | ||
| "//conditions:default": ["//:valgrind"], |
There was a problem hiding this comment.
This change is not needed. Having it refer to //third_party/valgrind is perfectly fine. Moreover, by not having this change, we'll avoid change sin our internal codebase.
score/os/linux/utils/BUILD
Outdated
| ], | ||
| deps = [ | ||
| "//third_party/libseccomp2", | ||
| "//:libseccomp2-dev", |
There was a problem hiding this comment.
Same as in the valgrind case above. Please leave deps to //third_party/... without change. Same applies to all other similar cases in this PR.
third_party/acl/acl.BUILD
Outdated
| "@platforms//cpu:aarch64": ["lib/aarch64-linux-gnu/libacl.a"], | ||
| "//conditions:default": ["lib/x86_64-linux-gnu/libacl.a"], |
There was a problem hiding this comment.
lib/x86_64-linux-gnu/libacl.ais a symlink in deb archive tousr/lib/x86_64-linux-gnu/libacl.a. I would avoid extra indirection, i.e. keepusr/lib- In case of aarch64, there is no such symlink, so the build will likely to fail, i.e. add
usr/in front
You can validate AArch64 builds locally with bazel build --config=bl-aarch64-linux //score/...
94efb57 to
2d62379
Compare
2d62379 to
da89fdc
Compare
|
|
||
| TEST(StdIntegralTypes, signess_and_bytes_number) | ||
| { | ||
| // Test identifies implementation defined behavior. |
There was a problem hiding this comment.
This change is not needed anymore. With last sync, it's already fixed.
- fix missing BUILD parameters for AArch64 - Correct file paths for static libraries in dev packages - Consolidate architecture specific Build file using select()
a5b7196 to
5194c4c
Compare
Suggested Changes to build aarch64 on Baselib