Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1keiQf-0001U2-BZ for pgsql-committers@arkaria.postgresql.org; Mon, 16 Nov 2020 17:36:37 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1keiQd-0002lu-VF for pgsql-committers@arkaria.postgresql.org; Mon, 16 Nov 2020 17:36:35 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1keiQd-0002lk-Pn for pgsql-committers@lists.postgresql.org; Mon, 16 Nov 2020 17:36:35 +0000 Received: from momjian.us ([72.94.173.45]) by makus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1keiQb-0004Yw-NW for pgsql-committers@lists.postgresql.org; Mon, 16 Nov 2020 17:36:34 +0000 Received: from bruce by momjian.us with local (Exim 4.92) (envelope-from ) id 1keiQa-0007Ya-83; Mon, 16 Nov 2020 12:36:32 -0500 Date: Mon, 16 Nov 2020 12:36:32 -0500 From: Bruce Momjian To: Alvaro Herrera Cc: pgsql-committers@lists.postgresql.org Subject: Re: pgsql: doc: clarify where to find pg_type_d.h (PG 11+) and pg_type.h Message-ID: <20201116173632.GD22426@momjian.us> References: <20201112202900.GA28098@alvherre.pgsql> <20201112231551.GF4506@momjian.us> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20201112231551.GF4506@momjian.us> User-Agent: Mutt/1.10.1 (2018-07-13) List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk On Thu, Nov 12, 2020 at 06:15:51PM -0500, Bruce Momjian wrote: > On Thu, Nov 12, 2020 at 05:29:00PM -0300, Álvaro Herrera wrote: > > On 2020-Nov-12, Bruce Momjian wrote: > > > > > doc: clarify where to find pg_type_d.h (PG 11+) and pg_type.h > > > > > > These files are in compiled directories and install directories. > > > > I think this patch is wrong, because in the install tree the files are in > > the server/ subdir. It should read: > > > > The OIDs of the built-in data types are defined in > > the file include/catalog/pg_type_d.h in the > > compiled source directory, and in > > include/server/catalog/pg_type_d.h in the install > > directory. > > > > That said, if the user is doing a VPATH build, then "compiled source > > directory" is a confusing way to say "the build directory", and I don't > > think it's an important location anyway, first because it's really a > > throwaway directory, and second it's not even there in a packaged > > installation; users just care about the *installed* file, don't they? > > So I suggest to rip that part out, and leave just: > > > > The OIDs of the built-in data types are defined in > > the file include/server/catalog/pg_type_d.h in the > > install directory. > > OK, how is this followup patch? Applied to all branches, thanks. -- Bruce Momjian https://momjian.us EnterpriseDB https://enterprisedb.com The usefulness of a cup is in its emptiness, Bruce Lee