- Issue created by @Gábor Hojtsy
- Merge request !71Introduce Drupal 10 to 11 environment check and remove Drupal 8 to 9 checks including utility class → (Merged) created by Gábor Hojtsy
- Status changed to Needs review
10 months ago 6:42pm 21 February 2024 - 🇭🇺Hungary Gábor Hojtsy Hungary
MySQL/MariaDB are not properly checked yet, but otherwise looks good on drupalpod/gitpod.
- 🇭🇺Hungary Gábor Hojtsy Hungary
Hm, Drupal 11.x has this code to match mariadb version numbering, which we already rely on:
protected function getMariaDbVersionMatch(): ?string { // MariaDB may prefix its version string with '5.5.5-', which should be // ignored. // @see https://github.com/MariaDB/server/blob/f6633bf058802ad7da8196d01fd19d75c53f7274/include/mysql_com.h#L42. $regex = '/^(?:5\.5\.5-)?(\d+\.\d+\.\d+.*-mariadb.*)/i'; preg_match($regex, $this->getServerVersion(), $matches); return (empty($matches[1])) ? NULL : $matches[1]; }
So its a
number (dot) number (dot) number, followed by anything (dash) "mariadb" followed by anything
.It would be 10.4.32 here if it would not be
(plus) "maria"
(without the dash and "db"). Wondering how widespread this is. - 🇭🇺Hungary Gábor Hojtsy Hungary
But the site status report identifies it correctly so its still on our code to fix.
- Status changed to RTBC
10 months ago 7:41pm 21 February 2024 - 🇭🇺Hungary Gábor Hojtsy Hungary
All good now. The MR also adds Drush 11 checking to Drupal 9 to 10 upgrades, which is useful IMHO. That was missing before. I also tested that today on a throwaway drupalpod Drupal 9 site with the MR.
-
Gábor Hojtsy →
committed 62b19439 on 4.x
Issue #3423014 by Gábor Hojtsy: Drupal 11 system requirements are now...
-
Gábor Hojtsy →
committed 62b19439 on 4.x
- Status changed to Fixed
10 months ago 7:45pm 21 February 2024 Automatically closed - issue fixed for 2 weeks with no activity.