eclipse-iceoryx / eclipse-iceoryx/iceoryx

Default `--//:feature_acl=off`

Open
#2,434 6 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.