Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

{perf}[gompi/2022a,gompi/2023b] CubeGUI 4.8.2 & Scalasca 2.6.1 for 2023b | Patch to fix configure after OpenMP #21135

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Thyre
Copy link
Contributor

@Thyre Thyre commented Aug 6, 2024

This PR adds Scalasca to 2023b, using the latest versions of Cube[x], OTF2 and Score-P.

In addition, a patch was added, which is being used on the JSC systems for some time (see here). This patch fixes a typo, which causes all checks after the OpenMP check to fail when they produce any output on stdout or stderr. This can cause issues in detecting compiler flags or the environment correctly.

@Thyre Thyre force-pushed the scalasca-2.6.1-nowarn-omp-pragmas branch from d19b40a to 86bd82f Compare August 16, 2024 14:31
@Thyre Thyre marked this pull request as draft August 26, 2024 08:10
@Thyre
Copy link
Contributor Author

Thyre commented Aug 26, 2024

For gompi/2023b CubeGUI 4.8.2 is missing. I'll add it to this PR. Marked as Draft until then.

Signed-off-by: Jan André Reuter <j.reuter@fz-juelich.de>
@Thyre Thyre force-pushed the scalasca-2.6.1-nowarn-omp-pragmas branch from 86bd82f to 16a200f Compare August 26, 2024 09:43
@Thyre Thyre changed the title {perf}[gompi/2022a,gompi/2023b] Scalasca 2.6.1 for 2023b, patch to fix configure after OpenMP {perf}[gompi/2022a,gompi/2023b] CubeGUI 4.8.2 & Scalasca 2.6.1 for 2023b | Patch to fix configure after OpenMP Aug 26, 2024
@Thyre Thyre marked this pull request as ready for review August 26, 2024 14:05
Signed-off-by: Jan André Reuter <j.reuter@fz-juelich.de>
@Thyre Thyre force-pushed the scalasca-2.6.1-nowarn-omp-pragmas branch from 16a200f to 42baaa1 Compare August 29, 2024 14:58
]

configopts = [
'--with-qt=$EBROOTQT5/bin ',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
'--with-qt=$EBROOTQT5/bin ',
'--with-qt=$EBROOTQT6/bin ',

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant