Re: [RFC 00/20] Refactoring and modernizations in the depcomp script

[Stefano Lattarini]

Reference:
<http://lists.gnu.org/archive/html/automake-patches/2012-10/msg00012.html>

On 10/18/2012 10:40 PM, Stefano Lattarini wrote:
> OK, after putting this in the backburner for few months, I've decided to
> finally go for some cleanup and refactoring in the 'depcomp' script.
> 
> Unfortunately, I can't test most of the less common depmodes first-hand,
> since I lack access to the required compilers or systems.  So I'd really
> like some feedback and real-world testing from possibly interested persons
> (some of which I'm CC:ing, given their past contributions to the depcomp
> script).  In particular, I'd appreciate if if anyone would able to run the
> depcomp tests in the following compilers:
> 
>   - Microsoft Visual C/C++;
> 
>   - modern versions of the AIX, Tru64 and IRIX vendor C compilers;
> 
>   - the HP aCC (C++) and HP ia64 compilers;
> 
>   - the Portland Group C/C++ compiler;
> 
>   - the IBM C/C++ compilers (xlc/xlC);
> 
>   - The Intel C/C++ compilers, version 8.0 or later.
> 
> Any feedback would be very appreciated!
> 
> Regards,
>   Stefano
> 
> P.S. Sorry for any delay in my replies, but I will be AFK in the
> next few days.  I'll try to catch up after.
> 
> -*-*-*-
> 
> Stefano Lattarini (20):
>   depcomp: be more idiomatic in checking for zero exit statuses
>   depcomp: remove useless quoting in variable definitions
>   depcomp: use "rmdir", not "rm -rf", to remove the empty lock directory
>   depcomp: correctly propagate exit status in exit trap
>   depcomp: safer quoting in variable expansion
>   depcomp: cosmetic fixlets to a comment
>   depcomp: don't rely on character ranges working as in C locale
>   depcomp: reduce code duplication
>   depcomp: avoid an extra fork when possible
>   depcomp: less duplication between AIX and Tru64 modes
>   depcomp: make some code more self-documenting
>   depcomp: shell code style changes
>   depcomp: whitespace fixes
>   depcomp: rewrite a sed invocation for less quoting
>   depcomp: remove an unused variable
>   depcomp: remove last relics of libtool 1.4 support
>   depcomp: improve comments about tcc support
>   tests: fix spurious failures with tcc
>   NEWS: update w.r.t. recent depcomp changes
>   depcomp: remove 'icc' depmode, add 'tcc' one
> 
>  NEWS           |  15 +++
>  lib/depcomp    | 407 
> +++++++++++++++++++++++++++++----------------------------
>  t/depcomp8a.sh |   9 +-
>  t/depcomp8b.sh |   4 +-
>  4 files changed, 228 insertions(+), 207 deletions(-)
> 
FYI: I will push this series (to master) in a couple of days if there is
no review, report or objection by then.

Regards,
  Stefano