Theme engines cannot use multi-site feature

- Remove commented debug code
- While we are here, those changed lines need to punctuate '. $variables .' according to code style.

What is a good way to test this? Does PHPTAL in contrib HEAD work well enough?

I don't know much about theme engines so I can't really comment on the merits of the patch (it looks good to me), but just to nitpick the debugging line needs to be removed and the string concatenation spacing needs to be fixed.

Why is this critical? It's not a regression compared to Drupal 4.7, is it?

This isn't a regression; rather, it's a feature that is intended to work in a particular way (allowing themes, modules, engines, etc to be stored in sites/all rather than the root drupal directory). It's not working in that intended fashion, mostly because we never tested it with additional non-core theme engines.

It changes no APIs, as none of the code involved is used externally; it just makes things work the way we say they do in the documentation, etc. :-)

Eaton: I know and we should fix this. It's just that it doesn't have to hold up a release, IMO. Critical bugs are meant to be showstopping bugs.

Fair enough -- normal is probably the best priority level for it. :)

Committed to HEAD.