Re: [xslt] Patch for xsl:sort lang support

[Daniel Veillard <veillard redhat com>]

On Tue, Jun 03, 2008 at 10:40:53PM +0300, Roumen Petrov wrote:
> Nick Wellnhofer wrote:
> >Daniel Veillard wrote:
> >>On Thu, May 15, 2008 at 02:46:17PM +0200, Nick Wellnhofer wrote:
> >>  Okay, thanks !
> >>I finally took the time to apply your patch (no I hadn't forgotten :-)
> >>it's now in SVN !
> >>Just one thing, it would be nice to add a couple of simple example
> >>to test taht this is working correctly in the regression tests, if you
> >>have some that would be a good addition.
> >
> >Yes, I had a test case in the first patch I sent. But you need to have 
> >the required locales installed with locale-gen on some systems, 
> >otherwise the tests won't work. I don't know if this is a problem.
> >
> >Another thing I'm not sure about is the configure check. Maybe it would 
> >be better to use AC_TRY_COMPILE instead of AC_TRY_RUN here?
> 
> AC_TRY_COMPILE is preferred in cross-compilation environment.
> Also instead AC_TRY_RUN we may prefer AC_RUN_IFELSE.

 hum, good point, maybe we should stick to AC_TRY_COMPILE indeed,

Daniel

-- 
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard      | virtualization library  http://libvirt.org/
veillard redhat com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/