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 <anatol....@belski.net> > Cc: Davey Shafik <da...@php.net>; PHP internals <internals@lists.php.net>; > paj...@php.net > Subject: Re: [PHP-DEV] Re: ext/curl update > > 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 <anatol....@belski.net > <mailto:anatol....@belski.net> > wrote: > > > Hi, > > > -----Original Message----- > > From: m...@daveyshafik.com <mailto:m...@daveyshafik.com> > [mailto:m...@daveyshafik.com <mailto:m...@daveyshafik.com> ] On Behalf Of > Davey > > Shafik > > Sent: Sunday, April 24, 2016 2:25 AM > > To: Pierrick Charron <pierr...@adoy.net <mailto:pierr...@adoy.net> > > > Cc: PHP internals <internals@lists.php.net > <mailto:internals@lists.php.net> >; paj...@php.net <mailto:paj...@php.net> > > Subject: [PHP-DEV] Re: ext/curl update > > > > 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 with a server push supporting HTTP/2 server against > which to > > push. > > > As from the patch, many constants have nothing to do with HTTP/2 > implementation and add just name/value without any further logic. If there > were > a reduced patch with only such cases, it would be acceptable for 7.0 as well > and > there were probably no collisions expected. What do you think? > So far I understood tests are exactly about HTTP2. Not sure how you would tests all the constants present in libcurl. Would need to rebuild with a dozen libcurl versions, but the documentation and compile time version check are already reliable things.
Thanks anatol -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
