-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Add a timeout for elpa repositories when downloading packages #3284
Comments
New file core-emacs-ext.el This is a basic monkey-patch solution but it will do the job for now. The timeout amount is not configurable for now. Tested on 24.5 and 24.3.1 fixes #3284
We should now have a basic timeout of 5 seconds when fetching the elpa archives. |
New file core-emacs-ext.el This is a basic monkey-patch solution but it will do the job for now. The timeout amount is not configurable for now. Tested on 24.5 and 24.3.1 fixes syl20bnr#3284
As someone who would like to improve package.el itself, could I ask for more information on this? |
New file core-emacs-ext.el This is a basic monkey-patch solution but it will do the job for now. The timeout amount is not configurable for now. Tested on 24.5 and 24.3.1 fixes syl20bnr#3284
This is still not fixed. Elpa is down at the moment and when spacemacs load I get this error:
Afterwards, spacemacs fails to load, syntax highlighting gets turned off, and i'm back to emacs shortcuts. |
@mb-dev this is fixed in 0.105.11 :-) |
@Malabarba Sorry for the late reply, Emacs starts normally, the issue was the default time of the duration out which seems hard coded and not available to the end user. If I recall correctly the time out was something like 90sec. or 120sec. What I did in Spacemacs is a pre-step to GET the repo URL with a |
Great @syl20bnr! thanks for fixing it. Spacemacs is becoming my favorite editor. |
If one of the elpa repositories is down for any reason...bootstrapping, package updates and/or new package install fails.
Proposal: add a timeout -- and gracefully fail (if possible).
This issue is based on a discussion in #3270, specifically see #3270 (comment).
The text was updated successfully, but these errors were encountered: