Date: Thu, 19 Sep 2024 08:45:49 -0400
From: Bruce Momjian <bruce@momjian.us>
To: Andrew Dunstan <andrew@dunslane.net>
Cc: pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: scripts: add Perl script to add links to release notes
Message-ID: <Zuwc_d1agerYf3xH@momjian.us>
References: <E1sqFV9-001ODn-9M@gemulon.postgresql.org>
 <b2465837-56df-4794-a0b5-5e6ed44ed870@dunslane.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <b2465837-56df-4794-a0b5-5e6ed44ed870@dunslane.net>
Archived-At: 
 <https://www.postgresql.org/message-id/Zuwc_d1agerYf3xH%40momjian.us>
Precedence: bulk

On Tue, Sep 17, 2024 at 09:47:57AM -0400, Andrew Dunstan wrote:
> 
> On 2024-09-16 Mo 1:27 PM, Bruce Momjian wrote:
> 
>     scripts:  add Perl script to add links to release notes
> 
>     Reported-by: jian he
> 
>     Discussion: https://postgr.es/m/ZuYsS5XdA7hVcV9l@momjian.us
> 
> 
> I realize I'm late to the party on this, but here's a review patch for this
> script. It's mostly stylistic changes (simpler use of print, use of POSIX
> xdigit class for hex digits), but it does correct one actual error: AFAIK "die"
> doesn't do printf style substitutions.

Patch applied.  Sorry for the delay.

-- 
  Bruce Momjian  <bruce@momjian.us>        https://momjian.us
  EDB                                      https://enterprisedb.com

  When a patient asks the doctor, "Am I going to die?", he means 
  "Am I going to die soon?"