Move block theme definition from system module to block module

Looks reasonable to me. Tests pass, so RTBC.

Downloaded patch,

Applied but has CR endings. Should be unix line endings.

Other than that everything seems to work fine. Tried switching to the Stark theme which uses the default block.tpl.php and everything worked as expected.

I think this patch makes sense, no reason in having drupal load code that isn't being used if the block module is disabled. It's also a very simple patch, mostly just moving code from one file to another, doesn't add anything "new".

Attached patch fixes the line ending issues.

Oops, looks like my patch failed to create the new file in modules/block/, I think this one should fix that. Changing to needs review so the test bot can have another go at it.

Assuming this passes testing I would say it is ready to be committed.

I think this one is ready to go. Like I mentioned before the original patch is really simple and really only moves some code from one place to another. Doesn't add any new functionality and makes perfect sense now that block module is optional. My re-roll was only done to eliminate line ending errors that I got when trying to apply the patch.

Seems logical to me! I also ran this by quicksketch and he pointed out that such a patch would allow betterblock.module to also define its own block.tpl.php so that themes wouldn't have to break. Seems like pretty straight-forward unfinished business from the making block module optional patch.

Unless I'm wrong, this doesn't seem to impact any themers directly, so can just be marked fixed without the need for update documentation, correct?

Correct. This shouldn't have any implications for themers, the only change is that they would now need to look in modules/block/ to find the block.tpl.php default implementation instead of modules/system/. Which, as someone who will probably be looking for it in the future, that makes a whole lot more sense as a place to start looking.