Skip to content

Allow setting the URL for config.sub #205

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tgross35
Copy link

@tgross35 tgross35 commented Jul 3, 2025

Unfortunately the Savannah server can be a bit flaky. Move the config URL to a variable so it can be set to point to a mirror.

Unfortunately the Savannah server can be a bit flaky. Move the config
URL to a variable so it can be set to point to a mirror.

Signed-off-by: Trevor Gross <[email protected]>
@tgross35
Copy link
Author

tgross35 commented Jul 3, 2025

For context, we use this makefile in rust-lang/rust ci and have had some intermittent failures from Savannah.

@userdocs
Copy link
Contributor

userdocs commented Jul 13, 2025

The Savannah git is regularly unreachable and the https://ftpmirror.gnu.org/gnu also goes offline periodically.

I don't understand what this achieves because as far as I understand it, there is no public alternative URL to use here.

All mirrors points to a readme - https://mirror.cyberbits.eu/gnu/config/ with this content

From now on, please fetch new versions of these scripts via the
following URLs:

http://git.savannah.gnu.org/gitweb/?p=config.git;a=blob_plain;f=config.guess;hb=HEAD
http://git.savannah.gnu.org/gitweb/?p=config.git;a=blob_plain;f=config.sub;hb=HEAD

Thanks,

Ben Elliston [email protected]

And there are no up to date git mirrors of the config repo that I saw.

So is the URL you used a self hosted / mirrored repo of config? If it is maybe just having point to a local file is the better option?

I don't argue a change here is helpful it's just I've come to the conclusion that if the url is down, just fall back to the last good version of the config.sub and have it stored in the repo itself.

@tgross35
Copy link
Author

That's basically correct. The "mirror" could just be your fork of the config repo on GH and then passing the raw link as CONFIG_SUB_URL with the right version, e.g. https://raw.githubusercontent.com/richfelker/musl-cross-make/6f3701d08137496d5aac479e3a3977b5ae993c1f/Makefile for this repo's makefile.

There are certainly other options like keeping the file, but in our case we mirror everything locally for CI anyway so most of that is set up.

@richfelker
Copy link
Owner

I wonder if we should just drop replacing config.sub except for really old tool versions. I don't think it should be necessary for anything remotely modern. We could just have a make variable to enable replacing it, that's off by default.

@userdocs
Copy link
Contributor

I think that makes sense. Only enable it if there is a problem, which solve the immediate problem of the host being so unreliable. The only option otherwise is some type of self hosting.

But, I think even if you did make this change, still allow users to set the URL as it still allows the self hosting option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants