Re: MSVC compiler support [patch 6]: Default program definition

[John W. Eaton]

On 18-Oct-2006, Michael Goffioul wrote:

| John W. Eaton a écrit :
| > On 17-Oct-2006, Michael Goffioul wrote:
| >
| > | Defines default programs (gnuplot and pager) correctly.
| > | 
| > | Index: aclocal.m4
| > | ===================================================================
| > | RCS file: /cvs/octave/aclocal.m4,v
| > | retrieving revision 1.97
| > | diff -p -c -r1.97 aclocal.m4
| > | *** aclocal.m4    17 Aug 2006 20:29:51 -0000      1.97
| > | --- aclocal.m4    17 Oct 2006 11:07:41 -0000
| > | *************** else
| > | *** 464,469 ****
| > | --- 464,472 ----
| > |       *-*-cygwin*)
| > |         octave_possible_pagers="$octave_possible_pagers more.com"
| > |       ;;
| > | +     *-*-msdos)
| > | +       octave_possible_pagers=
| > | +     ;;
| > |     esac
| > |   
| > |     AC_CHECK_PROGS(DEFAULT_PAGER, $octave_possible_pagers, [])
| >
| > I applied the second part of this patch, but not the first.
| >
| > Wouldn't more.com at least be available?
| >   
| 
| Yes it is, but when I first tried, I still had the problem that setvbuf 
| crashed on pipes, such
| that it didn't work; hence I just disabled it. The correct patch would 
| be to define
| octave_possible_pagers to "more.com" instead of leaving it empty.

I made the following change.  Is there any reason to not do this?  If
someone has less installed, I would assume it would be better to
to use that instead of more.com.

jwe


Index: aclocal.m4
===================================================================
RCS file: /cvs/octave/aclocal.m4,v
retrieving revision 1.99
diff -u -u -r1.99 aclocal.m4
--- aclocal.m4  18 Oct 2006 20:57:04 -0000      1.99
+++ aclocal.m4  26 Oct 2006 21:31:14 -0000
@@ -461,7 +461,7 @@
 else
   octave_possible_pagers="less more page pg"
   case "$canonical_host_type" in
-    *-*-cygwin*)
+    *-*-cygwin* | *-*-mingw32* | *-*-msdosmsvc)
       octave_possible_pagers="$octave_possible_pagers more.com"
     ;;
   esac