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
> 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: [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

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

[PHP-DEV] Re: ext/curl update

2016-04-23 Thread Pierrick Charron
Hi Davey, Thanks for your answer. I still have small projects on my pipeline that I want to finish but I'll start looking at libnghttp2 and if I get some time I'll recontact you to maybe help you on remaining tasks :-) Do you have specifics stuffs that you would like me to look at ? About the pat

[PHP-DEV] Re: ext/curl update

2016-04-23 Thread Davey Shafik
Hi Pierrick, This should be in master for 7.1, alongside my RFC'ed patch for server push support. You emailed me directly about the aforementioned patch so I'll just respond here as it's relevant: The patch should hit in 7.1 but it has been requested that tests be added — and we can't add tests