[SCM] make branch, master, updated. 4.3-89-gc5d4b7b

[Paul D. Smith]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "make".

The branch, master has been updated
       via  c5d4b7b2f260c85ba48db4e844c4a2e4e3004238 (commit)
       via  9e4b3d3f4c8241a2c3cd66e1a7695befa5a6a0aa (commit)
       via  5a96d5a066e86178886b80ad00130faf1cc079bf (commit)
       via  4488592ddae404fb11ce874d16589270b5e4c787 (commit)
       via  e7eb8b5962efe316b08a4a6209d2a63130d1f22f (commit)
       via  0c2fc00544b89314643561dcb6d78f35eb98da68 (commit)
       via  214df0e92a49203ca96fc8ed28799994f8bc6cb3 (commit)
       via  b2e898d0bfaeead6c2b4a2a12d3f7fdc6f47fb51 (commit)
       via  abb1e8d10b4751eeeb7304c77cead828d248094b (commit)
       via  cc6dc70b160b81ca93e50db6a760bba9a670f4b0 (commit)
       via  2638552859ef4ec0e75c1f9e681468934674a626 (commit)
      from  de48d6a4f71329a91d38b90510abd28110744ad9 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit c5d4b7b2f260c85ba48db4e844c4a2e4e3004238
Author: Paul Smith <psmith@gnu.org>
Date:   Tue Sep 7 01:02:25 2021 -0400

    Clean up more Windows warnings.
    
    * src/config.h.W32.template: Re-enable warnings that we don't violate.
    Add a new disabled warning: 4130 (logical operation on address of
    string constant).
    * src/w32/compat/posixfcn.c: Make makeint.h the first included header.
    * src/w32/subproc/sub_proc.c: Ditto.  It includes config.h already.

commit 9e4b3d3f4c8241a2c3cd66e1a7695befa5a6a0aa
Author: Dmitry Goncharov <dgoncharov@users.sf.net>
Date:   Tue Sep 7 00:08:27 2021 -0400

    [SV 60435] Ensure intermediate grouped files are removed
    
    If multiple intermediate files are built together be sure all of them
    are removed after make is finished.  Continue to ensure that targets
    that exist before make starts are not removed even if they appear to be
    intermediate.
    
    Add a number of tests to pattern rules to verify this behavior.
    
    * src/filedef.h (struct file): Add a new is_explicit bitfield.
    * src/file.c (rehash_file): Merge the is_explicit bit.
    (enter_prereqs): Set is_explicit if the file is explicitly mentioned.
    * src/implicit.c (pattern_search): Set intermediate on the file if it's
    not explicit.
    (record_files): Set is_explicit if a file is mentioned as a target.
    * src/remake.c (update_file_1): Set secondary on files that already
    exist so they won't be removed.
    * tests/scripts/features/double_colon: Add a test for double-colon
    pattern rules.
    * tests/scripts/features/patternrules: Update KGO for tests where more
    files are removed.
    Add new tests to verify handling removal of intermediate files in the
    context of grouped pattern targets.

commit 5a96d5a066e86178886b80ad00130faf1cc079bf
Author: Paul Smith <psmith@gnu.org>
Date:   Mon Sep 6 20:42:34 2021 -0400

    * src/default.c (default_variables): Use CPP in PREPROCESS.S.

commit 4488592ddae404fb11ce874d16589270b5e4c787
Author: Dmitry Goncharov <dgoncharov@users.sf.net>
Date:   Mon Sep 6 20:36:15 2021 -0400

    * src/main.c (decode_debug_flags): [SV 607777] Preserve -d options
    
    * tests/scripts/options/dash-d: Test that -d --trace keeps -d.

commit e7eb8b5962efe316b08a4a6209d2a63130d1f22f
Author: Dmitry Goncharov <dgoncharov@users.sf.net>
Date:   Mon Sep 6 20:20:28 2021 -0400

    [SV 57778] Don't ignore included makefiles that can't be read
    
    If we find an included makefile but it's not readable, stop
    immediately with an error rather than continuing to look in other
    directories.
    
    * src/read.c (eval_makefile): Only keep searching if the fopen error
    is ENOENT, else stop and fail.
    * tests/scripts/features/include: Add tests to verify this behavior.

