Michael Osipov
Michael Osipov
Made progress: There is no usable Horde_Text_Diff on packagist. I had to fork it because it not PHP 8 compatible. [@bytestream](https://github.com/bytestream) has forked a lot of the Horde modules, but...
I have intentionally switched to the `Native` diff impl in Text_Diff because the way `Xdiff` is used is that the context is always applied to the entire file and the...
One note on PEAR: I wasn't able to make it run with PHP 8.2, it just crashes and Horde_Text_Diff isn't available in a newer version anyway, I am considering to...
@bytestream I did indeed check Util yesterday already and happily used your approach. Kudos for that. All tests pass with PHP 8.2-beta2. From your point of view it does not...
@ralflang Looking into your fork...
@ralflang Here is the diff: https://github.com/maintaina-com/Text_Diff/compare/FRAMEWORK_6_0...websvnphp:Text_Diff:php-8.x I would start to create PRs against your fork, if you agree.
Tests also look good for me on my fork: ``` # ./vendor/bin/phpunit PHPUnit 9.5.24 #StandWithUkraine ...I....I...R... 16 / 16 (100%) Time: 00:00.019, Memory: 6.00 MB There was 1 risky test:...
> > But on your fork it does not even run: > > Give me a day or two to care about the basics. Your PR's are much appreciated. >...
@ralflang I have now created all of the PRs to align the code. As soon as those are approved, I will port from our fork to your fork by using...
@ralflang Thank you for helping out so quick. I have a tentative version working with `FRAMEWORK_6_0` branch. Will give you an update next week. Switched from composer to PEAR. One...