Skip to content

[ALICE3] Update lutmaker#15001

Open
njacazio wants to merge 2 commits intomasterfrom
njacazio-patch-7
Open

[ALICE3] Update lutmaker#15001
njacazio wants to merge 2 commits intomasterfrom
njacazio-patch-7

Conversation

@njacazio
Copy link
Collaborator

@njacazio njacazio commented Feb 17, 2026

No description provided.

@njacazio njacazio marked this pull request as ready for review February 17, 2026 09:20
@github-actions github-actions bot changed the title Update lutmaker [ALICE3] Update lutmaker Feb 17, 2026
@njacazio njacazio enabled auto-merge (squash) February 17, 2026 09:20
alibuild
alibuild previously approved these changes Feb 17, 2026
View reviewed changes
Copy link
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

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

Auto-approving on behalf of @njacazio.

@alibuild
Copy link
Collaborator

alibuild commented Feb 17, 2026

Error while checking build/O2Physics/o2 for 0d430eb at 2026-02-17 10:28:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/15001-slc9_x86-64/0/ALICE3/Tasks/alice3-lutmaker.cxx:328:45: error: unused parameter 'mcCollisions' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

@njacazio njacazio disabled auto-merge February 17, 2026 10:04
@njacazio njacazio enabled auto-merge (squash) February 17, 2026 10:04
alibuild
alibuild approved these changes Feb 17, 2026
View reviewed changes
Copy link
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

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

Auto-approving on behalf of @njacazio.

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

Reviewers

@alibuild alibuild alibuild approved these changes

@hscheid hscheid Awaiting requested review from hscheid hscheid is a code owner

Assignees

No one assigned

Labels

alice3

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@njacazio @alibuild