[Mono-dev] [PATCH] Add default code page to build system
kornelpal at hotmail.com
Tue Sep 13 05:52:47 EDT 2005
Mono 1.1.9 was released so we don't have to worry about branching. Can I
commit the patch?
----- Original Message -----
From: "Kornél Pál" <kornelpal at hotmail.com>
To: "Miguel de Icaza" <miguel at ximian.com>
Cc: <mono-devel-list at lists.ximian.com>
Sent: Tuesday, September 06, 2005 3:31 PM
Subject: Re: [Mono-dev] [PATCH] Add default code page to build system
> This was a reply to your comments on the patch. The patch is in the
> message. Note that my previos message a second reply as you simply ignored
> the first one so last time I tried to include only relevant information.
> I included a diff file based on the latest SVN revision for your
> The patch in not critical so can go in after 1.1.9 but I would like to get
> some feedback. Is it OK to commit? Should something be changed?
> Messages in this thread:
> Original message including the patch:
> Your reply:
> My first reply:
> My second reply:
> Your previous reply:
> ----- Original Message -----
> From: "Miguel de Icaza" <miguel at ximian.com>
> To: "Kornél Pál" <kornelpal at hotmail.com>
> Cc: <mono-devel-list at lists.ximian.com>
> Sent: Tuesday, September 06, 2005 3:07 PM
> Subject: Re: [Mono-dev] [PATCH] Add default code page to build system
>>> Our compiler, mcs uses ISO-28591 as the default code page. This means
>>> the whole class library is compiled using this code page and we seem to
>>> this code page (ISO-28591) as it compiles. The only exception is
>>> Microsoft.VisualBasic. It uses UTF-8 for a long time and it uses
>>> /codepage:650001 in it's Makefile.
>>> The patch does not change the code page we use. Currently it's primary
>>> is that we should specify our default code page on mcs code page instead
>>> in mcs source code that should use Encoding.Default as the default code
>>> page. Specifying default code page on command line let us use csc.exe as
>>> well on all systems at it defaults to Encoding.Default.
>>> Is the patch OK to commit or are there any other issues?
>> You forgot to attach the patch.
>> We are about to freeze the tree for the 1.1.9 release, so a patch like
>> this should not go in until after the release.
> Mono-devel-list mailing list
> Mono-devel-list at lists.ximian.com
More information about the Mono-devel-list