Re: [PATCH] Xtensa strcoll problem with LOCALE support

2008-09-18 Thread Carmelo AMOROSO
Chris Zankel wrote: Hi Carmelo, Sorry, I also forgot to add the 'signed-off' line. Looking at the log file, however, it seems that less than 5% of the commits are 'signed-off'. What's the policy? I'm used to sign-off my work being this a common/required policy on kernel side. I think

Re: [PATCH] Xtensa strcoll problem with LOCALE support

2008-09-15 Thread Bob Wilson
Carmelo AMOROSO wrote: Hi Bob, I don't see any problems in committing this on behalf of you, but proper signed-off-by should be added to these set of patches, as a general rule of thumbs. Possibly Acked-by Xtensa Maintainer (Chris Zankel) It looks like Chris already committed them, but

Re: [PATCH] Xtensa strcoll problem with LOCALE support

2008-09-15 Thread Carmelo AMOROSO
Bob Wilson wrote: Carmelo AMOROSO wrote: Hi Bob, I don't see any problems in committing this on behalf of you, but proper signed-off-by should be added to these set of patches, as a general rule of thumbs. Possibly Acked-by Xtensa Maintainer (Chris Zankel) It looks like Chris already

[PATCH] Xtensa strcoll problem with LOCALE support

2008-09-11 Thread Bob Wilson
The Xtensa-specific strcmp.S file is testing the __UCLIBC_HAS_LOCALE__ macro but it does not include features.h, so that macro is never defined. This causes problems with strcoll when locale support is enabled. Index: libc/string/xtensa/strcmp.S