On Wed, 25 Jun 2003, Berin Lautenbach wrote: Hi,
> WooHoo! Works like a charm. testXPath now runs on my 4.8-RELEASE box, > so very happy. :-)) > What I might do (if you have no objections), is incorporate the minimal > amount of changes into the current makefiles to get Freebsd to compile > and run properly. That is what I first did - I hope I still have these patches. My plan was to merge my patch and current CVS, sent it back via bugzilla and get it in and tested as fast as possible for as much platforms a possble. Then get the diff back from CVS before sending pr to FreeBSD gnats. OS390 is a special problem I think because they don't seem to use gcc(?) and have the .x moves and I don't have a plan what this is for. I think it might be an equivalent to .so files. The problem was the same as with xerces (library naming on elf system was broken). Also see my bug report for xerces-c: http://nagoya.apache.org/bugzilla/show_bug.cgi?id=13992 > I might leave the install piece out for now, although I think it's an no I will not do that ;-) I don't want to copy all this to the Makefile of the port. This is the wrong place. [more see above] > important piece - but if there is a risk of breaking OS390 I'd like to > take things slowly. I am pretty sure the maintainers from IBM will get this fixed (if broken) within minutes ;-) > I might also slightly modify the configure test you have built in to > actually run a piece of code to test mbstowcs, as this is also a problem > on NetBSD so it would be good to have a generic test. well this is no problem at all. What arch maintainers will have todo is add the ${WCSTOMBS} (formerly known as ${PCO}; already changed that locally) to their PLATFORM_COMPILE_OPTIONS. One can also do that for netbsd (if one includes the build options to Makefile.in) and have a more generic test like ... in configure.in: --- untested --- dnl check if mbstowcs is available and can correctly count only AC_HEADER_STDC AC_CHECK_FUNCS([mbstowcs],,,) if test x"$ac_cv_func_mbstowcs" = x"no"; then WCSTOMBS="-DXALAN_USE_XERCES_LOCAL_CODEPAGE_TRANSCODERS" else AC_MSG_CHECKING(if mbstowcs can count only) AC_TRY_RUN([ #if STDC_HEADERS #include <stdlib.h> #endif int main(int argc, char *argv[]) { wchar_t in[5] = { 0x74, 0x65, 0x73, 0x74, 0x00 }; if (wcstombs(0, in, 0) == -1) { exit(1); } exit(0); }], [AC_MSG_RESULT(yes)], [WCSTOMBS="-DXALAN_USE_XERCES_LOCAL_CODEPAGE_TRANSCODERS"; AC_MSG_RESULT(no)]) fi --- /untested --- > The only problem is that this might break the port that you have done, > but hopefully we can keep the breakage to an easily defined subset. no breakage allowed ;-)) Well I have found another Problem on a 4.6-STABLE box. pkg_info output has changed a bit and the direcotry from where the port had been built was not stored. So I also added defaults for this case that we find libiconv/icu there too. I will now start testing the icu chain and if it works release a new tarball and add the configure/install patch to bugzilla. -- Greetings Bjoern A. Zeeb bzeeb at Zabbadoz dot NeT 56 69 73 69 74 http://www.zabbadoz.net/
