r76302 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r76301‎ | r76302 | r76303 >
Date:14:16, 8 November 2010
Author:demon
Status:ok
Tags:
Comment:
(bug 13353) Diff3 version checks are too strict. Just check for the phrase "GNU diffutils" which is what we really want.
Modified paths:
  • /trunk/phase3/RELEASE-NOTES (modified) (history)
  • /trunk/phase3/includes/installer/Installer.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/installer/Installer.php
@@ -623,7 +623,7 @@
624624 */
625625 public function envCheckDiff3() {
626626 $names = array( "gdiff3", "diff3", "diff3.exe" );
627 - $versionInfo = array( '$1 --version 2>&1', 'diff3 (GNU diffutils)' );
 627+ $versionInfo = array( '$1 --version 2>&1', 'GNU diffutils' );
628628
629629 $diff3 = $this->locateExecutableInDefaultPaths( $names, $versionInfo );
630630
@@ -916,7 +916,7 @@
917917 $command = "\"$command\"";
918918 }
919919 $file = str_replace( '$1', $command, $versionInfo[0] );
920 - if ( strstr( wfShellExec( $file ), $versionInfo[1]) !== false ) {
 920+ if ( strstr( wfShellExec( $file ), $versionInfo[1] ) !== false ) {
921921 return $command;
922922 }
923923 }
Index: trunk/phase3/RELEASE-NOTES
@@ -407,6 +407,7 @@
408408 parameters
409409 * (bug 25175) HTML file cache now honor $wgCacheDirectory if
410410 $wgFileCacheDirectory is not set
 411+* (bug 13353) Diff3 version checks were too strict, did not detect working diff3
411412
412413 === API changes in 1.17 ===
413414 * (bug 22738) Allow filtering by action type on query=logevent.

Status & tagging log