As per title, required modules shouldn't be listed in default.profile. Add this to system.install (if it's needed at all? these modules must be enabled and can't be disabled...).

Patch to follow.

CommentFileSizeAuthor
#2 required.move.patch1.52 KBRobRoy

Comments

boris mann’s picture

boris mann commented

I tried just removing the required modules. Bad things happen -- it appears the default modules really aren't included at all, and it's really hard to get the system back into a usable state.

We need to put some entries into system.install, but I haven't been able to figure out what yet.

Related to #104763.

RobRoy’s picture

RobRoy commented
Component: other » update system
Status: Active » Needs review
StatusFileSize
new required.move.patch1.52 KB

Here's a patch that hardcodes the required core modules (listed in that package on the admin modules page) to drupal_verify_profile(). Tested it out by returning an empty array() for my install profile and the install worked.

drumm’s picture

drumm
he/him
Location NY, US
commented
Version: 5.x-dev » 6.x-dev
ChrisKennedy’s picture

ChrisKennedy commented
Status: Needs review » Closed (duplicate)

This is by design, see Eaton's comment at http://drupal.org/node/108272 and Ralf's proposed solution. Marking this one as the duplicate since 108272 has more discussion on it.