Skip to content

don't run autoreconf for jq 1.7.1, as a release tarball is used as source#24275

Merged
Micket merged 1 commit intoeasybuilders:developfrom
sam-maloney:sam-maloney-jq-1.71.1-GCCcore-13.2.0
Oct 16, 2025
Merged

don't run autoreconf for jq 1.7.1, as a release tarball is used as source#24275
Micket merged 1 commit intoeasybuilders:developfrom
sam-maloney:sam-maloney-jq-1.71.1-GCCcore-13.2.0

Conversation

@sam-maloney
Copy link
Copy Markdown
Contributor

@sam-maloney sam-maloney commented Oct 16, 2025

Bug fix for jq/1.7.1 on GCCcore/13.2.0

Problem: As per the jq build instructions "If you're not using the latest git version but instead building a released tarball (available on the release page), skip the autoreconf step, and flex or bison won't be needed."

Remove the preconfigopts line and Bison and flex from the builddependencies; otherwise, the built binary doesn't actually know what version it is, i.e. jq --version just outputs jq- instead of jq-1.7.1.

Also switch to using SOURCELOWER_TAR_GZ template constant for readability/maintainability

Problem: As per the jq build instructions "If you're not using the latest git version but instead building a released tarball (available on the release page), skip the autoreconf step, and flex or bison won't be needed."

Remove the preconfigopts and Bison and flex builddependencies; otherwise, the built binary doesn't actually know what version it is, i.e. jq --version just outputs jq- instead of jq-1.7.1.

Also switch to using SOURCELOWER_TAR_GZ template constant for readability/maintainability
@sam-maloney sam-maloney changed the title {tools}[GCCcore/13.2.0] jq-1.71.1 {tools}[GCCcore/13.2.0] jq-1.7.1 Oct 16, 2025
@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Oct 16, 2025

@boegelbot please test @ jsc-zen3
EB_ARGS="--installpath=/tmp/$USER/ecpr-24275"

@Thyre Thyre added this to the next release (5.2.0?) milestone Oct 16, 2025
@boegelbot
Copy link
Copy Markdown
Collaborator

@Thyre: 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=24275 EB_ARGS="--installpath=/tmp/$USER/ecpr-24275" EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_24275 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 3410674413 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

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
jsczen3c2.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/23b29aa3635e4d95257e31b6c07f3837 for a full test report.

@PhoenixEmik
Copy link
Copy Markdown
Contributor

Test report by @PhoenixEmik
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
rocky-8-build - Linux Rocky Linux 8.10, x86_64, AMD Ryzen 7 7840HS w/ Radeon 780M Graphics, Python 3.6.8
See https://gist.github.com/PhoenixEmik/fe6f929238c165837c76fc16bd940287 for a full test report.

@Micket Micket merged commit 1226044 into easybuilders:develop Oct 16, 2025
8 checks passed
@sam-maloney sam-maloney deleted the sam-maloney-jq-1.71.1-GCCcore-13.2.0 branch October 17, 2025 03:58
@boegel boegel changed the title {tools}[GCCcore/13.2.0] jq-1.7.1 don't run autoreconf for jq 1.7.1, as a release tarball is used as source Dec 23, 2025
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.

5 participants