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

Started by Hayato Kuroda (Fujitsu)almost 3 years ago6 messageshackers
Jump to latest
#1Hayato Kuroda (Fujitsu)
kuroda.hayato@fujitsu.com

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

Attachments:

add_test.patchapplication/octet-stream; name=add_test.patchDownload+1-0
#2Michael Paquier
michael@paquier.xyz
In reply to: Hayato Kuroda (Fujitsu) (#1)
Re: subscription/033_run_as_table_owner is not listed in the meson.build

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

#3Masahiko Sawada
sawada.mshk@gmail.com
In reply to: Michael Paquier (#2)
Re: subscription/033_run_as_table_owner is not listed in the meson.build

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

#4Masahiko Sawada
sawada.mshk@gmail.com
In reply to: Masahiko Sawada (#3)
Re: subscription/033_run_as_table_owner is not listed in the meson.build

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

Attachments:

0001-Add-missing-pg_basebackup-TAP-test-for-meson.patchapplication/octet-stream; name=0001-Add-missing-pg_basebackup-TAP-test-for-meson.patchDownload+1-1
#5Michael Paquier
michael@paquier.xyz
In reply to: Masahiko Sawada (#4)
Re: subscription/033_run_as_table_owner is not listed in the meson.build

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

#6Masahiko Sawada
sawada.mshk@gmail.com
In reply to: Michael Paquier (#5)
Re: subscription/033_run_as_table_owner is not listed in the meson.build

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