imho there should be a way to change the constants defined for the system requirements
as 'DRUPAL_MINIMUM_PHP' or 'DRUPAL_MINIMUM_PHP_MEMORY_LIMIT'.
there could be some complex install profiles that requires "more power" from the system.
for example there's a Commerce Kickstart issue about this : #1763632: The install needs to check Kickstart requirements before starting.

Files: 
CommentFileSizeAuthor
#28 drupal-7.x-allow_profile_change_sys_req-1772316-28.patch3.83 KBxtfer
PASSED: [[SimpleTest]]: [MySQL] 40,272 pass(es).
[ View ]
#23 drupal-7.x-allow_profile_change_sys_req-1772316-21.patch4.25 KBhswong3i
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch drupal-7.x-allow_profile_change_sys_req-1772316-21.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#23 drupal-8.x-allow_profile_change_sys_req-1772316-21.patch4.29 KBhswong3i
PASSED: [[SimpleTest]]: [MySQL] 57,993 pass(es).
[ View ]
#18 drupal7-allow_change_system-requirements-1772316-18.patch3.8 KBjsacksick
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch drupal7-allow_change_system-requirements-1772316-18.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#16 1772316_15-allow-profiles-to-override-php-requirements.patch5.11 KBtobyontour
PASSED: [[SimpleTest]]: [MySQL] 55,418 pass(es).
[ View ]
#12 1772316_12-allow-profiles-to-override-php-requirements.patch4.98 KBcweagans
FAILED: [[SimpleTest]]: [MySQL] 54,160 pass(es), 0 fail(s), and 35 exception(s).
[ View ]
#5 allow_change_system-requirements-1772316-5.patch3.83 KBvasike
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch allow_change_system-requirements-1772316-5.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#4 allow_change_system-requirements-1772316-4.patch3.88 KBvasike
PASSED: [[SimpleTest]]: [MySQL] 39,384 pass(es).
[ View ]
#2 allow_change_system-requirements-1772316-2.patch3.8 KBvasike
FAILED: [[SimpleTest]]: [MySQL] 39,564 pass(es), 0 fail(s), and 423 exception(s).
[ View ]
#1 allow_change_system-requirements_constants-1772316-1.patch605 bytesvasike
PASSED: [[SimpleTest]]: [MySQL] 39,391 pass(es).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new605 bytes
PASSED: [[SimpleTest]]: [MySQL] 39,391 pass(es).
[ View ]

here is a patch that allows that
bad approach

are there other system requirements constants that could be changed?

Title:Allow install profiles to change the system requirements constantsAllow install profiles to change the system requirements
StatusFileSize
new3.8 KB
FAILED: [[SimpleTest]]: [MySQL] 39,564 pass(es), 0 fail(s), and 423 exception(s).
[ View ]

here is a new approach for changing the php system requirements.
it uses the install_profile_info definition.

so what it's needed it's to specify new values in the info file of the profile

php = 5.3.12
php_memory_limit = 1234M

Status:Needs review» Needs work

The last submitted patch, allow_change_system-requirements-1772316-2.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new3.88 KB
PASSED: [[SimpleTest]]: [MySQL] 39,384 pass(es).
[ View ]

rebuilt the patch, i hope it's the right one

Version:7.x-dev» 8.x-dev
StatusFileSize
new3.83 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch allow_change_system-requirements-1772316-5.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

and here is the patch for Drupal 8

Component:configuration system» install system

Issue tags:+Needs reroll

Looks like some D7 install profiles have been using #4 and it no longer applies to HEAD as of 7.22. :(

Added tags Needs reroll but there's no way to specify thats a D7 reroll since this is also being tracked as an 8.x issue - sorry.

Status:Needs review» Needs work
Issue tags:+Needs reroll

The last submitted patch, allow_change_system-requirements-1772316-5.patch, failed testing.

Status:Needs work» Needs review

We need to get this into D8 before we worry about D7. I requeued the patch in #5 (I doubt it'll pass - last test run was in September). If that fails, lets reroll that, get it committed, then deal with D7.

Status:Needs review» Needs work

Ah, cross posted with the bot. So yeah, need to reroll that patch.

StatusFileSize
new4.98 KB
FAILED: [[SimpleTest]]: [MySQL] 54,160 pass(es), 0 fail(s), and 35 exception(s).
[ View ]

Let's give this a try. I moved around your new block of code in system.install so that we're not calling drupal_get_profile() multiple times - now it's just called once at the beginning of the function and used throughout.

Status:Needs work» Needs review

Status:Needs review» Needs work

The last submitted patch, 1772316_12-allow-profiles-to-override-php-requirements.patch, failed testing.

Status:Needs work» Needs review
Issue tags:-Needs reroll

Made system.install default to DRUPAL_MINIMUM_PHP_MEMORY_LIMIT if it's not set in the profile (this is what caused the previous patch to fail).

StatusFileSize
new5.11 KB
PASSED: [[SimpleTest]]: [MySQL] 55,418 pass(es).
[ View ]

Patch failed to attach to last post.

I'm not sure about "Drupal requires at least PHP %version.", as this is not true if the profile uses a different max version than Drupal. Drupal with [name of profile] requires this version, yes. Also the same issue in the message about the memory limit.

StatusFileSize
new3.8 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch drupal7-allow_change_system-requirements-1772316-18.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Here's the rerolled patch for d7, against the latest dev

Status:Needs review» Needs work

The last submitted patch, drupal7-allow_change_system-requirements-1772316-18.patch, failed testing.

Status:Needs work» Needs review

Status:Needs review» Needs work

The last submitted patch, drupal7-allow_change_system-requirements-1772316-18.patch, failed testing.

The issue is tagged with the 8.x version which means that the patch will obviously fail..., that's why I didn't change the status earlier... I rerolled the patch for Commerce Kickstart

StatusFileSize
new4.29 KB
PASSED: [[SimpleTest]]: [MySQL] 57,993 pass(es).
[ View ]
new4.25 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch drupal-7.x-allow_profile_change_sys_req-1772316-21.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Patch(es) reroll via all latest dev.

Status:Needs work» Needs review

Note: #309040: Add hook_requirements_alter() is another way to do this that should be a lot more flexible.

In some installations we have problems with mysql max_allowed_packet size set to low. Wouldn't ist make sense to set this settings also as requirement if needed?

Issue summary:View changes
StatusFileSize
new3.83 KB
PASSED: [[SimpleTest]]: [MySQL] 40,272 pass(es).
[ View ]

Updated for D7, as the patch was not applying.

Status:Needs review» Needs work

The last submitted patch, 28: drupal-7.x-allow_profile_change_sys_req-1772316-28.patch, failed testing.

Version:8.x-dev» 7.x-dev
Status:Needs work» Needs review

Switching to 7 to run the test.

Version:7.x-dev» 8.x-dev

Back to 8.x