On Thu, Nov 23, 2017 at 04:28:44PM +0000, Stuart Henderson wrote: > On 2017/11/23 17:15, Landry Breuil wrote: > > On Wed, Nov 22, 2017 at 10:55:14AM +0000, Stuart Henderson wrote: > > > On 2017/11/21 10:22, Rafael Sadowski wrote: > > > > PHP message: PHP Fatal error: Uncaught exception 'Exception' with > > > > message 'There is no suitable CSPRNG installed on your system' in > > > > /dokuwiki/vendor/paragonie/random_compat/lib/random.php:203 > > > > > > This came up before. The deps should be changed to install pecl-libsodium > > > or php-mcrypt. > > > > Funnily, i faced the exact same issue today when migrating my work > > dokuwiki install from 5.7 to 6.2, the previous testing instance was > > working fine on 6.1, and started failing with 6.2 (ie you can see the > > index, but it blows the same as soon as you try to login). > > Installing/enabling pecl-libsodium solved the issue, and since mcrypt is > > dead upstream, is deprecated in php 7.1 and seems replaced by libsodium > > in php 7.2 (ref: > > https://serverpilot.io/community/articles/how-to-install-the-php-mcrypt-extension.html), > > how about the following: > > > > Index: Makefile > > =================================================================== > > RCS file: /cvs/ports/www/dokuwiki/Makefile,v > > retrieving revision 1.23 > > diff -u -r1.23 Makefile > > --- Makefile 10 May 2017 12:32:48 -0000 1.23 > > +++ Makefile 23 Nov 2017 16:10:30 -0000 > > @@ -27,7 +27,8 @@ > > TINSTDIR = ${TRUEPREFIX}/dokuwiki > > SUBST_VARS = INSTDIR TINSTDIR > > > > -RUN_DEPENDS = lang/php/${MODPHP_VERSION},-gd > > +RUN_DEPENDS = lang/php/${MODPHP_VERSION},-gd \ > > + security/pecl-libsodium > > > > do-install: > > cp -Rp ${WRKDIST} ${INSTDIR} > > > > Worth backporting to 6.2 ? > > > > Needs REVISION bump, then it's ok with me. I doubt it's really > worth the backport though..
Digging a bit more, on the long term (for 6.3?) we'll have to switch to php7 by default, as only libsodium-php-1.x seems to support php 5.6, libsodium-php-2.x only targets 7.x, and the 1.x branch wont receive updates anymore. Landry
