-
Notifications
You must be signed in to change notification settings - Fork 59
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
Update atuin to v18.1.0 #1361
Update atuin to v18.1.0 #1361
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also the build seems to fail, we'd appreciate if you could have a look and fix it if possible. Thanks
I have filed atuinsh/atuin#1361 There are two things we can do:
The first option didn't pass tests, so I will need to setup an OmniOS version with X11 to make more progress. |
any progress ? |
Sorry haven't had time to put together the necessary patch on the atuin side. |
18.1.0 fixes this, and also fixes some undefined behaviour on illumos. I've updated the PR. |
Version 17.0.0 just released: https://github.com/atuinsh/atuin/releases/tag/v17.0.0
I haven't run through a build of this, but I am filing the PR anyways.