#45900 [Fbk->Opn]: timelib_structs.h include path issue on Windows

[email protected] ("Keisial at gmail dot com")
Newsgroups php.bugs
Message-ID <[email protected]>
 ID:               45900
 User updated by:  Keisial at gmail dot com
 Reported By:      Keisial at gmail dot com
-Status:           Feedback
+Status:           Open
 Bug Type:         Compile Failure
 Operating System: Windows
 PHP Version:      5.3CVS-2008-08-23 (snap)
 New Comment:

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.


Previous Comments:
------------------------------------------------------------------------

[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.