-
Notifications
You must be signed in to change notification settings - Fork 4
Move to URL instead of CURL #3
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
base: main
Are you sure you want to change the base?
Conversation
Curl is not an emacs package, this is removed.
|
@ethanmoss1 thanks for putting this together and making it an Emacs only package, I love that. I won't be able to test it for a couple days but I'm on-board with merging it and going this direction. I'll test and merge ASAP. Thanks again! |
|
Oh no, thank you! Especially for doing most of the work! It was fun learning the URL package/lib. I might tackle some of the long-term goals soon, so I'll create a pull request when I've had a go. Any issues just shout and I'll get them changed ready for merging. |
This upsets Elpaca as it trys to install url
The function can now check inputs as well as change priority on the message.
The priority should be 1 to 5. and now the checks use a known good string or number
Bumped version to 2 as this requires a different library and no need for CURL.
Fixes issue #1, This stops package.el, elpaca and straight from trying to find an emacs package called curl.
If we want i can tackle #2 ?