Re: [PHP-DEV] Re: ext/curl update

2016-05-05 Thread Nikita Popov
On Thu, Apr 28, 2016 at 11:30 PM, Davey Shafik wrote: > I seem to have created some confusion here: > > The reason _my_ patch for Server Push isn't merged is tests for it were > requested and are blocking it. I'm not saying tests for these constants > should be added. > For the record, I'm fine

RE: [PHP-DEV] Re: ext/curl update

2016-04-29 Thread Anatol Belski
Hi Davey, > -Original Message- > From: m...@daveyshafik.com [mailto:m...@daveyshafik.com] On Behalf Of Davey > Shafik > Sent: Thursday, April 28, 2016 11:30 PM > To: Pierrick Charron > Cc: Anatol Belski ; PHP internals > ; paj...@php.net > Subject: Re: [PHP-

Re: [PHP-DEV] Re: ext/curl update

2016-04-28 Thread Davey Shafik
gt; >>> >>> > -Original Message- >>> > From: pierr...@webstart.fr [mailto:pierr...@webstart.fr] On Behalf Of >>> Pierrick >>> > Charron >>> > Sent: Wednesday, April 27, 2016 6:20 PM >>> > To: Anatol Belski

Re: [PHP-DEV] Re: ext/curl update

2016-04-27 Thread Pierrick Charron
essage- > > From: pierr...@webstart.fr [mailto:pierr...@webstart.fr] On Behalf Of > Pierrick > > Charron > > Sent: Wednesday, April 27, 2016 6:20 PM > > To: Anatol Belski > > Cc: Davey Shafik ; PHP internals >; > > paj...@php.net > > Subject:

Re: [PHP-DEV] Re: ext/curl update

2016-04-27 Thread Pierrick Charron
tart.fr [mailto:pierr...@webstart.fr] On Behalf Of >> Pierrick >> > Charron >> > Sent: Wednesday, April 27, 2016 6:20 PM >> > To: Anatol Belski >> > Cc: Davey Shafik ; PHP internals < >> internals@lists.php.net>; >> > paj...@php.net >

RE: [PHP-DEV] Re: ext/curl update

2016-04-27 Thread Anatol Belski
> -Original Message- > From: pierr...@webstart.fr [mailto:pierr...@webstart.fr] On Behalf Of Pierrick > Charron > Sent: Wednesday, April 27, 2016 6:20 PM > To: Anatol Belski > Cc: Davey Shafik ; PHP internals ; > paj...@php.net > Subject: Re: [PHP-DEV] Re: ext/c

Re: [PHP-DEV] Re: ext/curl update

2016-04-27 Thread Pierrick Charron
-Original Message- > > From: pierr...@webstart.fr [mailto:pierr...@webstart.fr] On Behalf Of > Pierrick > > Charron > > Sent: Wednesday, April 27, 2016 2:20 PM > > To: Anatol Belski > > Cc: Davey Shafik ; PHP internals >; > > paj...@php.net > > Subjec

RE: [PHP-DEV] Re: ext/curl update

2016-04-27 Thread Anatol Belski
Hi, > -Original Message- > From: pierr...@webstart.fr [mailto:pierr...@webstart.fr] On Behalf Of Pierrick > Charron > Sent: Wednesday, April 27, 2016 2:20 PM > To: Anatol Belski > Cc: Davey Shafik ; PHP internals ; > paj...@php.net > Subject: Re: [PHP-DEV] Re

Re: [PHP-DEV] Re: ext/curl update

2016-04-27 Thread Pierrick Charron
I agree, but I don't really now how I could test those things since they almost all of the time only affect how libcurl will handle the request/cache and we have no way to retrieve options like curl_easy_getopt or something similar. On 27 April 2016 at 02:46, Anatol Belski wrote: > Hi, > > > ---

RE: [PHP-DEV] Re: ext/curl update

2016-04-26 Thread Anatol Belski
Hi, > -Original Message- > From: m...@daveyshafik.com [mailto:m...@daveyshafik.com] On Behalf Of Davey > Shafik > Sent: Sunday, April 24, 2016 2:25 AM > To: Pierrick Charron > Cc: PHP internals ; paj...@php.net > Subject: [PHP-DEV] Re: ext/curl update > > Hi Pierrick, > > This should be

Re: [PHP-DEV] Re: ext/curl update

2016-04-25 Thread Leigh
On Sun, 24 Apr 2016 at 01:25 Davey Shafik wrote: > Hi Pierrick, > > This should be in master for 7.1, alongside my RFC'ed patch for server push > support. I don't see why the additional constants that don't have a direct dependency on your server push patch can't be committed to 7.0 as well. Th