It does sound a lot like we're missing some test cases as if I
understand it correctly, that class can't have worked for a while?

I've used the go-vendor class on Scarthgap and have only started to

look at it on Wrynose with the recent release. It would have thrown

an error since whenever the insane.bbclass explicitly denied setting

S = "${UNPACKDIR}/git". It's probably slipped under the radar

because nothing uses it in the core layers.

John