Skip to content

Conversation

@julianmorillo
Copy link
Contributor

@julianmorillo julianmorillo commented Jun 5, 2025

This constant value is returned by get_cpu_architecture, which is called from get_arch_prefix, that returns it directly (lower case) to _get_gcc_prefix, causing the following error when trying to build LLVM in RISCV:

== FAILED: Installation ended unsuccessfully: Can't find GCC version 13.2.0 for architecture risc-v-64 in
/cvmfs/riscv.eessi.io/versions/20240402/software/linux/riscv64/generic/software/GCCcore/13.2.0/lib/gcc/risc-v-64-*/13.2.0 (took 1 min 25 secs)
== Results of the build can be found in the log file(s) /tmp/eb-0kc4ln6r/easybuild-LLVM-20.1.5-20250605.161517.VhpnA.log

ERROR: Installation of LLVM-20.1.5-GCCcore-13.2.0.eb failed: "Can't find GCC version 13.2.0 for architecture risc-v-64 in /cvmfs/riscv.eessi.io/versions/20240402/software/linux/riscv64/generic/software/GCCcore/13.2.0/lib/gcc/risc-v-64-*/13.2.0"

The correct path in my case would be /cvmfs/riscv.eessi.io/versions/20240402/software/linux/riscv64/generic/software/GCCcore/13.2.0/lib/gcc/riscv64-*/13.2.0

@boegel boegel added the change label Jun 17, 2025
@boegel boegel added this to the next release (5.1.1?) milestone Jun 17, 2025
@boegel boegel added the riscv label Jun 17, 2025
Copy link
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel boegel enabled auto-merge July 2, 2025 14:52
@boegel boegel changed the title Change the architecture constant RISCV64 Change the architectures constant RISCV32 and RISCV64 Jul 2, 2025
@boegel boegel changed the title Change the architectures constant RISCV32 and RISCV64 Change value of architectures constants RISCV32 and RISCV64 Jul 2, 2025
@boegel boegel merged commit 87eabdd into easybuilders:develop Jul 2, 2025
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants