Re: typo in comment in tests/defs.in

Hi William, * William Pursell wrote on Sat, Oct 11, 2008 at 06:58:47AM CEST: --- a/tests/defs.in +++ b/tests/defs.in @@ -152,7 +152,7 @@ do CC=icc export CC # There is no way to ask *only* the compiler's version. - # This tool always want to do something (by

parallel-tests: Ensure backward-compatible semantics. [2/4]

parallel-tests: Ensure backward-compatible semantics. For each test in Automake's test suite that uses TESTS, generate an identical one that uses the `parallel-tests' option, for coverage of backward-compatible functionality. * tests/gen-parallel-tests: New file,

Documentation for the parallel-tests driver. [4/4]

Documentation for the parallel-tests driver. * doc/automake.texi (Tests, Options): Document the `parallel-tests' option, including new features of the test driver. * NEWS: Update. diff --git a/NEWS b/NEWS index ed71e9e..6171e2e 100644 --- a/NEWS +++ b/NEWS @@ -101,6 +101,11

Re: magic variables for included fragments (was: Feature request)

[ moving to automake-patches; this is http://thread.gmane.org/gmane.comp.sysutils.automake.general/9824 ] * Ralf Wildenhues wrote on Sun, Oct 12, 2008 at 10:46:06PM CEST: I'll follow up on automake-patches with a patch to test. Here we go. WDYT? Cheers, Ralf Recursive `include

Re: [PATCH] Nicer 'make dist' output.

Hi Eric, thanks for the feedback. * Eric Blake wrote on Mon, Oct 13, 2008 at 04:18:03PM CEST: I like the shorter output. However, ... new_top_distdir=$$reldir; \ + echo (cd $$subdir $(MAKE) $(AM_MAKEFLAGS) top_distdir=$$new_top_distdir distdir=$$new_distdir \\; \ +

Re: Parallel tests execution [0/4]

Hi Jim, * Jim Meyering wrote on Wed, Oct 15, 2008 at 09:13:54PM CEST: Ralf Wildenhues [EMAIL PROTECTED] wrote: 12) allow for additional output on stdout/stderr? example: test was skipped because of $reason. This would be nice. It'd would be useful also to mark as surprising

Re: Parallel tests execution [0/4]

Hi Akim, thanks for the feedback! * Akim Demaille wrote on Thu, Oct 16, 2008 at 04:05:47PM CEST: RW == Ralf Wildenhues [EMAIL PROTECTED] writes: These four patches implement parallel execution of TESTS in Automake, adapted from the check.mk file Akim Demaille posted earlier

Re: Parallel test execution: new option `parallel-tests': [1/4]

Hi Akim, again, thanks for your and Jim's valuable feedback! * Akim Demaille wrote on Thu, Oct 16, 2008 at 04:58:48PM CEST: The logic to create this kind of message is really complex, and in retrospect, I don't think that the message delivered is much easier to read than something more

Re: Documentation for the parallel-tests driver. [4/4]

Hi Akim, * Akim Demaille wrote on Fri, Oct 17, 2008 at 12:15:42PM CEST: Consequently, there is little point in showing users how to write such a rule. I strongly disagree here. And your position is quite founded, if but a bit hard to swallow at first. ;-) There are several aspects to this

Fix DisjConditions module to be thread-safe for perl = 5.7.2.

creation of a new condition; skip test if perl is too old or ithreads are not available. * lib/Automake/tests/DisjConditions-t.pl: Likewise. * lib/Automake/tests/Makefile.am (TESTS): Add them. Signed-off-by: Ralf Wildenhues [EMAIL PROTECTED] diff --git a/lib/Automake

Re: AM_DEFAULT_SOURCE_EXT

* Akim Demaille wrote on Mon, Oct 20, 2008 at 12:56:23AM CEST: Le 18 oct. 08 à 03:02, Ralf Wildenhues a écrit : * Akim Demaille wrote on Thu, Oct 16, 2008 at 04:11:22PM CEST: I wish I could define AM_DEFAULT_SOURCE_EXT = .cc or whatever, so that I wouldn't have to define all

Let stderr output end up on fd 2 in testsuite.

A trivial cleanup patch, pushed. Cheers, Ralf Let stderr output end up on fd 2 in testsuite. * tests/defs.in (AUTOMAKE_run): Output recorded stderr on file descriptor 2. * tests/getopt.test: Fix erroneous multiple redirection, uncovered by above change. diff --git

Re: Python 3.0 support

Hi Johan, long time ago, you submitted patches for Python 3.0 support. Copyright papers are in place now (they've been for a few weeks but apparently I hadn't been notified, and I was intelligent enough to overlook them in the list last time). So I've applied your changes to the git master

Minor file checking optimization: set_dir_cache_file.

This patch has a bit nonobvious consequences. The dir_has_case_matching_file function in FileUtils.pm maintains a directory entry cache, and when we install an aux file, automake simply forgets the cache for that directory. This optimization simply updates the cache with the knowledge we have:

Parallel automake execution: AUTOMAKE_JOBS. [1/4]

This patch provides the basic functionality. It aims to provide unchanged code paths for the non-threaded case. Notable hack in Channels.pm: before exiting due to an internal error, it is necessary to flush stderr. Otherwise, if a worker thread exits here, its output may not appear. Strictly

Parallel automake --add-missing: serialized file installs. [4/4]

The final piece. A bit ugly because the changes in require_conf_file depend on nonlocal semantics in require_file_internal and in maybe_push_required_file. Oh well. Cheers, Ralf Parallel automake --add-missing: serialized file installs. * automake.in (QUEUE_CONF_FILE,

Fix maintainer-check failure.

Applied to evade the test in toplevel Makefile.am. Cheers, Ralf Fix maintainer-check failure. * tests/parallel-am.test: Rename variable to not match pattern used in maintainer-check. diff --git a/tests/parallel-am.test b/tests/parallel-am.test index de86271..8bd5585

Re: AM_DEFAULT_SOURCE_EXT

Hi Akim, hope I'm not going on your nerves yet .. ;-) * Ralf Wildenhues wrote on Tue, Oct 21, 2008 at 05:00:34AM CEST: * Akim Demaille wrote on Mon, Oct 20, 2008 at 12:56:23AM CEST: Le 18 oct. 08 à 03:02, Ralf Wildenhues a écrit : * Akim Demaille wrote on Thu, Oct 16, 2008 at 04:11:22PM

New maintainer target release-stats.

This patch is purely an aid for keeping the manual up to date. It is a bit system-dependent: you need pstops for it. And the output should be inspected for sanity, as some of the commands are pretty heuristic. Applied master and branch-1-10. Cheers, Ralf New maintainer target

Multiple 'make uninstall' should not fail even for TEXINFOS.

Hello help-texinfo readers, I'm considering putting this into Automake. It would let the uninstall rules generated by automake be idempotent, which seems sensible to me. Any reasons against doing so? Thanks! Ralf Multiple 'make uninstall' should not fail even for TEXINFOS. *

Re: AM_DEFAULT_SOURCE_EXT

* Ralf Wildenhues wrote on Sun, Nov 02, 2008 at 10:58:08PM CET: Choose default source extension: AM_DEFAULT_SOURCE_EXT. * automake.in (handle_source_transform): Accept unconditional literal extension in AM_DEFAULT_SOURCE_EXT as override for the default source extension

Testsuite fixes for ksh.

Pushed a couple of trivial fixes. Cheers, Ralf Testsuite fixes for ksh. * tests/check10.test: Add ':' as last command in subshell, for zero exit status of the subshell. * tests/parallel-am.test: Do not let failing 'unset' of nonexistent variable exit the

Re: gnupload and delete

Hi Karl, * Karl Berry wrote on Wed, Nov 12, 2008 at 08:12:10PM CET: It is not exactly simple to figure out how to delete (archive) files from ftp.gnu.org and alpha.gnu.org with the ftp directives, and anyway constructing the directive files and uploading them by hand is always a pain. This

Re: AM_DEFAULT_SOURCE_EXT

Hi Akim, * Akim Demaille wrote on Mon, Nov 17, 2008 at 09:47:31AM CET: RW == Ralf Wildenhues [EMAIL PROTECTED] writes: hope I'm not going on your nerves yet .. ;-) Well, I'm the one who should plead guilty :( I apologize to be so not responsive. Don't worry. The only thing

Re: [PATCH] more typo/grammar fixes in documentation

Hello William, thanks for the patches. I merged them and applied them, with one change, to master and branch-1-10: * William Pursell wrote on Thu, Nov 20, 2008 at 06:27:33AM CET: @@ -11504,7 +11504,7 @@ announcement that @command{automake} and @command{autoconf} had joined

Re: Question about automatic generation of GPLv3 COPYING file

, INSTALL is installed, and that COPYING is installed as GPLv3 if no COPYING file exists. * tests/license2.test: New test. * tests/Makefile.am: Update. * NEWS, THANKS: Update. Report by Brian Cameron. Signed-off-by: Ralf Wildenhues [EMAIL PROTECTED] diff --git a/NEWS b/NEWS

Re: [PATCH] Add xz compression support.

Hi Jim, * Jim Meyering wrote on Thu, Nov 20, 2008 at 10:09:03AM CET: Just to let you know that the latest changes in the upstream LZMA-utils repository (git://ctrl.tukaani.org/lzma-utils.git) make it so that the primary tool name is now xz rather than lzma. The commit log for

Re: [PATCH] Add xz compression support.

* Jim Meyering wrote on Sat, Nov 22, 2008 at 02:17:55PM CET: Ralf Wildenhues [EMAIL PROTECTED] wrote: ... Anyway, thanks for the patch. Committed to master with only minor modifications. It would be nice if you could test the new test to ensure that it passes with 'xz' installed (I

Re: [PATCH 1/3] quotes in configure.ac

Hi William, thanks for the patches. A couple of comments. * William Pursell wrote on Sat, Nov 22, 2008 at 02:20:56PM CET: --- a/configure.ac +++ b/configure.ac @@ -38,7 +38,7 @@ AM_INIT_AUTOMAKE([1.10a dist-bzip2 filename-length-max=99 color-tests]) # * Prereleases on the trunk are all

Re: [PATCH 1/3] quotes in configure.ac

* William Pursell wrote on Sat, Nov 22, 2008 at 05:19:20PM CET: Ralf Wildenhues wrote: How about this instead: For each @code{AC_REQUIRE_AUX_FILE([EMAIL PROTECTED]@var{file}}])}, @command{automake} will ensure that @[EMAIL PROTECTED] exists in the aux directory, and will complain

Re: maude_ vs LIBRARY_

Hello William, * William Pursell wrote on Sun, Nov 23, 2008 at 10:05:06AM CET: In the documentation, generic items appear as maude_PRIMARY and LIBRARY_PRIMARY. This strikes me as inconsistent. I suggest changing, eg, LIBRARY_LIBTOOLFLAGS to libmaude_LIBTOOLFLAGS. Well, one point of using

Re: maude_ vs LIBRARY_

* William Pursell wrote on Sun, Nov 23, 2008 at 03:04:36PM CET: Ralf Wildenhues wrote: * William Pursell wrote on Sun, Nov 23, 2008 at 10:05:06AM CET: I suggest changing, eg, LIBRARY_LIBTOOLFLAGS to libmaude_LIBTOOLFLAGS. Unless we can come up with a definite improvement in consistency, I

Re: [PATCH] documentation errors

Hello William, * William Pursell wrote on Sun, Nov 23, 2008 at 06:18:47PM CET: here's another set of corrections to the documentation. Thanks. There was a question I had that I notated by adding comments to the .texi file which appears in the patch below re: AM_PROG_CC_C_O. The

Re: [PATCH] documentation: page break

Hi William, * William Pursell wrote on Mon, Nov 24, 2008 at 08:49:22AM CET: You mentioned that I should not edit the ChangeLog--are you generating it automatically? Well, more or less. I have macros to covert from ChangeLog entry to git commit log entry and vice versa. And a script to

Automake and whitespace in pwd (was: [PATCH] gnulib-tool: do not use $(top_srcdir) unquoted; may be tainted)

. Cheers, Ralf 2008-11-26 Ralf Wildenhues [EMAIL PROTECTED] Jim Meyering [EMAIL PROTECTED] Cope with whitespace in $MISSING and $install_sh. * configure.ac (am_AUTOHEADER): New substitution, save the value of $AUTOHEADER before AM_INIT_AUTOMAKE may add $MISSING

Re: Automake and whitespace in pwd

Hi Jim, * Jim Meyering wrote on Thu, Nov 27, 2008 at 10:55:38AM CET: On one hand, I like the idea of accommodating the rare srcdir `pwd` that contains white space. As Ralf alreay noted, it is not that rare with w32 users. On the other hand, since so many Makefile.am files are unprepared

Re: automake manual: distclean

Hello Jan, Andreas, thanks for the bug report. * Andreas Schwab wrote on Wed, Nov 26, 2008 at 07:48:32PM CET: Jan Engelhardt [EMAIL PROTECTED] writes: This is not quite portable -- unless GNU find, which implies a path of . if nothing else is specified, Solaris's explicitly requires a

Fixup release rules.

I'm pushing this patch to fix up the release rules, master and branch-1-10. Cheers, Ralf Fixup release rules. * Makefile.am (cvs-diff): Remove. (git-dist): Do not use clcommit any more. Use new-style tag name. Pass $(AM_MAKEFLAGS) to $(MAKE). (git-diff): Adjust.

Re: magic variables for included fragments

Hi Akim, * Akim Demaille wrote on Wed, Dec 03, 2008 at 02:58:22PM CET: * Ralf Wildenhues wrote on Sun, Oct 12, 2008 at 10:46:06PM CEST: RW == Ralf Wildenhues [EMAIL PROTECTED] writes: Sorry I did not answer before. No problem of course. In the meantime I did come up with some patch

Re: [PATCH 2/3] Replace 'configure' with '@command{configure}' as appropriate.

Hi William, * William Pursell wrote on Mon, Dec 01, 2008 at 10:06:18PM CET: There are several occurences of the phrase 'at configure time' where it may be appropriate to make this change as well, but it is not clear to me that doing so is correct and I'm applying the principal of minimal

Re: [PATCH 3/3] Simple typographical and grammar errors in automake.texi.

Hi William, * William Pursell wrote on Mon, Dec 01, 2008 at 10:06:32PM CET: Fix object/article consistency (eg an flag becomes a flag), correct minor punctuation errors, etc. Thanks! Pushed with a couple of minor changes, as below. @@ -7839,7 +7839,7 @@ suppress the base name step. For

Re: [PATCH 1/3] Replace 'automake' with @command{automake} where appropriate in automake.texi

Hi William, * William Pursell wrote on Mon, Dec 01, 2008 at 10:06:10PM CET: The bare 'automake' occurs often in automake.texi where it should be @command{automake}. There is some inconsistency, however, in that often 'Automake' is used, and I am unsure if it is appropriate to change such

Re: magic variables for included fragments

* Akim Demaille wrote on Thu, Dec 04, 2008 at 09:30:45AM CET: Le 3 déc. 08 à 20:58, Ralf Wildenhues a écrit : A while ago (I'm sure it's been more than a couple of years now), ventured at changing depout.m4 to lift this limitation. I did have something working back then, but it required

Re: magic variables for included fragments

Hi Akim, * Akim Demaille wrote on Wed, Dec 10, 2008 at 10:21:50AM CET: Le 4 déc. 08 à 09:38, Ralf Wildenhues a écrit : I have a silly question, but... is it really known for a fact that some Make out there do not support include if it exists? This question is not relevant to the problem

Fix nonportable sed script in 'missing'.

Yeah, I actually had to stumble over this recently introduced buglet; Tru64 sed interprets everything between the first 't' and newline/end of script as a label, then complains about the label being longer than 6 characters. :-/ (Of course this is all duly documented in the Autoconf manual

Re: Bigger picture of automake variables

* LCID Fire wrote on Sun, Dec 21, 2008 at 07:40:00PM CET: Ralf Wildenhues wrote: So basically variables are not ever transported between Makefiles. Correct. Is this written down somewhere on the docs? Not really. Well, what really happens is that, with AC_SUBST([VARIABLE

Re: automake less verbose (iter 3)

thing about this approach is that in the packages I tested, it increased Makefile.in size by 1% up to 7% only. Feedback welcome. It'll be some more days before I can get back to this. Cheers, Ralf 2008-12-22 Jan Engelhardt jeng...@medozas.de Ralf Wildenhues ralf.wildenh...@gmx.de

Re: automake less verbose (iter 3.1)

Hello, Let's move this to automake-patches, too. * William Pursell wrote on Mon, Dec 15, 2008 at 09:19:38PM CET: Thanks for this Jan, it is really nice functionality. I don't know if this is a portability issue, but I think it would be nice to change $ to $? in this section: Yes, $ is a

[PATCH] Bump copyright years.

Committed to master and branch-1-10. Cheers, Ralf --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +2009-01-20 Ralf Wildenhues ralf.wildenh...@gmx.de + + Bump copyright years. + * aclocal.in (write_aclocal, version): Bump copyright years. + * automake.in (gen_copyright

Re: expand-before-require bug

Hi Eric, * Eric Blake wrote on Tue, Jan 20, 2009 at 10:30:05PM CET: So now the question becomes, is it okay to apply this patch for cosmetic reasons, and with a changed summary line? If you prefer so, then sure, why not (both branches). Thanks for fixing up the various projects. Cheers, Ralf

Re: Make depmode=cpp work with Microsoft Visual C++ on MSYS.

Hi Peter, * Peter Rosin wrote on Tue, Jan 27, 2009 at 02:32:01AM CET: When using MSVC on MSYS, there is nothing that prevents depmode=cpp from being used. It works fine. depmode=msvisualcpp doesn't work as it assumes that cygpath can be used. Thanks for the report and patch. What if the user

Re: Make depmode=cpp work with Microsoft Visual C++ on MSYS.

of it yet. Den 2009-01-28 09:12, skrev Ralf Wildenhues: It would be great to see how the Automake test suite fares with this compiler (see tests/README for how to enable verbose output), esp. the depcomp*.test tests. Sorry, but I get this configure: error: Autoconf 2.61a-341 or better

Re: Make depmode=cpp work with Microsoft Visual C++ on MSYS.

* Peter Rosin wrote on Thu, Jan 29, 2009 at 04:39:16PM CET: Den 2009-01-28 13:41, skrev Ralf Wildenhues: You can install Autoconf below a different --prefix. Yes, but MSYS is MSYS is MSYS is a bitch. I fail to get automake to build even with autoconf-2.63 (in a new prefix, first in $PATH

Re: gnupload self-url?

Hi Karl, * Karl Berry wrote on Thu, Jan 29, 2009 at 11:53:15PM CET: I was thinking it might be useful for gnupload --help to tell users (i.e., maintainers) how to get the latest version of the script. Wdyt? Pushed, both branches. I still need to finish reviewing Sergey's largish changes to

Re: gnupload self-url?

* Karl Berry wrote on Thu, Jan 29, 2009 at 11:53:15PM CET: +You can get the latest version of this script from savannah: +http://git.savannah.gnu.org/cgit/automake.git/plain/lib/gnupload + Report bugs to bug-autom...@gnu.org. BTW, I did enclose the URL in ... angles, consistent with the

Re: gmake check told me to Please report to ...

* Chris Hoogendyk wrote on Fri, Jan 30, 2009 at 01:57:27PM CET: Ralf Wildenhues wrote: * Chris Hoogendyk wrote on Thu, Jan 29, 2009 at 10:14:08PM CET: This was run on a Sun T5220 using gccfss with gmake under Solaris 10. Is there anything else I can do for you? Yes: please post

Re: CFLAGS is for the user in GNU

seems to fit better with the context (the variables which are listed are not all user variables), and it links to the `User Variables' node, and is mostly discusses the quirks. Thanks, Ralf 2009-02-16 Ralf Wildenhues ralf.wildenh...@gmx.de * doc/automake.texi (Program variables): Add cross

