Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Once #375397: Make Node module optional gets in, we should remove Node.module form the minimal profile. User editable content is not always a desired feature, not does it prevent any WTFs, which is why Block or DBlog are part of the minimal profile.
Comment | File | Size | Author |
---|---|---|---|
#7 | remove-node-from-minimal-1370708-7.patch | 1.23 KB | dagmar |
#2 | remove-node-from-minimal-1370708-2.patch | 962 bytes | Devin Carlson |
remove_node_from_minimal_00.patch | 391 bytes | Xano | |
Comments
Comment #2
Devin Carlson CreditAttribution: Devin Carlson commentedRerolled and changed variable_set('site_frontpage', 'node') to variable_set('site_frontpage', 'user').
Comment #3
Everett Zufelt CreditAttribution: Everett Zufelt commentedI'm pretty certain that profile.module refers to the Profile module.
Doesn't appear like there are components for the Core install profiles.
Comment #4
dman CreditAttribution: dman commentedProblem.
Applied patch (cleanly) onto clean checkout.
On running a new install (in a multisite) I got the following error at step "Install profile".
I'm reviewing now to isolate that this isn't a problem from my end or something odd, but right now it looks dodgy.
Comment #5
dman CreditAttribution: dman commentedYeah, I can confirm this kills things.
Comment #6
sunI'd appreciate a re-roll against latest HEAD here. :) #4 should be obsolete by now.
Comment #7
dagmarRe-rolled, and fixed issue mentioned in #4.
Comment #8
sunThis slightly conflicts with #1768758: Change default profile for tests from Testing to Minimal now, but I guess we can find a different solution for that issue.
However, to get buy-in from core committers here, I think we need a much better issue summary with solid arguments.
Comment #8.0
sunCleaned up argumentation.