Re: CFbot does not recognize patch contents

From: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
To: Tatsuo Ishii <ishii(at)sraoss(dot)co(dot)jp>
Cc: byavuz81(at)gmail(dot)com, pgsql-hackers(at)postgresql(dot)org
Subject: Re: CFbot does not recognize patch contents
Date: 2024-05-15 02:54:39
Message-ID: CA+hUKGKZJC+xqnMzeoRYKswzr2H1Mz65K1M3iJGqBD0YmnQ=YA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, May 12, 2024 at 10:11 PM Tatsuo Ishii <ishii(at)sraoss(dot)co(dot)jp> wrote:
> > I am able to apply all your patches. I found that a similar thing
> > happened before [0] and I guess your case is similar. Adding Thomas to
> > CC, he may be able to help more.
>
> Ok. Thanks for the info.

This obviously fixed itself automatically soon after this message, but
I figured out what happened: I had not actually fixed that referenced
bug in cfbot :-(. It was checking for HTTP error codes correctly in
the place that reads emails from the archives, but not the place that
downloads patches, so in this case I think when it tried to follow the
link[1] to download the patch, I guess it must have pulled down a
transient Varnish error message (I don't know what, I don't store it
anywhere), and tried to apply that as a patch. Oops. Fixed[2].

[1] https://www.postgresql.org/message-id/attachment/160138/v18-0001-Row-pattern-recognition-patch-for-raw-parser.patch
[2] https://github.com/macdice/cfbot/commit/ec33a65a877a88befc29ea220e87b98c89b27dcc

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message David Rowley 2024-05-15 03:03:00 Re: Fixup a few 2023 copyright years
Previous Message David Rowley 2024-05-15 02:24:15 Re: First draft of PG 17 release notes