pg.ddx.io pgsql-committers@postgresql.org mailing list archivehelp / color / mirror / Atom feed
pgsql: meson: ci: ensure tests are built before running them 3+ messages / 1 participants [nested] [flat]
* pgsql: meson: ci: ensure tests are built before running them @ 2025-02-04 23:19 Andres Freund <andres@anarazel.de> 0 siblings, 0 replies; 3+ messages in thread From: Andres Freund @ 2025-02-04 23:19 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org meson: ci: ensure tests are built before running them Meson 1.7 stopped building all the dependencies of tests as part of the default build target. But it does breaks CI because we only built the default target before running the test, and ran the tests with --no-rebuild. The simplest fix would be to remove --no-rebuild from MTEST_ARGS, but it seems better to explicitly build the test dependencies, so compiler warnings / errors are visible as part of the build step. Discussion: https://postgr.es/m/CAGECzQSvM3iSDmjF+=Kof5an6jN8UbkP_4cKKT9w6GZavmb5yQ@mail.gmail.com Backpatch: 16-, where meson was added Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/b2bdb972c0df3b7091e39d1a31484e5e065519f4 Modified Files -------------- .cirrus.tasks.yml | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) ^ permalink raw reply [nested|flat] 3+ messages in thread
* pgsql: meson: ci: ensure tests are built before running them @ 2025-02-04 23:19 Andres Freund <andres@anarazel.de> 0 siblings, 0 replies; 3+ messages in thread From: Andres Freund @ 2025-02-04 23:19 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org meson: ci: ensure tests are built before running them Meson 1.7 stopped building all the dependencies of tests as part of the default build target. But it does breaks CI because we only built the default target before running the test, and ran the tests with --no-rebuild. The simplest fix would be to remove --no-rebuild from MTEST_ARGS, but it seems better to explicitly build the test dependencies, so compiler warnings / errors are visible as part of the build step. Discussion: https://postgr.es/m/CAGECzQSvM3iSDmjF+=Kof5an6jN8UbkP_4cKKT9w6GZavmb5yQ@mail.gmail.com Backpatch: 16-, where meson was added Branch ------ REL_17_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/d38fd59d2a16469a9bf5797176094fb418792fae Modified Files -------------- .cirrus.tasks.yml | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) ^ permalink raw reply [nested|flat] 3+ messages in thread
* pgsql: meson: ci: ensure tests are built before running them @ 2025-02-04 23:19 Andres Freund <andres@anarazel.de> 0 siblings, 0 replies; 3+ messages in thread From: Andres Freund @ 2025-02-04 23:19 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org meson: ci: ensure tests are built before running them Meson 1.7 stopped building all the dependencies of tests as part of the default build target. But it does breaks CI because we only built the default target before running the test, and ran the tests with --no-rebuild. The simplest fix would be to remove --no-rebuild from MTEST_ARGS, but it seems better to explicitly build the test dependencies, so compiler warnings / errors are visible as part of the build step. Discussion: https://postgr.es/m/CAGECzQSvM3iSDmjF+=Kof5an6jN8UbkP_4cKKT9w6GZavmb5yQ@mail.gmail.com Backpatch: 16-, where meson was added Branch ------ REL_16_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/d9b5e2fc09b65a6512561c7dbb97e0836dcf933e Modified Files -------------- .cirrus.tasks.yml | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) ^ permalink raw reply [nested|flat] 3+ messages in thread
end of thread, other threads:[~2025-02-04 23:19 UTC | newest] Thread overview: 3+ messages (download: mbox mbox.gz follow: Atom feed) -- links below jump to the message on this page -- 2025-02-04 23:19 pgsql: meson: ci: ensure tests are built before running them Andres Freund <andres@anarazel.de> 2025-02-04 23:19 pgsql: meson: ci: ensure tests are built before running them Andres Freund <andres@anarazel.de> 2025-02-04 23:19 pgsql: meson: ci: ensure tests are built before running them Andres Freund <andres@anarazel.de>
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