pg.ddx.io  pgsql-bugs@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Thomas Munro <thomas.munro@gmail.com>
Cc: Jan Březina <2janbrezina@gmail.com>
Cc: pgsql-bugs@lists.postgresql.org
Subject: Re: BUG #18219: libpq does not take into consideration UNICODE define
Date: Thu, 30 Nov 2023 17:32:47 -0500
Message-ID: <526070.1701383567@sss.pgh.pa.us> (raw)
In-Reply-To: <CA+hUKGLHjbNEqJUBYYU6izDBUxysHPqcEC=JwJoFR7fuFREr3A@mail.gmail.com>
References: <18219-e052656ac68bab94@postgresql.org>
	<CA+hUKGJc-GqVixCSnm0WiDrcxsingWix3MuZPONr7xJxXiggcg@mail.gmail.com>
	<CABODNCzp04S1UKt+mr8GyY1LYxpXDLpT6PYOvBaS=+bBNfTEGw@mail.gmail.com>
	<CA+hUKGJPn7+Uxz2Y5o=O6+1Hwmg7e272onsnzhODmDc0vJggUg@mail.gmail.com>
	<CABODNCyDZLbA1pK0YyxkjrbpN2mEu=hDBkeGxhJbHR-LFR3mkw@mail.gmail.com>
	<CA+hUKGJmn+wj6g=rd5_VEmraDYQ4craOFrsB6Ff-Ob5XKupcEA@mail.gmail.com>
	<CA+hUKGLHjbNEqJUBYYU6izDBUxysHPqcEC=JwJoFR7fuFREr3A@mail.gmail.com>

Thomas Munro <thomas.munro@gmail.com> writes:
>> Seems like a non-starter.

> ... and I should probably explain why.  This project has difficulty
> maintaining Windows support, because of a general lack of PostgreSQL
> developers working on that platform (but we welcome more! patches
> welcome!) and a disproportionately high amount of maintenance work
> that it generates.  One factor is that there are several different
> Windows configurations (different compilers, C runtimes, build
> systems, file system semantics, partial POSIX emulations like MSYS and
> Cygwin...) to consider.  I doubt we'd want to add another dimension to
> that problem space by saying we have to maintain and test the UNICODE
> and non-UNICODE code paths in our libraries.

Yeah, I think there's no chance that we'd accept such a patch even if
one were submitted.  Quite aside from the initial development effort,
the ongoing maintenance cost would be large, and the value is just not
there (as evidenced by the approximately zero previous requests we've
had for this).

> Hence my intuition that we should be figuring out where the right
> place is to suppress or undefine UNICODE.

I'm not even excited about that.  Would we accept patches to #undef
the random other system-API-changing symbols that exist on other
platforms?  We've not done so in the past and I don't see a great
argument to start here.  IMO the submitter has simply misconfigured
his project.  He should not assume that he can build other peoples'
code with whatever configuration suits his own code.

			regards, tom lane





view thread (12+ messages)  latest in thread

Message-ID: <526070.1701383567@sss.pgh.pa.us>
Permalink:  ../526070.1701383567@sss.pgh.pa.us/
Also on:    postgresql.org/message-id/526070.1701383567@sss.pgh.pa.us

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-bugs@postgresql.org
  Cc: tgl@sss.pgh.pa.us, thomas.munro@gmail.com, 2janbrezina@gmail.com, pgsql-bugs@lists.postgresql.org
  Subject: Re: BUG #18219: libpq does not take into consideration UNICODE define
  In-Reply-To: <526070.1701383567@sss.pgh.pa.us>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox