ID: 45900
Updated by: [email protected]
Reported By: Keisial at gmail dot com
-Status: Feedback
+Status: Bogus
Bug Type: Compile Failure
Operating System: Windows
PHP Version: 5.3CVS-2008-08-23 (snap)
-Assigned To:
+Assigned To: pajoye
New Comment:
I have discussed with Keisal on IRC and the issue is fixed (clean co
and al).
Previous Comments:
------------------------------------------------------------------------
[2008-08-25 18:25:53] [email protected]
Is timelib_config.h generated or not when this happens? WHERE is it
generated to? Search for it, it might get generated but in a wrong
directory..
------------------------------------------------------------------------
[2008-08-23 21:28:33] Keisial at gmail dot com
You're right. The issue is not the switch style. Why is it always
documented with / ? :)
The error is:
.\ext/date/lib/timelib_structs.h(24) : fatal error C1083: Cannot open
include file: 'timelib_config.h': No such file or directory
I have gone deeper into it, and is not caused on compiling date but
when compiling ext\standard\microtime.c which #include
"ext/date/php_date.h"
Interestingly, i only see it with --enable-snapshot-build and running
again nmake skips it.
------------------------------------------------------------------------
[2008-08-23 18:00:54] [email protected]
-I works like /I
There is other extension using the -I (gd, pcre, etc.) without
problems. Which error do you get (exactly)?
------------------------------------------------------------------------
[2008-08-23 17:52:17] Keisial at gmail dot com
Description:
------------
Compiling date extension on windows has the same problem reported in
bug 37163.
ext/date/lib is included in the config.w32, but with the wrong switch.
Being for windows, it should use /I instead of -I
--- config.w32 2006-01-05 07:41:10.000000000 +0100
+++ config.w32 2008-08-23 19:45:58.000000000 +0200
@@ -1,7 +1,7 @@
// $Id: config.w32,v 1.5.2.1 2006/01/05 06:41:09 sebastian Exp $
// vim:ft=javascript
-EXTENSION("date", "php_date.c", false, "-Iext/date/lib");
+EXTENSION("date", "php_date.c", false, "/Iext/date/lib");
ADD_SOURCES("ext/date/lib", "astro.c timelib.c dow.c parse_date.c
parse_tz.c tm2unixtime.c unixtime2tm.c", "date");
AC_DEFINE('HAVE_DATE', 1, 'Have date/time support');
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=45900&edit=1
lmpx.com only provides a reader for public news (NNTP) servers. It is not
affiliated with the servers or forums shown here and is not responsible for
the content of articles, which is written by their respective authors.