|  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Request #75206 porting to php7
Submitted: 2017-09-14 05:38 UTC Modified: 2018-02-12 06:25 UTC
Avg. Score:5.0 ± 0.0
Reproduced:1 of 1 (100.0%)
Same Version:1 (100.0%)
Same OS:0 (0.0%)
From: at dot mitani at gmail dot com Assigned:
Status: Feedback Package: svn (PECL)
PHP Version: 7.0.23 OS: Linux
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
Block user comment
Status: Assign to:
Bug Type:
From: at dot mitani at gmail dot com
New email:
PHP Version: OS:


 [2017-09-14 05:38 UTC] at dot mitani at gmail dot com
From manual page:
I'd like to use svn library with php7. However even the latest version of svn package is not working with php7.
Therefore, I made a patch for it. It seems work in my environment as follows.

If you have interested in it, please check my patch.

Thank you.


php7again.diff (last revision 2017-11-27 16:12 UTC) by cpigott at rapitasystems dot com)
svn.patch (last revision 2017-09-14 05:39 UTC) by at dot mitani at gmail dot com)

Add a Patch

Pull Requests

Add a Pull Request


AllCommentsChangesGit/SVN commitsRelated reports
 [2017-11-07 13:37 UTC] david dot little2 at gmail dot com
Thank you!  This patch worked for me with the following setup:

OS: CentOS 7.4.1708
PHP: 7.1.11
Subversion: 1.7.14
gcc: 4.8.5
 [2017-11-20 03:46 UTC]
-Status: Open +Status: Feedback
 [2017-11-20 03:46 UTC]
This has been committed to SVN

testing would be appreciated before it get's released

 [2017-11-24 16:33 UTC] cpigott at rapitasystems dot com
Out of interest, when will the release come? I've got a thing that's waiting on this, and it'd be great if I didn't have to build it myself or downgrade to PHP5.6
 [2017-11-25 04:04 UTC]
As soon as I get confirmation that the code currently in SVN works

Otherwise it as to wait till I get time to test it.

 [2017-11-27 16:11 UTC] cpigott at rapitasystems dot com
Seems just as well I did. Didn't even compile on my Ubuntu 16.04 system

Patch attached fixes compilation issues, along with a few warnings (const "correctness", mostly) - still a lot of svn deprecated functions being used though.

Most notably, reverts r254248 - - not sure why this change was made, but it broke half the tests and seems counter intuative anyway. Didn't see it mentioned anywhere in the changelog either.

All tests except status (hardcoded path values) pass, but I get that those tests don't nearly cover everything - probably needs a realworld test still :)
 [2018-02-08 21:06 UTC] buccim2 at gene dot com
Any Updates on getting that additional patch merged in? Have to manually apply the patch to trunk before building in my docker env.
 [2018-02-12 06:25 UTC]
Ok - have another go - I've patched it with the last patch..

Hopefully that fixes it.
PHP Copyright © 2001-2018 The PHP Group
All rights reserved.
Last updated: Mon Jun 18 17:01:46 2018 UTC