Re: CFLAGS is for the user in GNU

* Karl Berry wrote on Tue, Feb 17, 2009 at 02:42:53AM CET: -Automake uses for compilations; for instance, you might need to do your -own compilation in some special cases. +Automake uses for compilations, and in which order (@pxref{Flag +Variables Ordering}); for instance, you

Backport Exit function from master, for easier cherry-picking.

I noticed that I messed up backporting tests from master to branch-1-10 several times already, due to the Exit function. So let's just avoid this hassle altogether. Pushed to branch-1-10. Cheers, Ralf Backport Exit function from master, for easier cherry-picking. * tests/defs.in

Fix recursive html and install-* doc rules for BSD make.

Unlike GNU make, BSD make implementations do not assume that a target marked phony which does not have a corresponding rule, is just a happy thing all by itself. Specifically, the second line of this is necessary in order to successfully run 'pmake html': html: html-am html-am: .PHONY:

Re: Make depmode=cpp work with Microsoft Visual C++ on MSYS.

Hi Peter, * Peter Rosin wrote on Fri, Jan 30, 2009 at 11:20:13AM CET: +2009-01-30 Peter Rosin p...@lysator.liu.se + + Add depmode=msvcmsys for Microsoft Visual C++ on MSYS. + * lib/depcomp [msvisualcpp]: Fork fewer processes. Filter out + libtool in the preprocessor invocation

