-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Failure to resolve PDO as required extension in tests #12878
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Is it not the phpt that comes with php-src, but the phpt that you created yourself? If so, you can implement skip by writing:
|
@SakiTakamachi that's a viable option indeed, thanks for the suggestion! Do you perhaps know what might be causing this difference between 8.2 and 8.3? |
@ltrk2 |
I see.. the exact version I'm using is 8.2.4. Perhaps the change occurred some time after that. Both versions (8.3.0 & 8.2.4) are built using the same process from source on my end. |
Excuse me, I seem to have made a mistake in some operation. Reproduced in 8.2. I identified the commit that made this change. Previously, even if we passed "PDO", it would be converted to lower case and then compared, but it seems that the process of converting to lower case has disappeared. |
@SakiTakamachi Indeed.. thank you for finding this! It all makes sense now. I greatly appreciate your help in figuring this out. I also found the answer to another strange occurrence: using PHP 8.0 and below the All in all, my options are either using |
Description
The following code:
When executed using the test runner as
Resulted in this output:
But I expected this output instead:
It works fine in PHP 8.2. I'd gladly just use
pdo
, but we have a set of tests that run on versions 7.0 through 8.3 and the lower versions only supportPDO
.What are your thoughts regarding this?
PHP Version
PHP 8.3.0
Operating System
Ubuntu 22.04
The text was updated successfully, but these errors were encountered: