On 10/2/21 11:34 PM, Tom Lane wrote: > Andrew Dunstan <and...@dunslane.net> writes: >> On 10/2/21 5:03 PM, Tom Lane wrote: >>> IIUC, the only problem for a non-updated animal would be that it'd >>> run the test twice? Or would it actually fail? If the latter, >>> we'd need to sit on the patch rather longer. >> The patch removes test.sh, so yes it would break. > Maybe we could leave test.sh in place for awhile? I'd rather > not cause a flag day for buildfarm owners. (Also, how do we > see this working in the back branches?) > >
Actually, I was wrong. The module just does "make check" for non-MSVC. For MSVC it calls vcregress.pl, which the patch doesn't touch (it should, I think). cheers andrew -- Andrew Dunstan EDB: https://www.enterprisedb.com