Hi, > -----Original Message----- > From: [email protected] [mailto:[email protected]] On Behalf Of Pierrick > Charron > Sent: Wednesday, April 27, 2016 2:20 PM > To: Anatol Belski <[email protected]> > Cc: Davey Shafik <[email protected]>; PHP internals <[email protected]>; > [email protected] > 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 <[email protected] > <mailto:[email protected]> > wrote: > > > Hi, > > > -----Original Message----- > > From: [email protected] <mailto:[email protected]> > [mailto:[email protected] <mailto:[email protected]> ] On Behalf Of > Davey > > Shafik > > Sent: Sunday, April 24, 2016 2:25 AM > > To: Pierrick Charron <[email protected] <mailto:[email protected]> > > > Cc: PHP internals <[email protected] > <mailto:[email protected]> >; [email protected] <mailto:[email protected]> > > 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