commit 0c2fc00544b89314643561dcb6d78f35eb98da68
Author: Paul Smith <psmith@gnu.org>
Date:   Mon Sep 6 17:47:04 2021 -0400

    [SV 60795] Don't remake phony included makefiles and show errors
    
    Change the handling of included makefiles which are phony targets to
    be similar to double-colon rules with no prerequisites: simply don't
    build them at all during the remake a makefile phase.
    
    Ensure that any included makefile which is needed but not built
    results in an error.
    
    Update the documentation to make this clear.
    Add tests to verify this behavior.
    
    * doc/make.texi (Remaking Makefiles): Clarify double-colon exception.
    Document that phony targets are handled the same way.
    (Phony Targets): Ditto.
    * src/main.c (main): Check for phony targets when skipping goals.
    Rather than throwing out skipped but failed goals keep them
    separately then report them as errors.
    * src/read.c (eval): Set the file location on included makefiles even
    when there's no error.
    * tests/scripts/features/include: Add tests for handling included
    makefiles with both phony and double-colon rules to rebuild them.

commit 214df0e92a49203ca96fc8ed28799994f8bc6cb3
Author: Paul Smith <psmith@gnu.org>
Date:   Mon Sep 6 10:57:43 2021 -0400

    * doc/make.texi (Environment): [SV 60841] Clarify variable exporting.

commit b2e898d0bfaeead6c2b4a2a12d3f7fdc6f47fb51
Author: Paul Smith <psmith@gnu.org>
Date:   Mon Sep 6 10:46:52 2021 -0400

    * doc/make.texi (Chained Rules): [SV 60904] Clarify intermediate files.

commit abb1e8d10b4751eeeb7304c77cead828d248094b
Author: Paul Smith <psmith@gnu.org>
Date:   Mon Sep 6 00:14:57 2021 -0400

    Remove UBSAN issues discovered via fuzzing tests.
    
    The arithmetic conversions in C say that if a binary operator has an
    unsigned and signed type as operands and the unsigned type has a greater
    rank then the signed value is converted to unsigned.  This is bad if the
    signed value is negative.
    
    There are a few places in the code which have this situation; convert
    the signed value to positive and add instead of subtracting.
    
    Reported by He Jingxuan <jingxuan.he@inf.ethz.ch>
    
    * src/read.c (find_map_unquote): Use a positive int in memmove().
    (find_char_unquote): Ditto.
    (find_percent_cached): Ditto.

commit cc6dc70b160b81ca93e50db6a760bba9a670f4b0
Author: Paul Smith <psmith@gnu.org>
Date:   Sun Sep 5 23:28:03 2021 -0400

    * doc/make.texi (Prerequisite Types): [SV 61069] Mention PHONY prereqs

commit 2638552859ef4ec0e75c1f9e681468934674a626
Author: Paul Smith <psmith@gnu.org>
Date:   Sun Sep 5 23:17:26 2021 -0400

    * doc/make.1: Clean up man page text.

-----------------------------------------------------------------------

Summary of changes:
 doc/make.1                          |   6 +-
 doc/make.texi                       | 109 +++++++++------
 src/config.h.W32.template           |   7 +-
 src/default.c                       |   4 +-
 src/file.c                          |   4 +
 src/filedef.h                       |   1 +
 src/implicit.c                      |  11 +-
 src/main.c                          | 143 +++++++++++--------
 src/read.c                          |  52 +++++--
 src/remake.c                        |   4 +
 src/w32/compat/posixfcn.c           |   3 +-
 src/w32/subproc/sub_proc.c          |   4 +-
 tests/scripts/features/double_colon |  15 +-
 tests/scripts/features/include      | 164 +++++++++++++++++++++-
 tests/scripts/features/patternrules | 267 +++++++++++++++++++++++++-----------
 tests/scripts/options/dash-d        |   9 ++
 16 files changed, 587 insertions(+), 216 deletions(-)
 create mode 100644 tests/scripts/options/dash-d


hooks/post-receive
-- 
make