Re: PR/39765 CVS commit: pkgsrc/devel/glibmm

[Joerg Sonnenberger <joerg%britannica.bec.de@localhost>]

On Tue, Feb 03, 2015 at 11:55:02AM +0000, Adam Ciarcinski wrote:
> The following reply was made to PR pkg/39765; it has been noted by GNATS.
> 
> From: "Adam Ciarcinski" <adam%netbsd.org@localhost>
> To: gnats-bugs%gnats.NetBSD.org@localhost
> Cc: 
> Subject: PR/39765 CVS commit: pkgsrc/devel/glibmm
> Date: Tue, 3 Feb 2015 11:50:43 +0000
> 
>  Module Name:	pkgsrc
>  Committed By:	adam
>  Date:		Tue Feb  3 11:50:43 UTC 2015
>  
>  Modified Files:
>  	pkgsrc/devel/glibmm: distinfo
>  Added Files:
>  	pkgsrc/devel/glibmm/patches: patch-glib_glibmm_ustring.cc
>  
>  Log Message:
>  Work-around NetBSD's native iconv lacking support for WCHAR_T.
>  See also PR 39765.
>  Need for graphics/inkscape 0.91.

The code is very suspicious. We do *not* store UCS4 in wchar_t and we do
*not* support the translation to wchar_t via iconv, because it is
completely undefined what such behavior should be. Really, this code
should be using mbstowcs or similar and iconv.

Joerg