-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
oil: update to 0.23.0 #51964
oil: update to 0.23.0 #51964
Conversation
ed38a9b
to
db3fb66
Compare
So, while we are renaming the package, should we do it like upstream suggests? https://github.com/oils-for-unix/oils/wiki/Oils-Packaging-Guidelines |
The definition itself is confusing, as there are two packages, one Python and one C, we should adopt more logical standards, this oils standard for package, is more correct. |
Okay, so this is the old Python-version, but there upstream says: https://www.oilshell.org/release/0.23.0/
So let's keep this (just adding the link is fine), and perhaps someone can contribute |
The definition itself is confusing, as there are two packages, one Python and one C, we should adopt more logical standards, this oils standard for package, is more correct.
Python version is slow, a few seconds, C++ version is slower than bash version. Python version is more stable and tested, has a history of having fewer bugs. Here is a speed test and some bugs of the C++ version: |
We don't need multiple versions of this package in Void, and the project author notes that the "reference" implementation might go away at some point, so just pick one to be the version packaged by Void. |
@ahesford doesnt matter, the new version should have a new package name anyway (and conflict, I guess) |
I've tried to package |
Also, I vote for keeping |
I keep the old name oil or rename it to oils, @meator voted to keep the old name, because I need to know what procedure I do, the package needs to be updated too! |
yes, please keep the old name. |
Done! |
what will happen for anyone using |
|
Testing the changes