GNU bug report logs - #10434
FAIL: depmod.tap 50 - tru64 [long VPATH] make & remake

Previous Next

Full log

View this message in rfc822 format

From: Stefano Lattarini <stefano.lattarini <at> gmail.com>
To: Peter Rosin <peda <at> lysator.liu.se>
Cc: Jim Meyering <jim <at> meyering.net>, automake-patches <at> gnu.org, 10434 <at> debbugs.gnu.org
Subject: bug#10434: FAIL: depmod.tap 50 - tru64 [long VPATH] make & remake
Date: Sun, 05 Feb 2012 14:16:59 +0100

[Message part 1 (text/plain, inline)]

On 02/02/2012 11:41 PM, Peter Rosin wrote:
> Stefano Lattarini skrev 2012-02-02 22:45:
>> Reference:
>>   <http://debbugs.gnu.org/cgi/bugreport.cgi?bug=10434>
>>
>> OK, the attached patch fixes the two spurious failures of GCC forced into
>> Tru64 mode.  About time I'd say.
>>
>> But I'm not sure whether we should apply this without first testing it
>> on a real Tru64 compiler, lest we cause a real regression just to fix a
>> spurious failure.  Thoughts?
> 
> I just had a look at that test, and it seems like a very crappy test
> to me.  I had some failures with cl, but figured it was the same as
> these Tru64 failures that I had seen flying past, and put it all on
> the back burner.  But the test is destined to cause troubles if IIUC.
> 
> It's just dead wrong to assume that feeding -M or -xM to the compiler
> (or whatever other random stuff depcomp might do) and not get an error
> is the same as dependencies magically appearing.  Or do I read the
> test wrong?  Please tell me that I do!
> 
Unfortunately you read the test right.  And in hindsight I must agree
with you: its approach is fundamentally flawed.

So, what about the attached patch, that overhauls (and hopefully improve)
the coverage for automatic dependency tracking support?  It is probably
possible to improve the patch even more (esp. w.r.t. optimizations for
speed), but that can be left for follow-up changes IMHO.

I will push (to master) in 72 hours if there is no objection by then.

Thanks,
  Stefano

[0001-tests-improve-and-rework-tests-on-dependency-trackin.patch (text/x-diff, attachment)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.