Skip to content

BUILD for aarch64 : Suggested changes#100

Draft
rahulthakre29 wants to merge 3 commits intoeclipse-score:mainfrom
rahulthakre29:rt/investigating_build_baselib_on_AArch64_Linux
Draft

BUILD for aarch64 : Suggested changes#100
rahulthakre29 wants to merge 3 commits intoeclipse-score:mainfrom
rahulthakre29:rt/investigating_build_baselib_on_AArch64_Linux

Conversation

@rahulthakre29
Copy link
Contributor

Suggested Changes to build aarch64 on Baselib

@github-actions
Copy link

The created documentation from the pull request is available at: docu-html

@rahulthakre29 rahulthakre29 force-pushed the rt/investigating_build_baselib_on_AArch64_Linux branch from 346bf99 to 79047c4 Compare February 10, 2026 10:09
@rahulthakre29 rahulthakre29 force-pushed the rt/investigating_build_baselib_on_AArch64_Linux branch from 79047c4 to b45bfdb Compare February 10, 2026 10:15
@rahulthakre29 rahulthakre29 force-pushed the rt/investigating_build_baselib_on_AArch64_Linux branch from b45bfdb to f8d4bb8 Compare February 10, 2026 10:25
@rahulthakre29 rahulthakre29 force-pushed the rt/investigating_build_baselib_on_AArch64_Linux branch from f8d4bb8 to 7c5e445 Compare February 10, 2026 11:01
@rahulthakre29 rahulthakre29 force-pushed the rt/investigating_build_baselib_on_AArch64_Linux branch from ffb641c to 9d7698f Compare February 12, 2026 12:07
@rahulthakre29 rahulthakre29 force-pushed the rt/investigating_build_baselib_on_AArch64_Linux branch from 9d7698f to 4446685 Compare February 17, 2026 06:01
@rahulthakre29 rahulthakre29 force-pushed the rt/investigating_build_baselib_on_AArch64_Linux branch from 4446685 to c8d2343 Compare February 18, 2026 05:27
@rahulthakre29 rahulthakre29 force-pushed the rt/investigating_build_baselib_on_AArch64_Linux branch from c8d2343 to 94efb57 Compare February 18, 2026 10:05
BUILD Outdated
Comment on lines 43 to 78
# 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"],
)
Copy link
Member

Choose a reason for hiding this comment

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

There are already aliases and cc_library targets in //third-party/... Let's not have one more indirection

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

deps = select({
"@platforms//os:qnx": [],
"//conditions:default": ["//third_party/valgrind"],
"//conditions:default": ["//:valgrind"],
Copy link
Member

Choose a reason for hiding this comment

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

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

],
deps = [
"//third_party/libseccomp2",
"//:libseccomp2-dev",
Copy link
Member

Choose a reason for hiding this comment

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

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Comment on lines 4 to 5
"@platforms//cpu:aarch64": ["lib/aarch64-linux-gnu/libacl.a"],
"//conditions:default": ["lib/x86_64-linux-gnu/libacl.a"],
Copy link
Member

Choose a reason for hiding this comment

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

  • lib/x86_64-linux-gnu/libacl.a is a symlink in deb archive to usr/lib/x86_64-linux-gnu/libacl.a. I would avoid extra indirection, i.e. keep usr/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/...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@rahulthakre29 rahulthakre29 force-pushed the rt/investigating_build_baselib_on_AArch64_Linux branch from 94efb57 to 2d62379 Compare February 23, 2026 08:34
@rahulthakre29 rahulthakre29 force-pushed the rt/investigating_build_baselib_on_AArch64_Linux branch from 2d62379 to da89fdc Compare February 23, 2026 08:42

TEST(StdIntegralTypes, signess_and_bytes_number)
{
// Test identifies implementation defined behavior.
Copy link
Member

Choose a reason for hiding this comment

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

This change is not needed anymore. With last sync, it's already fixed.

@4og 4og self-requested a review February 26, 2026 08:48
- fix missing BUILD parameters for AArch64
- Correct file paths for static libraries in dev packages
- Consolidate architecture specific Build file using select()
@rahulthakre29 rahulthakre29 force-pushed the rt/investigating_build_baselib_on_AArch64_Linux branch from a5b7196 to 5194c4c Compare February 27, 2026 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants