Re: Add PG CI to older PG releases

2023-08-14 Thread Nazir Bilal Yavuz
Hi, On Fri, 11 Aug 2023 at 02:00, Andres Freund wrote: > At the very least this would need to be combined with > > commit 950e64fa46b164df87b5eb7c6e15213ab9880f87 > Author: Andres Freund > Date: 2022-07-18 17:06:34 -0700 > > Use STDOUT/STDERR_FILENO in most of syslogger. 950e64fa46b164df8

Re: Add PG CI to older PG releases

2023-08-10 Thread Andres Freund
Hi, On 2023-08-10 18:09:03 -0400, Tom Lane wrote: > Nazir Bilal Yavuz writes: > > PG CI is added starting from PG 15, adding PG CI to PG 14 and below > > could be beneficial. So, firstly I tried adding it to the > > REL_14_STABLE branch. If that makes sense, I will try to add PG CI to > > other o

Re: Add PG CI to older PG releases

2023-08-10 Thread Andres Freund
Hi, On 2023-08-10 19:55:15 +0300, Nazir Bilal Yavuz wrote: > v2-0001-windows-Only-consider-us-to-be-running-as-service.patch is an > older commit (59751ae47fd43add30350a4258773537e98d4063). A couple of > tests were failing without this because the log file was empty and the > tests were comparing

Re: Add PG CI to older PG releases

2023-08-10 Thread Tom Lane
Nazir Bilal Yavuz writes: > PG CI is added starting from PG 15, adding PG CI to PG 14 and below > could be beneficial. So, firstly I tried adding it to the > REL_14_STABLE branch. If that makes sense, I will try to add PG CI to > other old PG releases. I'm not actually sure this is worth spending

Re: Add PG CI to older PG releases

2023-08-10 Thread Nazir Bilal Yavuz
Hi, On Thu, 10 Aug 2023 at 18:05, Peter Eisentraut wrote: > I see only one attachment, so it's not clear what these commit hashes > refer to. I split the commit into 3 parts. v2-0001-windows-Only-consider-us-to-be-running-as-service.patch is an older commit (59751ae47fd43add30350a4258773537e98d

Re: Add PG CI to older PG releases

2023-08-10 Thread Peter Eisentraut
On 10.08.23 16:43, Nazir Bilal Yavuz wrote: 1_ 76e38b37a5f179d4c9d2865ff31b79130407530b is added for debugging Windows. Also a couple of SSL tests were failing without this because the log file is empty. Example failures on 001_ssltests.pl: I see only one attachment, so it's not clear what thes