Skip to content

[spack mpd] Fix more assert on assignment causing parenthesis warning. Use CMake variables that exist to define test environment vars. - #37

Open
gartung wants to merge 2 commits into
developfrom
mpd-build-2
Open

[spack mpd] Fix more assert on assignment causing parenthesis warning. Use CMake variables that exist to define test environment vars.#37
gartung wants to merge 2 commits into
developfrom
mpd-build-2

Conversation

@gartung

@gartung gartung commented Jul 20, 2026

Copy link
Copy Markdown
Member

No description provided.

@gartung
gartung requested a review from tomjunk July 20, 2026 16:15
@FNALbuild

FNALbuild commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard

@FNALbuild

FNALbuild commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard

@FNALbuild

FNALbuild commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

✔️ CI build for DUNE Succeeded on slf7 for c14:prof -- details available through the CI dashboard

parent CI build details are available through the CI dashboard

@FNALbuild

FNALbuild commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

⚠️ CI build for DUNE Warning at phase ci_tests DUNE on slf7 for e26:prof -- details available through the CI dashboard

🚨 For more details about the warning phase, check the ci_tests DUNE phase logs

parent CI build details are available through the CI dashboard

@gartung

gartung commented Jul 29, 2026 •
edited
Loading

Copy link
Copy Markdown
Member Author

added changes from #39

@gartung gartung closed this Jul 29, 2026
@gartung gartung reopened this Jul 30, 2026
@gartung gartung changed the title Fix more assert on assignment causing parenthesis warning [spack mpd] Fix more assert on assignment causing parenthesis warning. Use CMake variables that exist to define test environment vars. Jul 30, 2026
@FNALbuild

FNALbuild commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard

@FNALbuild

FNALbuild commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard

@gartung
gartung requested a review from leoaliaga July 30, 2026 15:26
@FNALbuild

FNALbuild commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

✔️ CI build for DUNE Succeeded on slf7 for c14:prof -- details available through the CI dashboard

parent CI build details are available through the CI dashboard

@FNALbuild

FNALbuild commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

⚠️ CI build for DUNE Warning at phase ci_tests DUNE on slf7 for e26:prof -- details available through the CI dashboard

🚨 For more details about the warning phase, check the ci_tests DUNE phase logs

parent CI build details are available through the CI dashboard

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

Reviewers

@tomjunk tomjunk Awaiting requested review from tomjunk
@leoaliaga leoaliaga Awaiting requested review from leoaliaga

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@gartung @FNALbuild