Bug #79938 for UV: use Alien instead of including foreign source

Sun Sep 30 17:47:46 2012 dirkus [...] yopmail.com - Ticket created

Subject:

use Alien instead of including foreign source

Instead of shipping a load of foreign source code, use an Alien module to check if libuv is already installed on the system, and if not, then download and build it.

Sun Sep 30 19:18:03 2012 typester [...] gmail.com - Correspondence added

On Sun Sep 30 17:47:46 2012, dirk wrote: Show quoted text

> Instead of shipping a load of foreign source code, use an Alien module > to check if libuv is already installed on the system, and if not, then > download and build it.

I understand what you said though, libuv is under heavy development stage, the APIs are very version specific at this time. So *at this time*, bundling the library and statically linking to it is better practice IMO.

Sun Sep 30 19:18:04 2012 The RT System itself - Status changed from 'new' to 'open'

Fri Feb 06 02:53:39 2015 book [...] cpan.org - Correspondence added

On Sun Sep 30 19:18:03 2012, TYPESTER wrote: Show quoted text

> On Sun Sep 30 17:47:46 2012, dirk wrote:

> > Instead of shipping a load of foreign source code, use an Alien > > module > > to check if libuv is already installed on the system, and if not, > > then > > download and build it.

> > I understand what you said though, libuv is under heavy development > stage, the APIs are very > version specific at this time. > > So *at this time*, bundling the library and statically linking to it > is better practice IMO.

Two years later, has the development stabilized? Does the Alien:: option make miore sense? -- BooK

Mon Mar 14 13:03:17 2016 leonerd-cpan [...] leonerd.org.uk - Correspondence added

+1 to this. I'm currently investigating a bug in a module that uses UV (https://rt.cpan.org/Ticket/Display.html?id=113010) and I suspect the bug is fixed in upstream libuv source, which I can't currently make use of because of this static inclusion. -- Paul Evans

Mon Mar 14 13:17:16 2016 leonerd-cpan [...] leonerd.org.uk - Correspondence added

https://github.com/libuv/libuv/blob/764877fd9e4ea67c0cbe27bf81b2b294ed33b0f5/docs/src/migration_010_100.rst May be of interest. I'm currently working on that and attempting to fix UV.xs -- Paul Evans

Mon Mar 14 14:30:23 2016 leonerd-cpan [...] leonerd.org.uk - Correspondence added

On Mon Mar 14 13:17:16 2016, PEVANS wrote: Show quoted text

> https://github.com/libuv/libuv/blob/764877fd9e4ea67c0cbe27bf81b2b294ed33b0f5/docs/src/migration_010_100.rst > > May be of interest. I'm currently working on that and attempting to > fix UV.xs

I've made some progress. https://github.com/leonerd/p5-UV/tree/libuv_1.0 Has all but one issue fixed now. I've tried to retain the existing Perl-visible API, including such things as the IPv4-vs-IPv6 specific nature of the TCP and UDP functions. A proper fix for those would be API-changing at the Perl level, so you might want to neaten those up upstream. The one remaining issue is that `uv_read2_start` no longer exists. I'm not familiar enough with what it used to do to suggest how to replace it with the newer functions; at least, not without breaking the Perl-level API. -- Paul Evans

Thu Jun 29 20:34:10 2017 cwhitener [...] gmail.com - Correspondence added

There's a working Alien::libuv now. \o/

Sat Apr 21 22:20:05 2018 cwhitener [...] gmail.com - Correspondence added

Hi Everyone, The day has come. We have released a version of UV that depends on Alien::libuv. Please don't hesitate to report problems if you notice them! Thanks, Chase

Sat Apr 21 22:20:06 2018 cwhitener [...] gmail.com - Status changed from 'open' to 'resolved'

Sat Apr 21 22:20:06 2018 cwhitener [...] gmail.com - Taken