2009/6/3 Jani Taskinen <[email protected]>: > Kalle Sommer Nielsen kirjoitti: >> >> 2009/6/3 Hannes Magnusson <[email protected]>: >>> >>> On Wed, Jun 3, 2009 at 03:19, Kalle Sommer Nielsen <[email protected]> wrote: >>>> >>>> kalle Wed Jun 3 01:19:33 2009 UTC >>>> >>>> Modified files: (Branch: PHP_5_3) >>>> /php-src/ext/bcmath/libbcmath/src bcmath.h >>>> Log: >>>> MFH: config.h is in this directory, not in ext/bcmath root, fixes >>>> re-compilation for me >>> >>> Please read the commit history. This will probably break other stuff. >> >> Its broken for me, if its an issue for others then we need to isolate >> it, if i do a fresh nmake and then make a buildconf && configure && >> nmake (after i have made a change), it breaks the build because >> config.h isn't in ext/bcmath, but ext/bcmath/libbcmath/src/ > > Did you try on real OSes? Please keep in mind that Windows comes 2nd, if you > "fix" something and it breaks on other OSes, that fix will be reverted.
So "real OSes" about everything thats not produced by Microsoft? I don't see why we treat Windows as second class really, its just as a good OS as any of the other (*awaiting stupid comment*). Windows isn't lacking that much behind anymore, especially not with 5.3. Alternatively until the problem have been solved, then we could just directly include the config.h which is abit ugly and we're all "happy": #include "ext/bcmath/libbcmath/src/config.h" > > --Jani > > -- > PHP CVS Mailing List (http://www.php.net/) > To unsubscribe, visit: http://www.php.net/unsub.php > > -- regrads, Kalle Sommer Nielsen [email protected] -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
