pdo_cubrid/reference.xml: fix typo#3360
Conversation
| When you establish the connection to CUBRID, you should give username and | ||
| password except DSN. |
There was a problem hiding this comment.
I'm actually not sure what this note is attempting to say
There was a problem hiding this comment.
Frankly, I wouldn't worry about the wording, since that extension appears to be totally unmaintained (or whatever is going on there). https://pecl.php.net/package/CUBRID has it's latest release from 2017, but "browse source" refers to the old PHP SVN server, where the latest commit was "Tagging the 9.1.0.0001 release". I have no idea where the source code is developed now; the linked homepage http://www.cubrid.org/php_api_for_cubrid is 404.
There was a problem hiding this comment.
Is the "normal" CUBRID extension still maintained? If not we probably should remove it after the next archiving batch.
There was a problem hiding this comment.
Should we drop the docs then?
There was a problem hiding this comment.
I found binaries and sources on https://www.cubrid.org/downloads (click PHP and/or PDO tab). These appear to be more recent than the PECL releases (although apparently they don't offer PHP 8 versions, yet). So I conclude that the PECL package has been abandoned (for whatever reason). As such, it may make sense to clarify that on PECL, and to also remove the documentation. But that should likely be discussed on the respective mailing list.
There was a problem hiding this comment.
While we can have that discussion separately, this is very clearly a typo that we can fix
|
@mmalferov Could you rebase this? Once done this can be merged. As Daniel mentioned. The discussion to drop this extension completely can be done separately and should not block these type of PRs imho. |
6 participants
No description provided.