Skip to content

Depa ipc tracing feature flag usage - #1094

Open
devendrapat wants to merge 3 commits into
eclipse-score:mainfrom
devendrapat:depa_ipc_tracing_feature_flag_usage
Open

devendrapat wants to merge 3 commits into
eclipse-score:mainfrom
devendrapat:depa_ipc_tracing_feature_flag_usage

Conversation

@devendrapat

Copy link
Copy Markdown

Keeping the changes in this PR (https://github.com/eclipse-score/communication/pull/1022/changes) intact.
This PR also updates the version of score_baselibs for usage of required_typed_memory flag.

crimson11 and others added 2 commits September 16, 2026 10:57
To decide, whether typed-mem should
be used in case of IPC-tracing, now
the corresponding feature flag is
consulted.
Signed-off-by: devendrapat <76650287+devendrapat@users.noreply.github.com>
@devendrapat
devendrapat marked this pull request as ready for review September 16, 2026 05:32
Comment thread MODULE.bazel
bazel_dep(name = "score_baselibs", version = "0.2.12")
single_version_override(
module_name = "score_baselibs",
commit = "4abf22407106ec2461f5f80a619cd15b5d294238",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This would require all our customer to pin this exact version. Is a new release of baselibs planned? What is the ticket to track that? In general we only want to use release versions of baselibs.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I created #1102 that bumps baselibs to the new released version. The new version contain the changes you need, you should just remove this check once the mentioned PR is merged.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Currently we cannot bump because of eclipse-score/baselibs#527 (comment)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Bump is done, you should be able to continue discarding the bump changes

@github-actions

github-actions Bot commented Oct 3, 2026

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. If this PR is still relevant, please leave a comment or push new changes to keep it open.

@github-actions github-actions Bot added stale and removed stale labels Oct 3, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

3 participants