Skip to content

only consider stdout when checking config.guess in ConfigureMake easyblock#4020

Merged
boegel merged 1 commit intoeasybuilders:developfrom
Thyre:20251214020431_new_pr_configuremake
Dec 15, 2025
Merged

only consider stdout when checking config.guess in ConfigureMake easyblock#4020
boegel merged 1 commit intoeasybuilders:developfrom
Thyre:20251214020431_new_pr_configuremake

Conversation

@Thyre
Copy link
Copy Markdown
Collaborator

@Thyre Thyre commented Dec 14, 2025 •
edited
Loading

(created using eb --new-pr)

Avoids bogus configure options due to warnings coming from system libraries:

>> running shell command:
        ./configure --prefix=/data/EasyBuild-develop/software/GHC/9.12.2  --build=/usr/bin/bash: /data/EasyBuild-develop/software/GHC/9.12.2/lib/libncursesw.so.6: no version information available (required by /usr/lib/libreadline.so.8)
/usr/bin/lua: /data/EasyBuild-develop/software/GHC/9.12.2/lib/libncursesw.so.6: no version information available (required by /usr/lib/libreadline.so.8)
/bin/sh: /data/EasyBuild-develop/software/GHC/9.12.2/lib/libncursesw.so.6: no version information available (required by /usr/lib/libreadline.so.8)
x86_64-pc-linux-gnu  --host=/usr/bin/bash: /data/EasyBuild-develop/software/GHC/9.12.2/lib/libncursesw.so.6: no version information available (required by /usr/lib/libreadline.so.8)
/usr/bin/lua: /data/EasyBuild-develop/software/GHC/9.12.2/lib/libncursesw.so.6: no version information available (required by /usr/lib/libreadline.so.8)
/bin/sh: /data/EasyBuild-develop/software/GHC/9.12.2/lib/libncursesw.so.6: no version information available (required by /usr/lib/libreadline.so.8)
x86_64-pc-linux-gnu

Caused by:

eb-shell> ./config.guess
/bin/sh: /data/EasyBuild-develop/software/GHC/9.12.2/lib/libncursesw.so.6: no version information available (required by /usr/lib/libreadline.so.8)
x86_64-pc-linux-gnu

Was seen while working on easybuilders/easybuild-easyconfigs#24867, though this PR isn't necessarily required with upcoming changes done to that PR.

@Thyre Thyre added the bug fix label Dec 14, 2025
@boegel boegel self-assigned this Dec 15, 2025
@boegel boegel changed the title only consider stdout when checking config.guess in configuremake easyblock only consider stdout when checking config.guess in ConfigureMake easyblock Dec 15, 2025
@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 15, 2025

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS zlib-1.3.1-GCCcore-14.3.0.eb

  • SUCCESS libtool-2.4.7-GCCcore-13.3.0.eb

  • SUCCESS file-5.46-GCCcore-14.2.0.eb

  • SUCCESS cURL-8.7.1-GCCcore-13.3.0.eb

  • SUCCESS libxml2-2.14.3-GCCcore-14.3.0.eb

Build succeeded for 5 out of 5 (total: 3 mins 13 secs) (5 easyconfigs in total)
node3535.doduo.os - Linux RHEL 9.6, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.9.21
See https://gist.github.com/boegel/46ab17fd6e266c44affe3a15674a56c6 for a full test report.

@boegel boegel added this to the next release (5.2.0) milestone Dec 15, 2025
boegel
boegel approved these changes Dec 15, 2025
View reviewed changes
Copy link
Copy Markdown
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 merged commit 6ce074c into easybuilders:develop Dec 15, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

Assignees

@boegel boegel

Labels

bug fix

Projects

None yet

Milestone

5.2.0

Development

Successfully merging this pull request may close these issues.

2 participants

@Thyre @boegel