Re: [Libreoffice] [PATCH] add /usr/local/sbin and /usr/local/bin to the gen{brk, ccode, cmn} searchpath

On Thu, Nov 11, 2010 at 09:29:10PM +0100, Robert Nagy wrote:
> diff --git a/configure.in b/configure.in
> index b92e597..0b812c2 100644
> --- a/configure.in
> +++ b/configure.in
> @@ -5399,15 +5399,15 @@ if test -n "$with_system_icu" -o -n "$with_system_libs" && \
>     AC_LANG_PUSH([C++])
>     AC_MSG_CHECKING([for unicode/rbbi.h])
>     AC_TRY_CPP(unicode/rbbi.h, AC_MSG_RESULT([checked.]), AC_MSG_ERROR([icu headers not found.]))
> -   AC_PATH_PROG(SYSTEM_GENBRK, genbrk, [], [$PATH:/usr/sbin:/sbin])
> +   AC_PATH_PROG(SYSTEM_GENBRK, genbrk, [], [$PATH:/usr/sbin:/sbin:/usr/local/bin])
>     if test -z "$SYSTEM_GENBRK"; then
>        AC_MSG_ERROR([\"genbrk\" not found in \$PATH, install the icu development tool \"genbrk"\])
>     fi
> -   AC_PATH_PROG(SYSTEM_GENCCODE, genccode, [], [$PATH:/usr/sbin:/sbin])
> +   AC_PATH_PROG(SYSTEM_GENCCODE, genccode, [], [$PATH:/usr/sbin:/sbin:/usr/local/sbin])
>     if test -z "$SYSTEM_GENCCODE"; then
>        AC_MSG_ERROR([\"genccode\" not found in \$PATH, install the icu development tool 
> \"genccode"\])
>     fi
> -   AC_PATH_PROG(SYSTEM_GENCMN, gencmn, [], [$PATH:/usr/sbin:/sbin])
> +   AC_PATH_PROG(SYSTEM_GENCMN, gencmn, [], [$PATH:/usr/sbin:/sbin:/usr/local/sbin])
>     if test -z "$SYSTEM_GENCMN"; then
>        AC_MSG_ERROR([\"gencmn\" not found in \$PATH, install the icu development tool \"gencmn"\])
>     fi

Can that be made conditional on OpenBSD, please? I've seen already too many f*ups because
someone had old libraries/binaries in /usr/local (on Linux), so I'd want to prevent
that from happening if possible and this makes it even more likely to happen (though admittedly
this shouldn't be the case for the ICU utilities)

Grüße/Regards,

René