search

geerlingguy / jeffgeerling-com

Drupal Codebase for JeffGeerling.com
https://www.jeffgeerling.com
GNU General Public License v2.0
41 stars 2 forks source link

Switch default container images to ARM64 #134

Closed geerlingguy closed 3 years ago

geerlingguy commented 3 years ago

Since I'm now on an M1 Mac, I don't need the AMD64 images, because that just slows me down.

geerlingguy commented 3 years ago

Hrm... also trying to update to PHP 8 for simplicity's sake (https://github.com/geerlingguy/drupal-container/issues/32), but running into this with Drush install:

 14:37:45 ~/Development/jeffgeerling-com 
$ docker-compose exec drupal bash -c 'drush site:install minimal --db-url="mysql://drupal:$DRUPAL_DATABASE_PASSWORD@$DRUPAL_DATABASE_HOST/drupal" --site-name="Jeff Geerling" --existing-config -y'

 // You are about to DROP all tables in your 'drupal' database. Do you want to continue?: yes.                          

 [notice] Starting Drupal installation. This takes a while.
PHP Fatal error:  Declaration of Drupal\Core\Database\Statement::fetchAll($mode = null, $column_index = null, $constructor_arguments = null) must be compatible with PDOStatement::fetchAll(int $mode = PDO::FETCH_BOTH, mixed ...$args) in /var/www/html/web/core/lib/Drupal/Core/Database/Statement.php on line 169
 [warning] Drush command terminated abnormally.
geerlingguy commented 3 years ago

Huh, looks like Drupal is not compatible with PHP 8 yet. Hopefully that changes soon, I'll stick with 7.4 for now.