Re: gnupload improvements

Hello Sergey, again, apologies for the delay. * Sergey Poznyakoff wrote on Tue, Feb 17, 2009 at 09:56:09AM CET: Ralf Wildenhues ralf.wildenh...@gmx.de ha escrit: --symlink a b c d will create two symbolic links: a - b and c - d. This API looks rather unusual. Well, I make

lib/depcomp: Various portability and quoting nits.

Pushing this trivial cleanup to both branches. Cheers, Ralf * lib/depcomp: Various portability and quoting nits. diff --git a/lib/depcomp b/lib/depcomp index 47cf50a..0fb633e 100755 --- a/lib/depcomp +++ b/lib/depcomp @@ -1,7 +1,7 @@ #! /bin/sh # depcomp - compile a program generating

Re: [PATCH 1/3] Replace 'automake' with @command{automake} where appropriate in automake.texi

Hello William, working my way through my patch backlog ... sorry for the long delay. * William Pursell wrote on Thu, Dec 04, 2008 at 10:03:56AM CET: Ralf Wildenhues wrote: * William Pursell wrote on Mon, Dec 01, 2008 at 10:06:10PM CET: @@ -11677,7 +11677,7 @@ A major and long-awaited release

Re: gnupload improvements

Hi Karl, Sergey, thanks for the feedback! * Karl Berry wrote on Wed, Mar 04, 2009 at 11:55:20PM CET: Hmm, then I guess I don't like those options either; but also I don't like backward incompatibilities. Sigh. Since gnupload is only used by GNU maintainers, I do not consider

