Horton

Thread

Commits

  1. Add missing pg_basebackup TAP test for meson.

  2. Add missing subscription TAP test for meson

  1. subscription/033_run_as_table_owner is not listed in the meson.build

    Hayato Kuroda (Fujitsu) <kuroda.hayato@fujitsu.com> — 2023-06-15T07:16:06Z

    Dear hackers,

I have noticed that the testcase subscription/033_run_as_table_owner in the
subscription is not executed when meson build system is chosen. The case is not
listed in the meson.build.

Do we have any reasons or backgrounds about it?
PSA the patch to add the case. It works well on my env.

Best Regards,
Hayato Kuroda
FUJITSU LIMITED

  2. Re: subscription/033_run_as_table_owner is not listed in the meson.build

    Michael Paquier <michael@paquier.xyz> — 2023-06-15T08:04:22Z

    On Thu, Jun 15, 2023 at 07:16:06AM +0000, Hayato Kuroda (Fujitsu) wrote:
> I have noticed that the testcase subscription/033_run_as_table_owner in the
> subscription is not executed when meson build system is chosen. The case is not
> listed in the meson.build.
> 
> Do we have any reasons or backgrounds about it?
> PSA the patch to add the case. It works well on my env.

Seems like a thinko of 4826759 to me, that's easy to miss.  Will fix
in a bit..
--
Michael

  3. Re: subscription/033_run_as_table_owner is not listed in the meson.build

    Masahiko Sawada <sawada.mshk@gmail.com> — 2023-06-15T08:32:22Z

    On Thu, Jun 15, 2023 at 5:04 PM Michael Paquier <michael@paquier.xyz> wrote:
>
> On Thu, Jun 15, 2023 at 07:16:06AM +0000, Hayato Kuroda (Fujitsu) wrote:
> > I have noticed that the testcase subscription/033_run_as_table_owner in the
> > subscription is not executed when meson build system is chosen. The case is not
> > listed in the meson.build.
> >
> > Do we have any reasons or backgrounds about it?
> > PSA the patch to add the case. It works well on my env.
>
> Seems like a thinko of 4826759 to me, that's easy to miss.  Will fix
> in a bit..

Good catch.

Checking similar oversights,
src/bin/pg_basebackup/t/011_in_place_tablespace.pl seems not to be
listed in meson.build too.

Regards,

-- 
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com

  4. Re: subscription/033_run_as_table_owner is not listed in the meson.build

    Masahiko Sawada <sawada.mshk@gmail.com> — 2023-06-15T22:15:36Z

    On Thu, Jun 15, 2023 at 5:32 PM Masahiko Sawada <sawada.mshk@gmail.com> wrote:
>
> On Thu, Jun 15, 2023 at 5:04 PM Michael Paquier <michael@paquier.xyz> wrote:
> >
> > On Thu, Jun 15, 2023 at 07:16:06AM +0000, Hayato Kuroda (Fujitsu) wrote:
> > > I have noticed that the testcase subscription/033_run_as_table_owner in the
> > > subscription is not executed when meson build system is chosen. The case is not
> > > listed in the meson.build.
> > >
> > > Do we have any reasons or backgrounds about it?
> > > PSA the patch to add the case. It works well on my env.
> >
> > Seems like a thinko of 4826759 to me, that's easy to miss.  Will fix
> > in a bit..
>
> Good catch.
>
> Checking similar oversights,
> src/bin/pg_basebackup/t/011_in_place_tablespace.pl seems not to be
> listed in meson.build too.

Here is the patch for that.

Regards,

-- 
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com

  5. Re: subscription/033_run_as_table_owner is not listed in the meson.build

    Michael Paquier <michael@paquier.xyz> — 2023-06-15T22:41:51Z

    On Fri, Jun 16, 2023 at 07:15:36AM +0900, Masahiko Sawada wrote:
> On Thu, Jun 15, 2023 at 5:32 PM Masahiko Sawada <sawada.mshk@gmail.com> wrote:
>> Checking similar oversights,
>> src/bin/pg_basebackup/t/011_in_place_tablespace.pl seems not to be
>> listed in meson.build too.
> 
> Here is the patch for that.

Yes, good catch.
--
Michael

  6. Re: subscription/033_run_as_table_owner is not listed in the meson.build

    Masahiko Sawada <sawada.mshk@gmail.com> — 2023-06-16T01:38:19Z

    On Fri, Jun 16, 2023 at 7:42 AM Michael Paquier <michael@paquier.xyz> wrote:
>
> On Fri, Jun 16, 2023 at 07:15:36AM +0900, Masahiko Sawada wrote:
> > On Thu, Jun 15, 2023 at 5:32 PM Masahiko Sawada <sawada.mshk@gmail.com> wrote:
> >> Checking similar oversights,
> >> src/bin/pg_basebackup/t/011_in_place_tablespace.pl seems not to be
> >> listed in meson.build too.
> >
> > Here is the patch for that.
>
> Yes, good catch.

Pushed.

Regards,

-- 
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com