-
Notifications
You must be signed in to change notification settings - Fork 11
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
Docs outdated #11
Comments
I'm not sure what I want to do here either. The whole politics of PHP internals has kind of made a mess of things. |
I don't really have time to maintain the 1.x branch, the 2.x branch, and the php-src versions either. Indeed, the breaking changes required by PHP internals resulted in quite of a mess. We can write wrappers between the original and the PHP-internals API (actually someone already did that a while back), but it will be yet another thing to maintain. As much as I don't like the new API, it's too late, I think we don't have any options, but adopt it. |
Will there be any |
The documentation has been updated. I'm going to go through it today and make sure it's adequately clean/readable before I tag/sign a release. |
If you run
pecl install libsodium
(as currently recommended in the install guide in the docs) then you'll get libsodium2.x
which is BC breaking with1.x
. Extension is also now calledsodium.so
[1] and all the functions are in the global namespace[2] (so e.g. the version verification script in the docs does not work, as it uses\Sodium\...
instead of the global namespace).Not sure if you want to rewrite the docs for
2.x
, or include a version constraint in thepecl
install (1.x
is stated not to receive any more updates[3], so former likely preferable).References:
[1], [2], [3]: https://github.com/jedisct1/libsodium-php/blob/4e668284deaff4243c7ef36025525f44bbeb8078/README.md
The text was updated successfully, but these errors were encountered: