Re: Re[2]: [UNSURE] Fw: [UNSURE] Fw: [UNSURE] Re: [UNSURE] Re: [UNSURE] Fw: [UNSURE] Re: some success :)

Hi Tom, Making the suggested changes in the configuration process actually breaks the build for me on WinXP. Please can you give me details of: 1) the OS you're using 2) what comes back when you echo PHP_OS (in case we need to make finegrained OS-specific changes in the generator as you suggested below) 3) your current build env layout 4) the contents of vcvars32.bat (by default it lives in C:\Program Files\Microsoft Visual Studio\VC98\Bin) If we're going to debug this thing properly I need to know exactly what happens when you _don't_ make any changes to the CVS version, e.g. error messages and which point they turn up at. Would you mind very much renaming the php-gtk dir that works for you, getting a fresh check-out, following the instructions there (I sanitized the include paths now) and starting from scratch? Thank you for all your hard work in testing this yesterday, I know you spent some several hours on it! - Steph ----- Original Message ----- From: "Tom Rogers" <trogers@kwikin.com> To: "Steph" <steph@zend.com> Sent: Wednesday, September 07, 2005 9:05 AM Subject: Re[2]: [UNSURE] Fw: [UNSURE] Fw: [UNSURE] Re: [UNSURE] Re: [UNSURE] Fw: [UNSURE] Re: some success :) > Hi Steph, > > Getting closer :) > The reason overrides were not being processed is windows habit of > changing \n to \r\n which screwed the parser. I have a small fix for > that. > //overrides.php > > function _parse_block($block, $blocklineno, $file_name) > { > > if (strpos($block, "\n")){ > $test = preg_split('/\r\n|\n/',ltrim($block)); > $line = array_shift($test); > $rest = implode("\n",$test); > } > else { > $line = $block; > $rest = ''; > } > $words = preg_split('!\s+!', $line, -1, PREG_SPLIT_NO_EMPTY); > $command = array_shift($words); > switch ($command) { > > > > Next in config-utils.js need to add a backslash to the PATH_PROG() > function like this: > > if (place == true) { > > place = exe; > > } else if (place != false) { > > place = '"'+place +'\\'+ exe+'"'; > > } > > Or maybe a test to see if place has a trailing backslash? > > Also in config.w32.in I had to add another backslash to the single . > in the include line (also the ..\\..) > //Base includes > ADD_FLAG("CFLAGS_PHP_GTK", "/I .\\ /I ..\\ /I ..\\..\\ /I main /I ..\\main /I ext /I ..\\ext"); > > With those changes it compiles through to the dll. > It will run stock_browser.php with the release version and no crashes > > Whew :) > -- > regards, > Tom >