Should we switch to emacsmirror/nongnu_elpa rather than Savannah nongnu_elpa?

[milanglacier]

Savannah has recently experienced accessibility issues frequently, possibly due to frequent DDoS attacks. Consequently, we are unable to update packages or bootstrap Emacs.

Should we consider switching from https://git.savannah.gnu.org/git/emacs/nongnu.git to https://github.com/emacsmirror/nongnu_elpa.git in the straight-use-recipes call when defining the nongnu-elpa recipe in bootstrap.el?

[raxod502]

That seems reasonable, yes, if there's reasonable data parity between the two. For backwards compatibility and customizability, I might suggest introducing a new recipe repository name and user option, same as for GNU ELPA:

straight.el/bootstrap.el

Lines 124 to 132 in b89c950

	(if straight-recipes-gnu-elpa-use-mirror
	(straight-use-recipes
	'(gnu-elpa-mirror :type git :host github
	:repo "emacs-straight/gnu-elpa-mirror"
	:build nil))
	(straight-use-recipes `(gnu-elpa :type git
	:repo ,straight-recipes-gnu-elpa-url
	:local-repo "elpa"
	:build nil)))

[milanglacier]

I think a new defcustom for this should work.

I think #1191 is also indirectly related to this issue.

[martenlienen]

Would it be an acceptable first solution to fix the typo in #1147 and merge that? Non-GNU ELPA has had a lot of problems in the past two weeks and then users could switch out the repo URL for https://github.com/emacsmirror/nongnu_elpa or some other mirror. Alternatively, accessing the repo over its git URL git://git.savannah.gnu.org/emacs/nongnu.git also seemed to work fine, compared to the http that is in the code right now.

[raxod502]

Sure. As a local workaround, you can also use straight-recipe-overrides in your init-file before loading straight.el, which lets you override the recipe directly, no upstream changes needed.

[martenlienen]

The following snippet before the straight.el bootstrap section in my init.el works for me.

(custom-set-variables
 '(straight-recipe-overrides '((nil
                                (nongnu-elpa :type git
                                             :repo "https://github.com/emacsmirror/nongnu_elpa"
                                             :depth (full single-branch)
                                             :local-repo "nongnu-elpa"
                                             :build nil)
                                (ws-butler :type git
                                           :repo "https://github.com/emacsmirror/nongnu_elpa"
                                           :branch "elpa/ws-butler"
                                           :depth (full single-branch)
                                           :local-repo "ws-butler")))))

ws-butler is the package that triggered this for me in the first place when it moved to NonGNU ELPA.

[SeanMooney]

The following snippet before the straight.el bootstrap section in my init.el works for me.

(custom-set-variables
'(straight-recipe-overrides '((nil
(nongnu-elpa :type git
:repo "https://github.com/emacsmirror/nongnu_elpa"
:depth (full single-branch)
:local-repo "nongnu-elpa"
:build nil)
(ws-butler :type git
:repo "https://github.com/emacsmirror/nongnu_elpa"
:branch "elpa/ws-butler"
:depth (full single-branch)
:local-repo "ws-butler")))))

ws-butler is the package that triggered this for me in the first place when it moved to NonGNU ELPA.

This works well for me too, thanks for the suggestion.

[milanglacier]

An alternative approach, particularly if you feel you are concerning about "overriding" something in your config.

(straight-use-package
 '(ws-butler :type git
             :repo "https://github.com/emacsmirror/nongnu_elpa"
             :branch "elpa/ws-butler"
             :depth (full single-branch)
             :local-repo "ws-butler"))