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.I'm trying to use HEAD on the Acquia stack installer and realized that the PDO MySQL stuff doesn't let you specify the socket in the .ini file. It sounds like the only way to do this is via the dsn string. Here's a trivial patch to add this to the mysql driver.
| Comment | File | Size | Author |
|---|---|---|---|
| #16 | missing_brackets.patch | 1.51 KB | Berdir |
| #3 | dbtng_mysql_socket.patch | 1.33 KB | drewish |
| #2 | dbtng_mysql_socket.patch | 1.37 KB | drewish |
| dbtng_mysql_socket.patch | 812 bytes | drewish | |
Comments
Comment #1
drewish CreditAttribution: drewish commentedlets spell it correctly.
Comment #2
drewish CreditAttribution: drewish commentedDamZ pointed out that according to the docs you shouldn't specify host/port and socket at the same time.
Comment #3
drewish CreditAttribution: drewish commentedadding some comments.
Comment #5
drewish CreditAttribution: drewish commentedi want a re-test.
Comment #6
Crell CreditAttribution: Crell commentedThe approach here is sound, bot permitting. But it should probably be documented in the INSTALL.mysql.txt file as well. (It's too much of an edge case to put in settings.php directly, IMO.)
Comment #8
Crell CreditAttribution: Crell commentedSomeone want to get back to this or no? :-)
Comment #9
drewish CreditAttribution: drewish commentedi want to see what the test bot says.
Comment #11
Crell CreditAttribution: Crell commentedComment #12
Crell CreditAttribution: Crell commentedMarked #537880: Installation using socket rather than host as a duplicate.
Comment #13
BarisW CreditAttribution: BarisW commentedI have the same issue. The patch from #2 is working. Without this, I was unable to install Drupal 7 on my Reseller server.
Is there a reason not to include this in V7? I'm not sure how many people have this issue, but I think I'm not the only one. If we decide to postpone this to Drupal8, we should at least set a message with the description of the error.
Comment #14
Crell CreditAttribution: Crell commentedIt's not in D7 because no one was able to drive it home before code freeze. I am willing to make an exception for it if webchick is. The approach seems sound, but it needs to pass the test bot. I also don't quite know how we can test this. :-)
Comment #15
Damien Tournoud CreditAttribution: Damien Tournoud commentedThis is a bug report. Bugs are fixed in the current development version before being backported. This should never has been pushed to D8.
Comment #16
BerdirRe-roll, the ?: needs brackets around the whole statement.
Comment #17
duellj CreditAttribution: duellj commentedPatch works correctly. Was able to connect to alternate socket, and if socket is specified, then host and port are ignored (expected behavior).
Comment #18
BarisW CreditAttribution: BarisW commentedGood work! Thanks ;)
Comment #19
webchickCommitted to HEAD, thanks!
I don't think there's a way to add a test for this, so marking as fixed.