On Sun, Jan 20, 2008 at 02:41:55PM -0500, Jay Berkenbilt wrote:
> Aurelien Jarno <[EMAIL PROTECTED]> wrote:
> 
> > Package: icu
> > Version: 3.8-5
> > Severity: important
> > Tags: patch
> >
> > icu fails to build on GNU/kFreeBSD because it doesn't known about this
> > platform. The check is for the kernel, but the result is currently used
> > to decide about userland stuff. It may be a good idea to replace the
> > check for the Linux kernel by a check for the GNU libc. However, as I
> > don't know the goal of this check in the future I have written a patch
> > to add GNU/kFreeBSD in the list of platform.
> >
> > Please find this patch below. Could you please apply it to the next
> > upload, and forward it to upstream? Thanks in advance.
> 
> Yes, I'll take care of that.  Just so you know, I'm not touching the

Thanks a lot.

> ICU packages again until after 3.8 makes it into Lenny.  The
> transition was one day away and then got blocked by a gcc-related
> problem.  One ICU transitions, my next upload will be 3.8.1, which has
> already come out.  (No ABI changes there.)  If your patch applies
> cleanly to 3.8.1 (or I can adjust it, which seems likely), then I'll
> apply it and report upstream.  Otherwise, I'll ask you to regenerate
> it against 3.8.1.

The patch mostly applies, but a small change is needed. Please find
below a patch against version 3.8.1.


--- icu.orig/source/common/putil.c      2007-12-12 19:57:26 +0100
+++ icu/source/common/putil.c   2008-01-20 21:08:57 +0100
@@ -105,7 +105,7 @@
 #   define ICU_NO_USER_DATA_OVERRIDE 1
 #elif defined(OS390)
 #include "unicode/ucnv.h"   /* Needed for UCNV_SWAP_LFNL_OPTION_STRING */
-#elif defined(U_DARWIN) || defined(U_LINUX) || defined(U_BSD)
+#elif defined(U_DARWIN) || defined(U_LINUX) || defined(U_BSD) || 
defined(U_KBSD)
 #include <limits.h>
 #include <unistd.h>
 #elif defined(U_QNX)
@@ -573,10 +573,10 @@
 extern U_IMPORT char *U_TZNAME[];
 #endif
 
-#if !UCONFIG_NO_FILE_IO && (defined(U_DARWIN) || defined(U_LINUX) || 
defined(U_BSD))
+#if !UCONFIG_NO_FILE_IO && (defined(U_DARWIN) || defined(U_LINUX) || 
defined(U_BSD) || defined(U_KBSD))
 /* These platforms are likely to use Olson timezone IDs. */
 #define CHECK_LOCALTIME_LINK 1
-#if defined(U_LINUX)
+#if defined(U_LINUX) || defined(U_KBSD)
 #define TZDEFAULT       "/etc/localtime"
 #define TZZONEINFO      "/usr/share/zoneinfo/"
 #else
@@ -1427,7 +1427,7 @@
         */
         name = "eucjis";
     }
-#elif defined(U_LINUX)
+#elif defined(U_LINUX) || defined(U_KBSD)
     if (locale != NULL && uprv_strcmp(name, "euc") == 0) {
         /* Linux underspecifies the "EUC" name. */
         if (uprv_strcmp(locale, "korean") == 0) {
--- icu.orig/source/configure   2007-12-12 19:58:06 +0100
+++ icu/source/configure        2008-01-20 21:07:42 +0100
@@ -8291,6 +8291,7 @@
 case "${host}" in
        *-*-solaris*)   platform=U_SOLARIS ;;
        *-*-linux*)     platform=U_LINUX ;;
+       *-*-k*bsd*-gnu*) platform=U_KBSD ;;
        *-pc-gnu)       platform=U_HURD ;;
        *-*-*bsd*|*-*-dragonfly*)       platform=U_BSD ;;
        *-*-aix*)       platform=U_AIX ;;
--- icu.orig/source/configure.in        2007-12-12 19:58:06 +0100
+++ icu/source/configure.in     2008-01-20 21:07:42 +0100
@@ -996,6 +996,7 @@
 case "${host}" in
        *-*-solaris*)   platform=U_SOLARIS ;;
        *-*-linux*)     platform=U_LINUX ;;
+       *-*-k*bsd*-gnu*) platform=U_KBSD ;;
        *-pc-gnu)       platform=U_HURD ;;
        *-*-*bsd*|*-*-dragonfly*)       platform=U_BSD ;;
        *-*-aix*)       platform=U_AIX ;;

-- 
  .''`.  Aurelien Jarno             | GPG: 1024D/F1BCDB73
 : :' :  Debian developer           | Electrical Engineer
 `. `'   [EMAIL PROTECTED]         | [EMAIL PROTECTED]
   `-    people.debian.org/~aurel32 | www.aurel32.net



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to