Skip to content

Improve README, link unbundle RFC #5

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.

Sign up for GitHub

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

Merged
merged 3 commits into from
Jun 1, 2025
Merged

Improve README, link unbundle RFC #5

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
b952639
[doc] Update README about PHP 8.4 and why this repository exists
gido Jul 8, 2024
b88bb3c
Update README.md
mvorisek Jun 1, 2025
ff7df58
Update README.md
mvorisek Jun 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@

Use the PDO_OCI extension to access Oracle Database.

This repository is for PHP 8.3+ as it [was decided to unbundle](https://wiki.php.net/rfc/unbundle_imap_pspell_oci8) PDO OCI / OCI8 extension from the PHP source code.

Documentation is at https://www.php.net/pdo_oci

Use `pecl install pdo_oci` to install for PHP 8.2.
Copy link
Member

@cmb69 cmb69 Aug 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This appears to be completely outdated, since there have been no PECL releases for almost 20 years. Are there any plans regarding a new release? At least the RFC about unbundling pdo_oci states:

The proposal is to unbundle them: remove them from the PHP source distribution, move them to PECL, and create an initial release for them.

/cc @derickr

Copy link
Collaborator

@cjbj cjbj Aug 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll get to it when I next have time; I will merge the test PR as is, and do some testing, and do a release.

Lack of PECL releases was not specific to PDO_OCI. E.g https://pecl.php.net/package/pdo, https://pecl.php.net/package/pdo_mysql both haven't had releases. Back in the day it was decided (or became true) that there was a tight coupling between the vendor layer and the generic layer, and no releases were made on PECL.

Copy link
Member

@cmb69 cmb69 Aug 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, PDO and pdo_mysql are still bundled with php-src, so there is no need to have new releases. But since pdo_oci has been unbundled, it would make sense to re-activate the PECL page. At least for Windows users that would be helpful, to be able to download pre-built DLLs.

I'll try to come up with a PR regarding building (and testing) on CI (for Windows).

Expand Down