D7 Remove the open_basedir requirement check

This can probably go in, but the Drupal 8 issue needs to get fixed first - postponing it on that.

Removing tag until D8 fix goes in, instead bumping priority so it doesn't get forgotten.

On second thoughts, this probably makes more sense.

Adding a patch from the parent issue (#323110-53: Remove the open_basedir requirement check ) here so we can run tests properly.

I have tested it manually and simpletest still works when this check is removed and open_basedir is enabled.

And the patch :)

Just to spell out the tl;dr of the rather long and meandering parent issue:

PHP's open_basedir used to cause problems with simpletest's usage of curl, so a requirements check was added pointing out the conflict with the testing framework.

In #1671200: Simpletest broken: CURLOPT_COOKIEJAR cannot be NULL (and maybe others) the conflicts with open_basedir were resolved.

So the requirements check is no longer required, and in fact could be harmful as it recommends disabling a security feature unnecessarily.

The patch looks good, thanks!

Thanks everyone!

Added credits from the parent issue.