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

Reply via email to