skypjack / uvw

Header-only, event based, tiny and easy to use libuv wrapper in modern C++ - now available as also shared/static library!
MIT License
1.82k stars 207 forks source link

create v2 branch for bug fixes #287

Closed fcelda closed 1 year ago

fcelda commented 1 year ago

uvw 2.12.1 currently doesn't compile with GCC 13 (Fedora 38) because type_info.hpp doesn't have correct include for standard integers.

/home/fcelda/devel/uvw/src/uvw/type_info.hpp:17:37: error: ‘uint32_t’ in namespace ‘std’ does not name a type; did you mean ‘wint_t’?
   17 | [[nodiscard]] static constexpr std::uint32_t fnv1a(const char *curr) noexcept {
      |                                     ^~~~~~~~
      |                                     wint_t

The fix is trivial, see: https://github.com/skypjack/uvw/compare/v2.12.1_libuv_v1.44...fcelda:uvw:v2?expand=1

Are you planning on maintaining 2.x for a while? I think it would be useful at least for bug fixes.

If so, please, could you create a branch for 2.x? I'm not able to send a PR without it.

Thanks!

skypjack commented 1 year ago

Are you planning on maintaining 2.x for a while?

Define maintaining. 🙂 I'm fine with creating a bug fix branch for the last v2 version, of course. However, I don't plan to port uvw v2 to the new versions of libuv, I'm sorry. I don't have enough bandwidth for that, you know, it's a side open source project after all.

fcelda commented 1 year ago

Define maintaining. :slightly_smiling_face:

I don't expect any significant work to be done on 2.x but I think it's desirable to allow for bug fixes for 2.x at least until 3.x matures and people get comfortable migrating. I tried migrating and hit some issues that I'm slowly getting through so I choose to stick on 2.x for now because I don't need the new libuv features.

If you could create the branch for 2.x, I will be happy to contribute my fix (and maybe others in the future) so that people using 2.x can benefit it. If you think the bug fixes piled up eventually and cut a new release it would be awesome but not necessary.

I understand that you want to focus on 3.x and I fully respect that.

skypjack commented 1 year ago

Branch v2 available upstream. 👍

fcelda commented 1 year ago

Thank you!