- Notifications
You must be signed in to change notification settings - Fork4.9k
Commitd9b5e2f
committed
meson: ci: ensure tests are built before running them
Meson 1.7 stopped building all the dependencies of tests as part of thedefault build target. But it does breaks CI because we only built the defaulttarget 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 seemsbetter 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.comBackpatch: 16-, where meson was added1 parentad80e58 commitd9b5e2f
1 file changed
+10
-7
lines changedLines changed: 10 additions & 7 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
17 | 17 |
| |
18 | 18 |
| |
19 | 19 |
| |
| 20 | + | |
| 21 | + | |
| 22 | + | |
20 | 23 |
| |
21 | 24 |
| |
22 | 25 |
| |
| |||
99 | 102 |
| |
100 | 103 |
| |
101 | 104 |
| |
102 |
| - | |
| 105 | + | |
103 | 106 |
| |
104 | 107 |
| |
105 | 108 |
| |
| |||
176 | 179 |
| |
177 | 180 |
| |
178 | 181 |
| |
179 |
| - | |
| 182 | + | |
180 | 183 |
| |
181 | 184 |
| |
182 | 185 |
| |
| |||
378 | 381 |
| |
379 | 382 |
| |
380 | 383 |
| |
381 |
| - | |
382 |
| - | |
| 384 | + | |
| 385 | + | |
383 | 386 |
| |
384 | 387 |
| |
385 | 388 |
| |
| |||
498 | 501 |
| |
499 | 502 |
| |
500 | 503 |
| |
501 |
| - | |
| 504 | + | |
502 | 505 |
| |
503 | 506 |
| |
504 | 507 |
| |
| |||
571 | 574 |
| |
572 | 575 |
| |
573 | 576 |
| |
574 |
| - | |
| 577 | + | |
575 | 578 |
| |
576 | 579 |
| |
577 | 580 |
| |
| |||
629 | 632 |
| |
630 | 633 |
| |
631 | 634 |
| |
632 |
| - | |
| 635 | + | |
633 | 636 |
| |
634 | 637 |
| |
635 | 638 |
| |
|
0 commit comments
Comments
(0)