pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
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
Date: Thu, 19 Sep 2024 08:45:49 -0400
Message-ID: <Zuwc_d1agerYf3xH@momjian.us> (raw)
In-Reply-To: <b2465837-56df-4794-a0b5-5e6ed44ed870@dunslane.net>
References: <E1sqFV9-001ODn-9M@gemulon.postgresql.org>
	<b2465837-56df-4794-a0b5-5e6ed44ed870@dunslane.net>

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?"





view thread (9+ messages)

Message-ID: <Zuwc_d1agerYf3xH@momjian.us>
Permalink:  ../Zuwc_d1agerYf3xH@momjian.us/
Also on:    postgresql.org/message-id/Zuwc_d1agerYf3xH@momjian.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-committers@postgresql.org
  Cc: bruce@momjian.us, andrew@dunslane.net, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: scripts: add Perl script to add links to release notes
  In-Reply-To: <Zuwc_d1agerYf3xH@momjian.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