prepare-ChangeLog: Look for a text editor in a few more variables.
authorkubo@profusion.mobi <kubo@profusion.mobi@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Jan 2012 19:05:57 +0000 (19:05 +0000)
committerkubo@profusion.mobi <kubo@profusion.mobi@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Jan 2012 19:05:57 +0000 (19:05 +0000)
https://bugs.webkit.org/show_bug.cgi?id=76289

Reviewed by Ryosuke Niwa.

A text editor was looked for only in the CHANGE_LOG_EDITOR and
CHANGE_LOG_EDIT_APPLICATION environment variables before falling back
to the OS X-specific open(1) command.

As there does not seem to be a common way of looking for a text editor
among the scripts in Tools/Scripts right now, just look at $VISUAL and
$EDITOR (set much more often than the other variables in Unix systems)
before falling back to open(1).

* Scripts/prepare-ChangeLog:
(openChangeLogs):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@104955 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/prepare-ChangeLog

index a0cf9b7d7a0d68ba5a1df9f0e17a2e0a72d4c6c7..1ef73e3c45fe936063fd925924d047f1bcca3924 100644 (file)
@@ -1,3 +1,22 @@
+2012-01-13  Raphael Kubo da Costa  <kubo@profusion.mobi>
+
+        prepare-ChangeLog: Look for a text editor in a few more variables.
+        https://bugs.webkit.org/show_bug.cgi?id=76289
+
+        Reviewed by Ryosuke Niwa.
+
+        A text editor was looked for only in the CHANGE_LOG_EDITOR and
+        CHANGE_LOG_EDIT_APPLICATION environment variables before falling back
+        to the OS X-specific open(1) command.
+
+        As there does not seem to be a common way of looking for a text editor
+        among the scripts in Tools/Scripts right now, just look at $VISUAL and
+        $EDITOR (set much more often than the other variables in Unix systems)
+        before falling back to open(1).
+
+        * Scripts/prepare-ChangeLog:
+        (openChangeLogs):
+
 2012-01-13  Gabor Rapcsanyi  <rgabor@webkit.org>
 
         [GTK] gtkdoc.py ignoring LDFLAGS and CFLAGS environment variables
index 3c09655b8954d816955e3f6dac85ede3c6c0c50e..243551559d7fb6d24516de975a3599caf39b3d14 100755 (executable)
@@ -532,7 +532,7 @@ sub openChangeLogs($)
     my ($changeLogs) = @_;
 
     print STDERR "  Opening the edited ChangeLog files.\n";
-    my $editor = $ENV{CHANGE_LOG_EDITOR};
+    my $editor = $ENV{CHANGE_LOG_EDITOR} || $ENV{VISUAL} || $ENV{EDITOR};
     if ($editor) {
         system ((split ' ', $editor), @$changeLogs);
     } else {