Skip to content

Stop depending on savannah cgit#526

Merged
jian-lin merged 3 commits intonix-community:masterfrom
linj-fork:pr/atom->git-ls-remote
Dec 15, 2025
Merged

Stop depending on savannah cgit#526
jian-lin merged 3 commits intonix-community:masterfrom
linj-fork:pr/atom->git-ls-remote

Conversation

@jian-lin
Copy link
Copy Markdown
Member

  • Update Emacs repo url on savannah
  • Replace git:// with https:// for Emacs savannah url
  • Stop depending on savannah cgit

@acid-bong
Copy link
Copy Markdown
Contributor

related to this error message? was just looking for related issues

@jian-lin
Copy link
Copy Markdown
Member Author

jian-lin commented Dec 14, 2025

@acid-bong Yes, that error was fixed in #524

@akirak
Copy link
Copy Markdown

akirak commented Dec 14, 2025

FYI, I have been reliably mirroring savannah repositories from cgit over the git protocol using this workflow: https://github.com/elpa-mirrors/automation/blob/master/.github/workflows/mirror.yml. Security doesn't look like an issue to the user.

@jian-lin
Copy link
Copy Markdown
Member Author

@akirak Sorry, I am a bit confused what you are conveying here. Could you make it more explicit?

@akirak
Copy link
Copy Markdown

akirak commented Dec 14, 2025

@jian-lin No, it's fine. I meant cloning from a git url like git://cgit.git.savannah.gnu.org/emacs/elpa.git/ could be reliable if it directly accesses cgit, but that wouldn't be necessary. Sorry for bothering you.

@jian-lin
Copy link
Copy Markdown
Member Author

@akirak I see, thanks.

BTW, it seems this url git://cgit.git.savannah.gnu.org/emacs/elpa.git/ is not listed anywhere now. On their cgit web page, they list:

akirak added a commit to elpa-mirrors/automation that referenced this pull request Dec 14, 2025
It seems that they have changed the official clone URLs.

See the comment by LIN, Jian (@jian-lin):
nix-community/emacs-overlay#526 (comment)
Savannah disables their cgit snapshot service again.  We have
migrated[1] away from savannah cgit snapshot.  To be more
future-proofing, this patch removes our last dependency of savannah
cgit.

The last dependency of savannah cgit is to fetch atom feeds from it to
get the latest commit and the date of the latest commit.  Now we get
those info using "git ls-remote" and "nix-prefetch-git".

[1]: 8b221ef
@jian-lin jian-lin force-pushed the pr/atom->git-ls-remote branch from 1b68102 to 9429698 Compare December 15, 2025 05:43
@jian-lin
Copy link
Copy Markdown
Member Author

resolved conflicts

@jian-lin jian-lin merged commit 70e1e41 into nix-community:master Dec 15, 2025
1 check passed
@jian-lin jian-lin deleted the pr/atom->git-ls-remote branch December 15, 2025 05:44
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