homebrew_bootsnap: require some developer tools.

[MikeMcQuaid]

Do a very basic developer tools check to ensure that we can compile things. We cannot use DevelopmentTools.installed? because this has much higher speed requirements and needs to be run before we require anything else.

CC @Bo98 who mentioned this

[BrewTestBot]

Review period skipped due to critical label.

[Bo98]

Looks good. I actually think the check should be only on the LoadError case. If they previously installed bootsnap correctly, then we shouldn't care if the dev tools later disappear, like on a Big Sur update. (I'm not sure how we handle bootsnap updates though - I think we just aren't.)

I think there's some edge cases (like outdated CLT versions) that could go pass this - would it make any sense to make brew doctor invokations not install any bundler gems, so that easy debugging can be performed? I know I'm complicating things - sorry!

[MikeMcQuaid]

I actually think the check should be only on the LoadError case.

Makes sense to me 👍🏻

I think there's some edge cases (like outdated CLT versions) that could go pass this - would it make any sense to make brew doctor invokations not install any bundler gems, so that easy debugging can be performed? I know I'm complicating things - sorry!

I'd rather not special case commands but the failure case should probably be made more graceful.

[Bo98]

the failure case should probably be made more graceful.

Yeah, that would make sense.

[Bo98]

I'll merge this version now since there were a couple of reported problems from people with no dev tools.

[MikeMcQuaid]

Thanks for merging @Bo98.