- Notifications
You must be signed in to change notification settings - Fork28
Commitf285f23
committed
Fix up confusion over how to use EXTRA_INSTALL.
Some makefiles were trying to do this:temp-install: EXTRA_INSTALL=contrib/test_decodingbut that no longer works as of commitaa019da: the macro is nowconsulted by the checkprep target, one level down, and apparentlygmake doesn't propagate such macro settings recursively.The problem is masked since42e61c7 because pgxs.mk also sets upEXTRA_INSTALL, and correctly applies it to the checkprep target.Unfortunately I'd not risked back-patching that to before v11.Sinceaa019da was pushed back to v10, it broke test_decodingthere (the only module for which this actually makes a differenceat present).Hence, back-patch42e61c7 to v10. Also, remove some demonstrablyuseless settings of EXTRA_INSTALL in v10 and v11 (they'd alreadybeen cleaned up in HEAD).Per buildfarm.Discussion:https://postgr.es/m/CAEepm=1pEJdwv6DSGmOfpX0EaX7L7sT28c1nXpqvQvmLfEWb1g@mail.gmail.com1 parent89d52b9 commitf285f23
2 files changed
+0
-4
lines changedLines changed: 0 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
65 | 65 |
| |
66 | 66 |
| |
67 | 67 |
| |
68 |
| - | |
69 |
| - |
Lines changed: 0 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
43 | 43 |
| |
44 | 44 |
| |
45 | 45 |
| |
46 |
| - | |
47 |
| - |
0 commit comments
Comments
(0)