Skip to content

remove unneccessary env variable which has nothing to do with mcpl#24476

Merged
Micket merged 1 commit intoeasybuilders:developfrom
ebknudsen:20251106172534_new_pr_mcpl220
Nov 7, 2025
Merged

remove unneccessary env variable which has nothing to do with mcpl#24476
Micket merged 1 commit intoeasybuilders:developfrom
ebknudsen:20251106172534_new_pr_mcpl220

Conversation

@ebknudsen
Copy link
Copy Markdown
Contributor

@ebknudsen ebknudsen commented Nov 6, 2025

By a mistake the easyconfig sets the environment variable MCXTRACE which originates from a different easyconfig (McXtrace which can use mcpl). This PR removes that line.

@github-actions github-actions Bot added 2023a 2024a issues & PRs related to 2024a common toolchains change labels Nov 6, 2025
Micket
Micket approved these changes Nov 7, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

lgtm

@Micket
Copy link
Copy Markdown
Contributor

Micket commented Nov 7, 2025

@boegelbot please test @ jsc-zen3

@Micket Micket added this to the next release (5.2.0?) milestone Nov 7, 2025
@Micket Micket added the bug fix label Nov 7, 2025
@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Nov 7, 2025

@Micket: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de

PR test command 'if [[ develop != 'develop' ]]; then EB_BRANCH=develop ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/develop source init_env_easybuild_develop.sh; fi; EB_PR=24476 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_24476 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 8672

Test results coming soon (I hope)...

Details

- notification for comment with ID 3501392485 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Nov 7, 2025

Test report by @boegelbot
SUCCESS
Build succeeded for 2 out of 2 (total: 5 mins 17 secs) (2 easyconfigs in total)
jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.6, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21
See https://gist.github.com/boegelbot/bf10eb8dffd8eaf89ea4ba32e07941ab for a full test report.

@Micket Micket merged commit a6e4674 into easybuilders:develop Nov 7, 2025
8 checks passed
@boegel boegel removed the change label Dec 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Micket Micket Micket approved these changes

Assignees

No one assigned

Labels

2023a 2024a issues & PRs related to 2024a common toolchains bug fix

Projects

None yet

Milestone

5.2.0

Development

Successfully merging this pull request may close these issues.

4 participants

@ebknudsen @Micket @boegelbot @boegel