eclipse-iceoryx / eclipse-iceoryx/iceoryx
Default `--//:feature_acl=off`
- Dominant language
- C++
- Stars
- 2.2k
- Forks
- 492
- Avg merge
- 18h 57m
- Merged PRs (30d)
- 1
Description
## Brief feature description
The iceoryx Bazel build expects libacl to be available from the host on linux systems.
This assumption does not always hold and breaks hermeticity.
This line should default to `off` instead of `auto` (or, default to `off` and remove `auto`):
https://github.com/eclipse-iceoryx/iceoryx/blob/7592b2adbf425d04a06d24571711e1a19a977d92/BUILD.bazel#L27
This is already done in the iceoryx BCR module: https://github.com/bazelbuild/bazel-central-registry/blob/main/modules/iceoryx/2.95.3.bcr.1/patches/build.patch and it would be better to just upstream this patch.
Contributor guide
Research direction
Inspect BUILD.bazel at line 27 and compare its feature_acl default with the existing iceoryx BCR build.patch. The work is done when the upstream Bazel configuration matches the patch by defaulting feature_acl to off, without relying on host libacl on Linux.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- build-system
- Issue type
- Bug
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 35/100