Skip to content
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

PDO_OCI for PHP8 #496

Closed
1 task done
samdark opened this issue Sep 18, 2021 · 6 comments
Closed
1 task done

PDO_OCI for PHP8 #496

samdark opened this issue Sep 18, 2021 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@samdark
Copy link

samdark commented Sep 18, 2021

Describe the feature

Could not install pdo_oci on PHP 8.0.10

Version

  • I have checked releases, and the feature is missing in the latest patch version of v2.
@samdark samdark added the enhancement New feature or request label Sep 18, 2021
@samdark
Copy link
Author

samdark commented Sep 18, 2021

@shivammathur
Copy link
Owner

@samdark
I could not reproduce the issue.
Test workflow: https://github.com/shivammathur/test-setup-php/actions/runs/1249207383/workflow

Extensions pdo_oci and oci8 should install correctly for all PHP versions. Might be a temporary issue.

Can you switch to shivammathur/setup-php@verbose, and if it fails again, provide me the logs.

@terabytesoftw
Copy link

@samdark
I could not reproduce the issue.
Test workflow: https://github.com/shivammathur/test-setup-php/actions/runs/1249207383/workflow

Extensions pdo_oci and oci8 should install correctly for all PHP versions. Might be a temporary issue.

Can you switch to shivammathur/setup-php@verbose, and if it fails again, provide me the logs.

When I put the verbose version it works perfectly, when I put the normal version it doesn't work, here you can see it.

Verbose work:

https://github.com/terabytesoftw/db/runs/3641761681?check_suite_focus=true

No verbose:

https://github.com/terabytesoftw/db/runs/3641793839?check_suite_focus=true

@shivammathur
Copy link
Owner

@samdark @terabytesoftw Recent release broke cache support. Fixing it now.

@shivammathur
Copy link
Owner

shivammathur commented Sep 18, 2021

@samdark @terabytesoftw

Fixed in 48aac87
Please test and let me know.

@samdark
Copy link
Author

samdark commented Sep 19, 2021

Worked. Thank you very much for fixing it so fast 👍

https://github.com/yiisoft/db/runs/3644169663

@samdark samdark closed this as completed Sep 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants