note 41820 added to ref.curl
| From: | quentin dot bennett dot nospamatinfinityconz at rn2 dot php dot net | Date: | Fri, 23 Apr 2004 01:38:43 +0000 |
| Subject: | note 41820 added to ref.curl | ||
| Groups: | php.notes | ||
| Request: | Send a blank email to php-notes+get-68563@lists.php.net to get a copy of this message | ||
I ran into this problem, and found the solution:
On Fri, 26 Mar 2004, Vladimir Brajovic wrote:
> Compiling ext/curl/curl.c fails because struct HttpPost is not declared. The
> Curl change log indicates that '#define HttpPost curl_httppost' has been
> removed from header file. However this seems to be causing this particular
> problem since PHP still refers to HttpPost structure. By adding '#define
> HttpPost curl_httppost' into curl.h, or by changing all references to
> HttpPost in curl.c, I was able to eliminate this problem.
The official API has been using curl_httppost for more than 2 years. I
consider that enough time for everyone to move over.
This change was also available in daily snapshots and the 7.11.1 pre-release
in good time before the public release was made which should've given everyone
who cared enough to test it.
Not a single soul identified this as a problem before release.
So, now everyone need to patch their *PHP source code*. The correct fix is to
rename the structs in the PHP source code (or add the define in there). It is
not to put back the define in the curl header. IMHO.
----
Manual Page -- http://www.php.net/manual/en/ref.curl.php
Edit -- http://master.php.net/manage/user-notes.php?action=edit+41820
Delete -- http://master.php.net/manage/user-notes.php?action=delete+41820&report=yes
Reject -- http://master.php.net/manage/user-notes.php?action=reject+41820&report=yes
Search -- http://master.php.net/manage/user-notes.php