http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5748





------- Additional Comments From [EMAIL PROTECTED]  2007-12-23 19:37 -------
Created an attachment (id=4213)
 --> (http://issues.apache.org/SpamAssassin/attachment.cgi?id=4213&action=view)
Proposed fix, patch file can be applied both to trunk and 3.2 branch

This was a surprise. The sort that was producing the error message suggesting
LC_ALL='C' is not in perl, it is a call out to the system sort program in the
open call in sub rule2xs. The fix is to change "sort $FILE |" to
 "LC_ALL='C' sort $FILE |" and then sa-compile completes fine.

I'm uneasy about this because calling out to sort seems like it would be
platform dependent, and adding in the LC_ALL='C' prefix to it seems even more
so. Is sa-update only supposed to work on unix/linux platforms? Do we want to
use some pure perl sort in this place?

I haven't committed this patch to trunk yet because I would like to hear
opinions first.




------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Reply via email to