testsuite: SKIP compile tests if configure found no compiler.

configure nicely exits 77 if the first compiler test failed; so let's use that to skip those tests that otherwise fail due to this. Pushed both branches. Cheers, Ralf testsuite: SKIP compile tests if configure found no compiler. * tests/depend6.test: configure will exit 77 if

Improve `make -n dist' and `make -n distcheck' for GNU make.

I noticed that `make -n dist' touches the build tree, in that it creates directories $(distdir) and below. Also, `make -n distcheck' bails out with some error due to missing directories. (BTW, I'm unsure whether this has been reported before; if yes, speak up and I'll thank ya!) This happens

Re: automake less verbose (iter 3)

Hi Jan, while reviewing the threads about this topic, I found this: * Jan Engelhardt wrote on Wed, Dec 24, 2008 at 11:47:36PM CET: On Monday 2008-12-22 22:28, Ralf Wildenhues wrote: [ from http://thread.gmane.org/gmane.comp.sysutils.automake.general/9995 ] I haven't found another way yet

[PATCH 1/4] New channel `portability-recursive'.

[ moving from automake@ ] * Ralf Wildenhues wrote on Sat, Mar 07, 2009 at 04:16:33PM CET: git clone git://git.savannah.gnu.org/automake.git git branch je-silent origin/je-silent (thanks Jan for the typo fix!) Detailed patch descriptions will follow on automake-patches, This first patch

[PATCH 2/4] Implement `silent' build rules.

...@medozas.de Ralf Wildenhues ralf.wildenh...@gmx.de Implement `silent' build rules. * automake.in (ccer): New field in the language structure. Initialize it for all registered languages. (verbose_var, verbose_flag, verbose_dispatch) (silent_flag

[PATCH 4/4] New automake command line option `--silent-rules'.

For this patch, I'm unsure if we should even add it at all. It adds an automake option to enable silent build rules. Here's why I wrote it in the first place: if you are a distribution and want to change all your packages to build silently, you may not want to change each package's configure.ac

Re: Make compile wrapper eat win32 paths

* Peter Rosin wrote on Mon, Mar 09, 2009 at 04:53:11PM CET: Here's a tiny fix for a typo in the new compile2.test. With this fix I get exitcode 77 between the two subtests. Thanks, applied. Cheers, Ralf commit 3f63a77c0647199d4121c54ea29d28d509539369 Author: Peter Rosin p...@lysator.liu.se

Re: [PATCH 4/4] New automake command line option `--silent-rules'.

* Ralf Corsepius wrote on Mon, Mar 09, 2009 at 03:57:58PM CET: Jan Engelhardt wrote: On Monday 2009-03-09 15:44, Ralf Corsepius wrote: Ralf Wildenhues wrote: For this patch, I'm unsure if we should even add it at all. FWIW: I am opposed to it. I suppose you are opposed to the whole topic

Re: [PATCH 1/4] New channel `portability-recursive'.

* Ralf Wildenhues wrote on Sun, Mar 08, 2009 at 10:18:51AM CET: This warning channel is currently turned on by -Wall, but it should also be turned on by -Wportability, I guess; or get its own switch. As noted in some of the earlier discussions, recursive $(var1$(var2)) variable expansions

Re: automake less verbose branch

* Ralf Wildenhues wrote on Sun, Mar 08, 2009 at 10:07:06AM CET: * Jan Engelhardt wrote on Sat, Mar 07, 2009 at 07:17:03PM CET: I am missing the definition of am__v_GEN in the generated Makefile that is designed for use with manual rules. Like, # -*- Makefile

Re: Make depmode=cpp work with Microsoft Visual C++ on MSYS.

* Peter Rosin wrote on Mon, Mar 09, 2009 at 09:51:14PM CET: Den 2009-03-04 00:07 skrev Ralf Wildenhues: Not a really nice fix, but I've come up with this below. Can you try it (at least the tests/dep* tests from automake, please)? depcomp.test is ok, depcomp2.test and depcomp3.test

Re: [PATCH 4/4] New automake command line option `--silent-rules'.

* Ralf Wildenhues wrote on Sun, Mar 08, 2009 at 10:46:40AM CET: The current patch still has a couple of warts in that --silent-rules should turn off portability-recursive warnings independently of the command line argument order. This is another reason I don't like this addition much

Re: [PATCH 4/4] New automake command line option `--silent-rules'.

* Jan Engelhardt wrote on Wed, Mar 11, 2009 at 09:34:20PM CET: On Wednesday 2009-03-11 21:06, Ralf Wildenhues wrote: Do we want to allow a command line knob (--silent-rules) to turn off `silent' mode, or do we force developers to either touch the AUTOMAKE_OPTIONS variable in Makefile.am

Re: [PATCH 4/4] New automake command line option `--silent-rules'.

* Jan Engelhardt wrote on Wed, Mar 11, 2009 at 10:55:02PM CET: On Wednesday 2009-03-11 22:43, Ralf Wildenhues wrote: The command line option --silent-rules does the same as the argument `silent' to the AM_INIT_AUTOMAKE macro in configure.ac: AM_INIT_AUTOMAKE([silent]) Both enable silent

Re: automake/533: DISTFILES containing a directory and files in that directory

whether to apply this patch. Cheers, Ralf 2009-03-13 Peter Breitenlohner p...@mppmu.mpg.de Ralf Wildenhues ralf.wildenh...@gmx.de For PR automake/533: DISTFILES containing a directory and files in that directory. When the source tree contains non-writable files

* NEWS: Update.

Pushed to master and branch-1-10 similarly. Cheers, Ralf * NEWS: Update. diff --git a/NEWS b/NEWS index e08a0ce..00ea9cf 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,9 @@ New in 1.10.2a: + - The `gnupload' script has been revamped. + + - The `depcomp' and `compile' scripts now work with

[PATCH 0/N] parallel tests execution in Automake

[ moving from automake@ ] [ N will be a dozen or so, apologies for the mail bombing ] * Ralf Wildenhues wrote on Sat, Mar 14, 2009 at 11:33:21AM CET: I've cleaned up the patch set for parallel tests execution in Automake, and put it in a public branch for people to poke with: git clone git

[PATCH 02] parallel-tests: Ensure backward-compatible semantics.

This patch is virtually unchanged compared to the previous version. Cheers, Ralf parallel-tests: Ensure backward-compatible semantics. For each test in Automake's test suite that uses TESTS, generate an identical one that uses the `parallel-tests' option, for coverage of

[PATCH 03] New tests for `parallel-tests'.

file, test concurrency. * tests/parallel-tests4.test: New file, test suffix rules. * tests/parallel-tests5.test: New file, demonstrate compile/test concurrency. * tests/defs.in: Add a `required' check for rst2html. * tests/Makefile.am: Update. Signed-off-by: Ralf Wildenhues

Re: [PATCH 04] Documentation for the parallel-tests driver.

Compared to http://article.gmane.org/gmane.comp.sysutils.automake.patches/3229 this patch has - docs adjusted to DISABLE_HARD_ERRORS changes and TEST_EXTENSIONS renaming, - VERBOSE change of semantics: hint user to produce verbose tests by default, as suggested by Akim. Cheers, Ralf

Re: [PATCH 05] Overhaul of tests/README.

Overhaul of tests/README. * tests/README: General overhaul. Mention parallel test suite execution, the test-suite.log file; document `Exit' vs. `exit', `defs' vs. `defs-p', the naming of `parallel-tests' tests. * README: Simplify, point to tests/README. diff --git

[PATCH 07] Use append mode to capture parallel test output.

D'oh. Wonder how often we will keep encountering this. Cheers, Ralf Use append mode to capture parallel test output. * tests/lisp8.test: Use append mode for output from `make -j', to avoid dropped lines. * tests/parallel-tests3.test: Likewise. diff --git a/tests/lisp8.test

[PATCH 08] Match XFAIL_TESTS correctly with Solaris make.

Without this patch, Solaris make did not match any XFAIL_TESTS in a VPATH setup. Cheers, Ralf Match XFAIL_TESTS correctly with Solaris make. * lib/am/check.am [PARALLEL_TESTS]: VPATH rewriting may have changed test names, so srcdir needs to be taken into account explicitly

[PATCH 10] Fix parallel-tests with empty $(TESTS) for BSD make.

TESTS= TEST_LOGS=$(TESTS:=.log) would otherwise barf with an error that the file `.log' could not be remade. Since this file name also looks like a single suffix rule, there is no way to portably write a rule to ignore it. So in this case we have to overwrite TESTS. Cheers, Ralf Fix

Re: [PATCH 11] Fix LAZY_TEST_SUITE handling and $(TEST_SUITE_LOG) recreation.

Oh, what a joyful patch; cost me most brains. Here's why: It would have been cool if make check LAZY_TEST_SUITE=yes on an up to date test directory would have no work to do at all. Well. Couple of things wrong with the prior semantics: - there would be no summary output on stdout in that

proposed patch: parallel-tests: also record logs of SKIPped tests.

This is a patch that I don't know whether I want it or not. Package authors may be interested in SKIPs and why they happen, or not. (Could also be solved with leaving the choice to the package author and/or user, by some TEST_RECORD_SKIPS variable or so). Comments appreciated. Thanks, Ralf

proposed patch: parallel-tests: per-extension test driver: ext_COMPILE.

This needs some thinking about the naming scheme to use. Here's why: The current Automake default compile naming scheme has the assumption that file extensions have only one default processing output. For example, .c and .cpp become .$(OBJEXT), .y become .c, and so on. Texinfo is an exception,

Re: [PATCH 07] Use append mode to capture parallel test output.

* Akim Demaille wrote on Sat, Mar 14, 2009 at 01:24:28PM CET: Le 14 mars 09 à 11:56, Ralf Wildenhues a écrit : D'oh. Wonder how often we will keep encountering this. I have not understood what's going on here. Are these files running concurrently in the same pwd? No. The issue

Do not create conditional installation directories

Hello, I have a patch to let 'make install' not create installation directories in which no files will end up. This half-fixes a long-standing TODO item; so far, one had to work around this with conditionals and hacks in order to achieve it. However, there is a downside: this patch does not fix

Re: --silent-rules: status?

Hi Jim, * Jim Meyering wrote on Wed, Mar 18, 2009 at 08:24:40AM CET: I am using your new --silent-rules option (from the je-silent branch) in coreutils, albeit only privately for now. I would like to make it permanent/public, and noticed that this is not yet on next. It should be, though: $

Re: proposed patch: parallel-tests: also record logs of SKIPped tests.

Hi Jim, * Jim Meyering wrote on Sat, Mar 14, 2009 at 12:07:15PM CET: Ralf Wildenhues wrote: This is a patch that I don't know whether I want it or not. Package authors may be interested in SKIPs and why they happen, or not. (Could also be solved with leaving the choice to the package

Re: Do not create conditional installation directories

* Ralf Corsepius wrote on Thu, Mar 19, 2009 at 03:01:45PM CET: Ralf Wildenhues wrote: * Akim Demaille wrote on Wed, Mar 18, 2009 at 06:31:50PM CET: RW == Ralf Wildenhues ralf.wildenh...@gmx.de writes: But the question remains: how do you think we should address the bug

Re: [PATCH 11] Fix LAZY_TEST_SUITE handling and $(TEST_SUITE_LOG) recreation.

Hi Akim, thanks for working through this with me! * Akim Demaille wrote on Sat, Mar 14, 2009 at 01:39:07PM CET: For the records, here is the version I use currently, some of bugs you mentioned being fixed. Thanks. I am applying this improvement: Cheers, Ralf Minor optimization in

SunOS4 compatibility (was: automake/533: DISTFILES containing a directory and files in that directory)

[ adding automake@; this is http://thread.gmane.org/gmane.comp.sysutils.automake.patches/3439 ] * Ralf Wildenhues wrote on Sun, Mar 22, 2009 at 10:17:06AM CET: --- a/lib/am/distdir.am +++ b/lib/am/distdir.am @@ -202,10 +202,16 @@ endif %?TOPDIR_P% ## directory exists only in $(srcdir

Re: How to use compiler wrappers

Hello John, * John R. Cary wrote on Mon, Mar 23, 2009 at 08:02:43PM CET: But on another note, I was using b0201.bassi$ automake --version automake (GNU automake) 1.10.2 and it seems that --tag showed up for f77 code, but not for f90 (--tag=FC). Thanks for mentioning that. It looks like a

<    1   2   3   4   5   6   7   8   9   10   >