Replace hardcoded drupal_required_modules with parsing .info files for 'required = TRUE'

Holy OMG crap, I think this might work! Just did a test reset and install, and it worked. Patch for review.

More accurate title. This would be sweet. Ran several installs and tests are passing so far...

This is a nice improvement, and I'd like to get this in, but I'd also like someone else's eyes on it first, since I was part of the original group talking about this.

I should mention, one particular concern I have is with required modules that have dependencies. For example, if Module A has direct function calls in it to Module B, esp. in its hook_install(). Module A is earlier in the alphabet, therefore Module B is not enabled at this time. All hell breaks loose?

Naturally, that's what would happen normally. The current function actually gets the required modules in the same order as the hard-coded array. I'd say in that case, that's a perfect use case for hook_modules_installed. :)

Oooookaaayyy. Let's try this. :) Committed to HEAD.

Docs please!

webchick, what and where would you like me to document? I could improve the PHPdoc of drupal_required_modules a little better, but how should we go about letting people know that hidden = TRUE and required = TRUE can be set in .info files? Using those parameters needs to be used with caution. Really the only case for using required = TRUE is a pre-packaged Drupal and should not in any way be used by any contrib modules.

Hm. Good point. Let's call this fixed then. :)

required = TRUE for seems quite useful for custom modules that a site depends on - e.g. client_foo.module. i agree it makes little sense for contrib.

Automatically closed -- issue fixed for two weeks with no activity.

Automatically closed -- issue fixed for two weeks with no activity.

How about Writing .info files (Drupal 7.x)