Opened 18 months ago
Closed 17 months ago
#58217 closed defect (bug) (fixed)
The URL defined "Text_Diff_Engine_native" is not working as expected.
Reported by: | utsav72640 | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 6.3 | Priority: | normal |
Severity: | normal | Version: | 6.2 |
Component: | External Libraries | Keywords: | has-patch changes-requested |
Focuses: | docs | Cc: |
Description
The URL defined "Text_Diff_Engine_native" is not working as expected.
Path : wp-includes/Text/Diff/Engine/native.php
I have attached a list of URLs that returned a 404 error.
http://www.perl.com/CPAN/authors/id/N/NE/NEDKONZ/Algorithm-Diff-1.06.zip
http://opensource.org/licenses/lgpl-license.php
Attachments (6)
Change History (11)
#1
@
17 months ago
- Focuses docs added
@utsav72640 Thank you for the report
Due to the publication of a new licence version for the LGPL, I inspected those URLs and discovered that they all redirect to 404. I have added an attachment patch and updated it with a new URL.
Thanks
#2
@
17 months ago
- Keywords has-patch changes-requested added; dev-feedback removed
- Milestone changed from Awaiting Review to 6.3
Hi there, thanks for the patch!
- Per the Text_Diff page on GitHub, it has been superseded by the Horde_Text_Diff package, which uses version 2.1 of the LGPL license, so https://opensource.org/license/lgpl-2-1/ would likely be the correct URL here as well, instead of the 3.0 version.
- The
Algorithm::Diff
URL should still link to version 1.06 for accuracy, so either of these URLs would be correct:
Note: See
TracTickets for help on using
tickets.
wp-includes/Text/Diff/Engine/native.php