Since core supports InnoDB now, there's no reason to force MyISAM tables in the .install file. If a site wants to use InnoDB, we should let them. ;)

Note: to avoid patch conflicts, this one should be applied after my latest patch in http://drupal.org/node/197561 -- http://drupal.org/files/issues/simple_access_197561.patch.txt

CommentFileSizeAuthor
simple_access_no-MyISAM.patch.txt1.07 KBdww

Comments

gordon’s picture

gordon commented
Status: Needs review » Fixed

Fixed in 1.x-dev, New version coming soon.

dww’s picture

dww
we/he/they
commented
Status: Fixed » Patch (to be ported)

Again, you only committed this to HEAD, not the DRUPAL-5 stable branch. Thanks.

gordon’s picture

gordon commented
Status: Patch (to be ported) » Fixed
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

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

  • Commit 3008fa7 on 5.x-2.x, 6.x-2.x, master, 8.x-3.x by gordon: 
    * #203179 - Don't force MyISAM in MySQL