Re: remove separate postgres.(sh)description files

From: Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>
To: Heikki Linnakangas <hlinnaka(at)iki(dot)fi>
Cc: John Naylor <john(dot)naylor(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: remove separate postgres.(sh)description files
Date: 2020-01-16 21:39:49
Message-ID: 20200116213949.2lb7rdafmtztls66@development
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On Wed, Jan 08, 2020 at 02:33:23PM +0200, Heikki Linnakangas wrote:
>On 31/12/2019 02:08, John Naylor wrote:
>>I'm guessing the initial data for pg_(sh)description is output into
>>separate files because it was too difficult for the traditional shell
>>script to maintain enough state to do otherwise.
>Yeah, I guess so. The roots of postgres.description goes all the way
>back to 1997, when not only genbki was a shell script, but also
>>With Perl, it's just as easy to assemble the data into the same
>>format as the rest of the catalogs and then let the generic code path
>>output it into postgres.bki. The attached patch does that and
>>simplifies the catalog makefile and initdb.c.
>Nice cleanup! Looks like we didn't have any mention of the
>postgres.(sh)decription files in the docs, so no doc updates needed.
>Grepping around, there are a few stray references to
>postgres.description still:
>$ git grep -r -I postgres.shdescript .
>src/backend/catalog/Makefile:# postgres.bki, postgres.description,
>src/tools/msvc/clean.bat:if %DIST%==1 if exist
>src\backend\catalog\postgres.shdescription del /q
>Barring objections, I'll remove those too, and commit this.

+1 from me. Let's remove these small RFC patches out of the way.


Tomas Vondra
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Tomas Vondra 2020-01-16 21:45:25 Re: Run-time pruning for ModifyTable
Previous Message Tomas Vondra 2020-01-16 21:38:20 Re: Recognizing superuser in pg_hba.conf