Re: [PATCH] Improve AM_SILENT_RULES coverage

2015-10-12 Thread Mark Wielaard
On Wed, 2015-10-07 at 13:40 +0200, Mark Wielaard wrote: > On Tue, 2015-10-06 at 18:00 -0700, Josh Stone wrote: > > Pushed! > > Thanks. I like it so much that I want to make it the default. > Any objections to the following patch? Didn't hear any objections, so AM_SILENT_RULES([yes]) is now the de

Re: [PATCH] Improve AM_SILENT_RULES coverage

2015-10-07 Thread Mark Wielaard
On Tue, 2015-10-06 at 18:00 -0700, Josh Stone wrote: > Pushed! Thanks. I like it so much that I want to make it the default. Any objections to the following patch? From 79ffaa7639fd3330d3968f0b8308a66f140dcc15 Mon Sep 17 00:00:00 2001 From: Mark Wielaard Date: Wed, 7 Oct 2015 13:37:56 +0200 Subj

Re: [PATCH] Improve AM_SILENT_RULES coverage

2015-10-06 Thread Josh Stone
On 10/06/2015 03:02 PM, Mark Wielaard wrote: > On Tue, 2015-10-06 at 10:23 -0700, Josh Stone wrote: >> On 10/06/2015 06:00 AM, Mark Wielaard wrote: >>> On Mon, 2015-10-05 at 11:15 -0700, Josh Stone wrote: Note, elfutils does not explicitly enable AM_SILENT_RULES. It's only available star

Re: [PATCH] Improve AM_SILENT_RULES coverage

2015-10-06 Thread Mark Wielaard
On Tue, 2015-10-06 at 10:23 -0700, Josh Stone wrote: > On 10/06/2015 06:00 AM, Mark Wielaard wrote: > > On Mon, 2015-10-05 at 11:15 -0700, Josh Stone wrote: > >> Note, elfutils does not explicitly enable AM_SILENT_RULES. It's only > >> available starting from automake 1.11 > > > > Note that we al

Re: [PATCH] Improve AM_SILENT_RULES coverage

2015-10-06 Thread Josh Stone
On 10/06/2015 06:00 AM, Mark Wielaard wrote: > On Mon, 2015-10-05 at 11:15 -0700, Josh Stone wrote: >> Note, elfutils does not explicitly enable AM_SILENT_RULES. It's only >> available starting from automake 1.11 > > Note that we already require 1.11 or higher for parallel-tests. > I wouldn't min

Re: [PATCH] Improve AM_SILENT_RULES coverage

2015-10-06 Thread Mark Wielaard
On Mon, 2015-10-05 at 11:15 -0700, Josh Stone wrote: > Note, elfutils does not explicitly enable AM_SILENT_RULES. It's only > available starting from automake 1.11 Note that we already require 1.11 or higher for parallel-tests. I wouldn't mind if we also just added AM_SILENT_RULES([yes]) by defau

[PATCH] Improve AM_SILENT_RULES coverage

2015-10-05 Thread Josh Stone
Note, elfutils does not explicitly enable AM_SILENT_RULES. It's only available starting from automake 1.11, but starting from automake 1.13 silent rules are always generated, defaulting to verbose. $(AM_V_foo) additions should be no-ops on systems that don't support silent rules. To be silent, u