Re: depcomp for Portland Group Compilers: get rid of file locking hack?

2012-07-31 Thread Dave Goodell
On Jul 20, 2012, at 9:02 AM CDT, Dave Goodell wrote: > On Jul 12, 2012, at 3:43 PM CDT, Stefano Lattarini wrote: > >> In particular, I propose the attached patch. Can you give it a try to check >> whether it works? > > I'm currently in the process of running the te

Re: depcomp for Portland Group Compilers: get rid of file locking hack?

2012-07-20 Thread Dave Goodell
On Jul 12, 2012, at 3:43 PM CDT, Stefano Lattarini wrote: > Reading the documentation provided here: > > > > I see that the the '-MD' option should also accept an argument: > > -MD,filename (pgcpp only) Generate make dependence lists and print >

Re: [PATCH] silent-many-generic: be stricter in silent rules pattern

2012-02-28 Thread Dave Goodell
Your patch does not work for me. Attached is a fixed version of your patch that matches the offending $CXX value correctly, handles spaces in compiler wrapper arguments, and fixes a typo. If you have a better alternative to the conservative "case" statement that I used, please use it. Between

[PATCH] add pgcc support to depcomp

2012-02-27 Thread Dave Goodell
Portland Group C Compiler support based on a code from Jeff Daily @ PNNL via the automake list and automake bug #8880: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=8880 --- lib/depcomp | 73 +++ 1 files changed, 73 insertions(+), 0 deletio

[PATCH] silent-many-generic: be stricter in silent rules pattern

2012-02-27 Thread Dave Goodell
Otherwise non-silent output such as "pgCC" or "sunCC" could unintentionally match. --- tests/silent-many-generic.test |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/tests/silent-many-generic.test b/tests/silent-many-generic.test index a7a0487..4d254ee 100755 --- a/tests

Re: bug#8880: [PATCH] add pgcc support to depcomp

2012-02-27 Thread Dave Goodell
On Feb 25, 2012, at 3:08 AM CST, Stefano Lattarini wrote: > On 02/25/2012 03:22 AM, Dave Goodell wrote: >> Portland Group C Compiler support based on a code from Jeff Daily @ PNNL >> via the automake list and automake bug #8880: >> > By a very cursory look, this patch se