-
Notifications
You must be signed in to change notification settings - Fork 64
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
2.8.x / 2.9.x release? #187
Comments
Not intentional. The build machines have been gone for some time. It would be nice to get travis to push images. |
My experience in building C software doesn't extend much further than |
@SinisterRectus which platform are you interested in? |
I have a decent amount of experience in compiling and writing C software as well as devops stuff. I can potentially help out here. |
I personally develop on 64-bit Windows, but having my own copy of luvi 2.8.0 would not be helpful for my users, who I instruct to use the get-lit script(s). Discord supports transport compression with zlib streams, so I was looking to test the new miniz bindings, but they have not made transport compression mandatory in their protocol, so it's not critical that I have access to this at the moment. |
Travis just added beta support for windows... We could potentially leverage that. |
This will help for Linux builds... #181 |
There are already 46 commits to master since tag 2.8.0. Maybe it is time for 2.9.0? |
2.8.1 or 2.9.0 might be appropriate, depending on what the changes are. Would probably be best to have a reliable build procedure for building 2.8.0 and then worry about the rest. |
We are stuck to version 2.7.6 in Buildroot, as it is the latest version which provides a release tarball. Unfortunately, this version does not compile with OpenSSL 1.1.x I'd be delighted if there was a new release with a downloadable tarball for the source 🙏 Note, that I am not interested in any target binaries, but in a tarball containing all the source files (like luvi-src-v2.7.6.tar.gz). |
This? |
No, as the deps folders are all empty. |
This. |
The Makefile already has the necessary target Maybe one of the maintainers can tag a release and run |
2.8.1 or 2.9.0? |
I tend towards 2.9.0. |
After merging #189 |
Let's get this out. At a minimum we can publish the tarball for builtroot. The binaries are harder depending on how many we want to support. I had a pile of raspberry pi machines, 64-bit arm boards and virtualbox images that I was using before to manually build all the variants. |
@joerg-krause is anything extra needed to support the newer openssl or is it already in the newer luvi commits? |
@creationix No, I think everything is prepared. Buildroot uses an external OpenSSL, but still requires the bundled lua-openssl, which has been bumped to the latest version. Thanks for taking care! |
@joerg-krause please test https://github.com/luvit/luvi/releases/tag/v2.9.0 It's been a while and my old scripts/tools aren't working. I created this release manually and uploaded the tarball manually, but I think it will work. |
I also manually built and uploaded linux x64 binaries on an ubuntu 14.04 system. The docker-based stuff isn't working for me. |
@creationix Thanks for the source tarball! It is working perfectly, but the
|
Ok, rebuilt the tarball. I think I had built it before creating the new tag in git. |
Thanks! That worked:
|
@creationix here's a full set of Windows binaries/libs if you want to add them to the release: https://www.ryanliptak.com/misc/luvi-windows-2.9.0.zip Built with MSVC 2015 Update 3 on Windows 7 and includes this PR.
EDIT: Might want to release a 2.9.1 after merging the PR linked above? |
Here are Windows binaries/libs for 2.9.1 (the same as the 2.9.0 ones above except for the version string):
|
I think my linux binaries are wrong. They appear to have dynamically linked libuv. |
Yeah, they would have since you built them before #191. @rphillips released v2.9.1 with that fix. |
Thanks, I published linux binaries for the v2.9.1 release. They also appear to dynamically link openssl, but I think that's on purpose now for linux?
|
That's the default now, yeah. You can add these Windows binaries to the release too if you'd like: https://www.ryanliptak.com/misc/luvi-windows-2.9.1.zip |
Thanks! Just for sanity, I ran the binaries through clamav. |
I just noticed that there is a tag for 2.8.0, but no release, and therefore no luvit release that contains luvi 2.8.0. Is this intentional?
The text was updated successfully, but these errors were encountered: