Skip to content

fix source URL in git-annex easyconfigs#25426

Merged
boegel merged 5 commits intoeasybuilders:developfrom
pavelToman:patch-30
Mar 9, 2026
Merged

fix source URL in git-annex easyconfigs#25426
boegel merged 5 commits intoeasybuilders:developfrom
pavelToman:patch-30

Conversation

@pavelToman
Copy link
Copy Markdown
Collaborator

@pavelToman pavelToman commented Feb 25, 2026 •
edited
Loading

On some systems you are not able to git clone from git://...git.
Now the sources are fetched from different official git repository that has https://...git available.

@github-actions github-actions Bot added 2022b 2023a 2024a issues & PRs related to 2024a common toolchains labels Feb 25, 2026
@pavelToman
Copy link
Copy Markdown
Collaborator Author

pavelToman commented Feb 25, 2026

@boegelbot please test @ jsc-zen3

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Feb 25, 2026

@pavelToman: 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=25426 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_25426 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 3959437322 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).

jfgrimm
jfgrimm previously requested changes Feb 25, 2026
View reviewed changes
Comment thread easybuild/easyconfigs/g/git-annex/git-annex-10.20230802-GCCcore-12.2.0.eb Outdated
Comment on lines +15 to +16
# main source https://git.joeyh.name/git/git-annex
# alternative source https://github.com/datalad/git-annex
Copy link
Copy Markdown
Member

@jfgrimm jfgrimm Feb 25, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

this is not correct, the official main source is:

The main git repository for git-annex is git://git-annex.branchable.com/ [gitweb]
(You can push changes to this wiki from that anonymous git checkout.)
Other mirrors of the git repository:
git://git.joeyh.name/git-annex or https://git.joeyh.name/git/git-annex.git [gitweb]

Copy link
Copy Markdown
Member

@jfgrimm jfgrimm Feb 25, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

besides, you could just switch it to https://git-annex.branchable.com
edit: never mind, that doesn't work :D

Copy link
Copy Markdown
Collaborator Author

@pavelToman pavelToman Feb 25, 2026

Choose a reason for hiding this comment

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

yes, https://git-annex.branchable.com does not work :/ so I go with https://git.joeyh.name/git/git-annex instead

Copy link
Copy Markdown
Member

@boegel boegel Mar 9, 2026

Choose a reason for hiding this comment

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

No need to mention this alternative if we're not going to use it...

I'll clean up the comment and get this merged

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Feb 25, 2026

Test report by @boegelbot
SUCCESS
Build succeeded for 4 out of 4 (total: 1 hour 31 mins 33 secs) (4 easyconfigs in total)
jsczen3c3.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.7, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.23
See https://gist.github.com/boegelbot/b6ee332fcfd700ca488d32c693410825 for a full test report.

@pavelToman
Copy link
Copy Markdown
Collaborator Author

pavelToman commented Feb 25, 2026

Test report by @pavelToman
SUCCESS
Build succeeded for 16 out of 16 (total: 2 hours 16 mins 21 secs) (4 easyconfigs in total)
node4216.shinx.os - Linux RHEL 9.6, x86_64, AMD EPYC 9654 96-Core Processor, Python 3.9.21
See https://gist.github.com/pavelToman/954486c38cee72cc0a8e63375fa8497f for a full test report.

@boegel boegel self-assigned this Mar 9, 2026
@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 9, 2026

@boegelbot please test @ jsc-zen3
EB_ARGS="--force-download --sourcepath /tmp/$USER/pr25426 --installpath /tmp/$USER/pr25426"

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Mar 9, 2026

@boegel: 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=25426 EB_ARGS="--force-download --sourcepath /tmp/$USER/pr25426 --installpath /tmp/$USER/pr25426" EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_25426 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 4025778525 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 Mar 9, 2026

Test report by @boegelbot
SUCCESS
Build succeeded for 4 out of 4 (total: 1 hour 39 mins 20 secs) (4 easyconfigs in total)
jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.7, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.25
See https://gist.github.com/boegelbot/25701756ddf356170dfe30bde948412b for a full test report.

@boegel boegel added this to the next release (5.2.2?) milestone Mar 9, 2026
boegel
boegel approved these changes Mar 9, 2026
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 dismissed jfgrimm’s stale review March 9, 2026 21:41

requested changes made

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 9, 2026

Going in, thanks @pavelToman!

@boegel boegel merged commit aa6ce39 into easybuilders:develop Mar 9, 2026
6 checks passed
@boegel boegel added bug fix and removed change labels Mar 9, 2026
@boegel boegel changed the title Update git-annex easyconfigs source fix source URL in git-annex easyconfigs Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jfgrimm jfgrimm jfgrimm left review comments

@boegel boegel boegel approved these changes

Assignees

@boegel boegel

Labels

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

Projects

None yet

Milestone

5.3.0

Development

Successfully merging this pull request may close these issues.

4 participants

@pavelToman @boegelbot @boegel @jfgrimm