From owner-freebsd-ports@FreeBSD.ORG Wed Dec 19 14:45:11 2007 Return-Path: Delivered-To: ports@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 7870F16A41B; Wed, 19 Dec 2007 14:45:11 +0000 (UTC) (envelope-from jdc@parodius.com) Received: from mx01.sc1.parodius.com (mx01.sc1.parodius.com [72.20.106.3]) by mx1.freebsd.org (Postfix) with ESMTP id 6CD4313C45A; Wed, 19 Dec 2007 14:45:11 +0000 (UTC) (envelope-from jdc@parodius.com) Received: by mx01.sc1.parodius.com (Postfix, from userid 1000) id 4D8641CC038; Wed, 19 Dec 2007 06:45:11 -0800 (PST) Date: Wed, 19 Dec 2007 06:45:11 -0800 From: Jeremy Chadwick To: Barry Byrne Message-ID: <20071219144511.GA15338@eos.sc1.parodius.com> References: <000f01c8422b$3515d7b0$c5010c0a@SUNYA> <20071219113418.GA11378@eos.sc1.parodius.com> <006501c8424b$975d8c90$c5010c0a@SUNYA> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <006501c8424b$975d8c90$c5010c0a@SUNYA> User-Agent: Mutt/1.5.16 (2007-06-09) Cc: ports@FreeBSD.org, hiromi@tac.tsukuba.ac.jp, beech@FreeBSD.org, perky@freebsd.org Subject: Re: FreeBSD Port: horde-base-3.1.5_1 X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 19 Dec 2007 14:45:11 -0000 On Wed, Dec 19, 2007 at 02:29:40PM -0000, Barry Byrne wrote: > > -----Original Message----- > > From: Jeremy Chadwick [mailto:koitsu@FreeBSD.org] > > Sent: 19 December 2007 11:34 > > To: Barry Byrne > > Cc: beech@FreeBSD.org; ports@FreeBSD.org; > > hiromi@tac.tsukuba.ac.jp; ache@freebsd.org; perky@freebsd.org > > > > Checked the patched file (NLS.php) and I had something like this; > > > > > > \xa0 \xa0 \xa0 \xa0/* avoid FreeBSD issapce(3) bug */ > > > \xa0 \xa0 \xa0 \xa0if(NLS::getCharset() == "UTF-8"){ > > > \xa0 \xa0 \xa0 \xa0 \xa0 \xa0setlocale(LC_CTYPE,"C"); > > > \xa0 \xa0 \xa0 \xa0} > > > > > > When I replaced the \xa0 with tabs, all was fine again. > > > > The parse error is probably because PHP does not have Unicode or > > multibyte character support enabled. The patch file in question > > contains actual Unicode characters. You might try rebuilding PHP with > > Zend support for multibyte characters (it's an option in > > "make config") > > and see if that improves things. > > Jeremy, > > Thanks for the quick response. > > You're correct in assuming I don't have Multibyte enabled. > > However, I just rebuilt php on another machine and enabled multibyte, > then built horde-base on that, but still had the same characters > in the NLS.php file on that, so not sure what the problem is. Okay, so it's probably a Unicode vs. UTF-8 parser thing. My comment about the code needing to remain in ASCII stands, though (this isn't your fault, of course). The patch should really be fixed to use literal whitespace (ASCII 0x20) or tabs (ASCII 0x09). I have commit access to fix this, but I want to clear it + discuss with those involved with the PR first. -- | Jeremy Chadwick jdc at parodius.com | | Parodius Networking http://www.parodius.com/ | | UNIX Systems Administrator Mountain View, CA, USA | | Making life hard for others since 1977. PGP: 4BD6C0CB |