Re: pr-msvc-support merge

2010-06-22 Thread Peter Rosin
Hi Ralf, Den 2010-06-22 21:24 skrev Ralf Wildenhues: * Peter Rosin wrote on Tue, Jun 22, 2010 at 01:53:49PM CEST: (also folding in the "cl\ * | */cl\ *" cases suggested by Ralf in another threadlet) Since the following is way over the top IMHO: [cC][lL] | *[/\\][cC][lL] | [cC][lL]\ * | *[/

Re: [PATCH 0/6] Lex, Yacc and explicit declarations of dependencies.

2010-06-22 Thread Stefano Lattarini
At Thursday 20 May 2010, Stefano Lattarini wrote: > This patch series stems from a discussion on bug-automake: > [] > > In short, some make implementations tend to use their built-in ".y > => .o" and ".l => .o" rules instead of res

Re: [PATCH 0/2] Bugfix w.r.t. $(AM_LFLAGS)/$(LFLAGS) and $(AM_YFLAGS)/$(YFLAGS) precedence.

2010-06-22 Thread Ralf Wildenhues
* Stefano Lattarini wrote on Tue, Jun 22, 2010 at 01:15:04PM CEST: > At Monday 21 June 2010, Ralf Wildenhues wrote: > > I'm falling more and more behind on patches, it seems. > In this last period, it seems quite the opposite to me :-) > Thanks for your work! You've done most of the actual work, g

Re: pr-msvc-support merge

2010-06-22 Thread Ralf Wildenhues
* Peter Rosin wrote on Tue, Jun 22, 2010 at 01:53:49PM CEST: > Den 2010-06-22 05:38 skrev Charles Wilson: > >What if $CC is specified as 'cl.exe'? IIUC, this means that the compile > >script will be invoked as "compile cl.exe " but then this case: > > > >+ cl | *[/\\]cl) > >+func_cl_wrappe

Re: pr-msvc-support merge

2010-06-22 Thread Peter Rosin
Den 2010-06-22 14:04 skrev Charles Wilson: On 6/22/2010 7:53 AM, Peter Rosin wrote: I added compile5.test. I don't think $build is part of the equation for the purpose of this test, and the only supported $host at this point is mingw. Is it what you had in mind? Well, if $build is cygwin, then

Re: [PATCH] New requirement "cc" for tests using a C compiler

2010-06-22 Thread Stefano Lattarini
> Next step will be to run the whole automake testsuite with `CC=no', > to see which tests need a C compiler without explicitly requiring > it. Which obviously doesn't help, because a configure script containing the expansion of AC_PROG_CC exits with status `77' if it can't find a working C com

[PATCH] Minor improvements and fixes in tests `depcomp*.test'.

2010-06-22 Thread Stefano Lattarini
Yet another testsuite-tweaking patch. Regards, Stefano -*-*-*- * tests/depcomp.test: Do not create useless dummy source files. Add a trailing `:' command. * tests/depcomp2.test: Use `unset' on the CFLAG variable to ensure it's not in in the environment, rather than exporting it with an empty

[PATCH] New requirement "cc" for tests using a C compiler (was: Fix code for requiring gfortran and g77 in tests)

2010-06-22 Thread Stefano Lattarini
Ralf wildenhues wrote: > > As a first approximation, this could then check either > > for presence of gcc/g++/gfortran/g77 but not set the $CC/... > > variable (this is only to ensure that some compiler for the > > language is available), or could assume that if the $CC/... > > variable has al

Re: pr-msvc-support merge

2010-06-22 Thread Peter Rosin
Hi Chuck, Den 2010-06-22 14:04 skrev Charles Wilson: On 6/22/2010 7:53 AM, Peter Rosin wrote: I added compile5.test. I don't think $build is part of the equation for the purpose of this test, and the only supported $host at this point is mingw. Is it what you had in mind? Well, if $build is c

Re: pr-msvc-support merge

2010-06-22 Thread Charles Wilson
On 6/22/2010 7:53 AM, Peter Rosin wrote: > I added compile5.test. I don't think $build is part of the equation > for the purpose of this test, and the only supported $host at this > point is mingw. Is it what you had in mind? Well, if $build is cygwin, then it'll work, because cygwin always has cy

Re: pr-msvc-support merge

2010-06-22 Thread Peter Rosin
Den 2010-06-22 05:38 skrev Charles Wilson: What if $CC is specified as 'cl.exe'? IIUC, this means that the compile script will be invoked as "compile cl.exe " but then this case: + cl | *[/\\]cl) +func_cl_wrapper "$@" # Doesn't return... +;; won't fire. (also folding in the

Re: [PATCH] Fix code for requiring gfortran and g77 in tests.

2010-06-22 Thread Stefano Lattarini
At Monday 21 June 2010, Ralf Wildenhues wrote: > * Stefano Lattarini wrote on Sun, Jun 20, 2010 at 11:07:01PM CEST: > > At Sunday 20 June 2010, Ralf Wildenhues wrote: > > > * Stefano Lattarini wrote on Sun, Jun 13, 2010 at 10:03:33PM CEST: > > > > Another improvements to 'tests/defs', factored out

Re: [PATCH 0/2] Bugfix w.r.t. $(AM_LFLAGS)/$(LFLAGS) and $(AM_YFLAGS)/$(YFLAGS) precedence.

2010-06-22 Thread Stefano Lattarini
At Monday 21 June 2010, Ralf Wildenhues wrote: > Hi Stefano, > I'm falling more and more behind on patches, it seems. In this last period, it seems quite the opposite to me :-) Thanks for your work! I've just a minor nit about the follow-up patch... > Ensure generated C files contain a declarat

Re: [PATCH] New test: distributed broken symlinks cause failure.

2010-06-22 Thread Stefano Lattarini
At Monday 21 June 2010, Ralf Wildenhues wrote: > Hi Stefano, > > > Do we still care about these systems? > > That is the question, right. There have been bug reports against > Autoconf about a year ago, so someone is still interested in this. > Not many people, I presume. > > Oh well. Let's le

Re: [PATCH 3/3] Regenerate autoools files

2010-06-22 Thread Stefano Lattarini
Updated, for consistency. Regards, Stefano From 8915e3d9afd43ac45e20b4fbc54d50e116f6d207 Mon Sep 17 00:00:00 2001 From: Stefano Lattarini Date: Sat, 19 Jun 2010 12:03:57 +0200 Subject: [PATCH 3/3] Regenerate autoools files. --- Makefile.in | 15 ++- aclocal.m4

Re: [PATCH 2/3] Fix cscope w.r.t. VPATH rewrites.

2010-06-22 Thread Stefano Lattarini
Updated, for consistency. Regards, Stefano From 7292a5e31cca3be31d8f44be87949e54da7af9e1 Mon Sep 17 00:00:00 2001 From: Stefano Lattarini Date: Sat, 19 Jun 2010 11:57:19 +0200 Subject: [PATCH 2/3] Fix cscope w.r.t. VPATH rewrites. * lib/am/tags.am (cscopelist): Take into account the possibili

Re: [PATCH 1/3] Refactor and extend tests on cscope functionality.

2010-06-22 Thread Stefano Lattarini
At Monday 21 June 2010, Ralf Wildenhues wrote: > > > > --- a/tests/cscope3.test > > +++ b/tests/cscope3.test > > [CUT] > > ./configure > > $MAKE cscope > > +# FIXME (in automake.in): We'd like cscope.files not to be > > +#created, as there are no source files in `.' nor in `sub'. > > +test ! -s c