svn commit: samba r21396 - in branches: SAMBA_3_0/source SAMBA_3_0/source/nsswitch SAMBA_3_0_25/source SAMBA_3_0_25/source/nsswitch

[Timur I. Bakeyev]

Hi, guys!

On Mon, Feb 19, 2007 at 05:43:43AM -0800, Herb Lewis wrote:
> Go ahead and back it out if you like. I don't
> see the problem with leaving it in as the new
> tests actually check for working conversions
> being available as well and should catch any
> problem cases unless I am missing something.
> 
> I'll just have it as a local patch.

I hoped, that we agreed once and forever that biconv won't strike back
in the Samba code. And, what do I see now?

https://bugzilla.samba.org/show_bug.cgi?id=4832

The damn thing just doesn't work, it doesn't recognize UCS-2LE encoding,
so all the tests for iconv fail. But due to the logic in aclocal.m4, the
perfectly working libiconv is not seen, so, effectively leaving end users
without codepage support at all.

The library biconv was written as a replacement for iconv by FreeBSD
hacker, for FreeBSD and it gives bad results with Samba on FreeBSD. From
other side, well known and widely deployed libiconv works perfectly ok
on FreeBSD, so I see absolutely NO reason to have support for the broken
library in the Samba code. I may sound outraged here, but we had to deal
with this brokeness since 2005 at least, I convinced Jerry to remove
this code in late 2006 and now we are back where we were before.

I really hope not to see this library in upcoming 3.0.25c release and
leave it in it's tomb where it belongs to.

With regards,
Timur Bakeyev.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.samba.org/archive/samba-technical/attachments/20070729/48804164/attachment.bin