Core DB layer API gives developers a function which is db_query_temporary(), this function explicitely creates a temporary table. When you use MySQL, if the "CREATE TEMPORARY TABLES" is not set for the database user, this function will fail.

IRC people told me that Drupal core does not uses it, that is not a good point. The fact is Drupal is not only a CMS, but also a full and comprehensive framework for modules and sites building, and as such, it must, either make this works out the box, either tell users in the installation documentation that might be a problem.

Some modules will probably use it, if an end user download one of them, and aptempt to install it using the current INSTALL.mysql.txt file, it will fail, and he will never now why (end users, even some developers won't even understand the "Acces denied for user X" in watchdog message).

Attached a simple patch for the txt file. Feel free to use it. This is a real bug, please fix this. If I remember well, Drupal 5 documentation told to set this grant (I might be wrong on this statement).

Files: 
CommentFileSizeAuthor
#6 INSTALL.mysql_.txt-d6.x-temp_table.patch607 bytespounard
FAILED: [[SimpleTest]]: [MySQL] 190 pass(es), 0 fail(s), and 46 exception(es).
[ View ]
INSTALL.mysql_.txt.patch519 bytespounard
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch INSTALL.mysql_.txt.patch.
[ View ]

Comments

Status:Active» Reviewed & tested by the community

Looks like a nice documentation improvement to me.

This patch should apply cleanly, but please see http://drupal.org/patch/create for creating proper patches, next time.

Status:Reviewed & tested by the community» Needs work

The last submitted patch, INSTALL.mysql_.txt.patch, failed testing.

How that comes this patch fails? WTF?!
However, I will next time.

EDIT: Thanks again for your rapid answer, that's nice to see you're active! :)

Status:Needs work» Needs review
StatusFileSize
new607 bytes
FAILED: [[SimpleTest]]: [MySQL] 190 pass(es), 0 fail(s), and 46 exception(es).
[ View ]

Status:Needs review» Needs work

The last submitted patch, INSTALL.mysql_.txt-d6.x-temp_table.patch, failed testing.

What? Test failure messages are just not related to this patch.

Version:6.17» 6.x-dev
Status:Needs work» Needs review

Let's go for 6.x.

Status:Needs review» Reviewed & tested by the community

Still apply just fine.

Status:Reviewed & tested by the community» Fixed

Committed this, thanks!

Status:Fixed» Closed (fixed)

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