From patchwork Mon May 8 20:02:06 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Zack Weinberg X-Patchwork-Id: 20315 Received: (qmail 130909 invoked by alias); 8 May 2017 20:02:10 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 130897 invoked by uid 89); 8 May 2017 20:02:10 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-24.2 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_SORBS_SPAM, RP_MATCHES_RCVD, SPF_PASS autolearn=ham version=3.3.2 spammy=whatsoever, 1085, so, 1014 X-HELO: mailbackend.panix.com X-Gm-Message-State: AN3rC/7qIYx2nxaJfNywtO36ymFKnJZbfQDQ883It4uwGT4QvacF8Dmk C62jP+7weM3/z5x9cim8zoVWf1kxjg== X-Received: by 10.107.153.82 with SMTP id b79mr55437464ioe.200.1494273727219; Mon, 08 May 2017 13:02:07 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <20170322125504.4863-1-zackw@panix.com> <20170322125504.4863-3-zackw@panix.com> From: Zack Weinberg Date: Mon, 8 May 2017 16:02:06 -0400 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 2/2] Suppress internal declarations for most of the testsuite. To: Andreas Schwab Cc: GNU C Library On Mon, May 8, 2017 at 10:01 AM, Andreas Schwab wrote: > On Mär 22 2017, Zack Weinberg wrote: > > In fact, cppflags-iterator.mk is misnamed since a long time, since it no > longer sets CPPFLAGS-*. Would you care to rename it in a separate > patch? How's this? --- Rename cppflags-iterator.mk to libof-iterator.mk, remove extra-modules.mk. cppflags-iterator.mk no longer has anything to do with CPPFLAGS; all it does is set libof-$(foo) for a list of files. extra-modules.mk does the same thing, but with a different input variable, and doesn't let the caller control the module. Therefore, this patch gives cppflags-iterator.mk a better name, removes extra-modules.mk, and updates all uses of both. Tested on x86-64-linux-gnu. Should have no effect whatsoever on installed files. * extra-modules.mk: Delete file. * cppflags-iterator.mk: Rename to ... * libof-iterator.mk: ...this. Adjust comments. * Makerules, extra-lib.mk, benchtests/Makefile, elf/Makefile * elf/rtld-Rules, iconv/Makefile, locale/Makefile, malloc/Makefile * nscd/Makefile, sunrpc/Makefile, sysdeps/s390/Makefile: Use libof-iterator.mk instead of cppflags-iterator.mk or extra-modules.mk. * benchtests/strcoll-inputs/filelist#en_US.UTF-8: Remove extra-modules.mk and cppflags-iterator.mk, add libof-iterator.mk. --- Makerules | 10 ++++------ benchtests/Makefile | 2 +- benchtests/strcoll-inputs/filelist#en_US.UTF-8 | 3 +-- elf/Makefile | 2 +- elf/rtld-Rules | 2 +- extra-lib.mk | 2 +- extra-modules.mk | 9 --------- iconv/Makefile | 2 +- iconvdata/Makefile | 2 +- cppflags-iterator.mk => libof-iterator.mk | 5 +++-- locale/Makefile | 2 +- malloc/Makefile | 2 +- nscd/Makefile | 2 +- sunrpc/Makefile | 2 +- sysdeps/s390/Makefile | 2 +- 15 files changed, 16 insertions(+), 30 deletions(-) delete mode 100644 extra-modules.mk rename cppflags-iterator.mk => libof-iterator.mk (47%) diff --git a/cppflags-iterator.mk b/libof-iterator.mk similarity index 47% rename from cppflags-iterator.mk rename to libof-iterator.mk index 89a472bd82..31560ce231 100644 --- a/cppflags-iterator.mk +++ b/libof-iterator.mk @@ -1,5 +1,6 @@ -# This file is included several times in a row, once -# for each element of $(lib)-routines and $(lib)-sysdeps_routines. +# This file is included several times in a row, once for each element +# $(cpp-src) of $(cpp-srcs-left). It sets libof-$(cpp-src) to $(lib) +# for each. cpp-src := $(firstword $(cpp-srcs-left)) cpp-srcs-left := $(filter-out $(cpp-src),$(cpp-srcs-left)) diff --git a/extra-modules.mk b/extra-modules.mk deleted file mode 100644 index 5f8e693b15..0000000000 --- a/extra-modules.mk +++ /dev/null @@ -1,9 +0,0 @@ -# This file is included several times in a row, once -# for each element of $(modules-names). $(extra-modules-left) -# is initialized first to $(modules-names) so that with each -# inclusion, we advance $(module) to the next name. - -module := $(firstword $(extra-modules-left)) -extra-modules-left := $(filter-out $(module),$(extra-modules-left)) - -libof-$(notdir $(module)) := extramodules diff --git a/extra-lib.mk b/extra-lib.mk index 2552049135..72f8d2e1df 100644 --- a/extra-lib.mk +++ b/extra-lib.mk @@ -108,5 +108,5 @@ endif # This will define `libof-ROUTINE := LIB' for each of the routines. cpp-srcs-left := $($(lib)-routines) $($(lib)-sysdep_routines) ifneq (,$(cpp-srcs-left)) -include $(patsubst %,$(..)cppflags-iterator.mk,$(cpp-srcs-left)) +include $(patsubst %,$(..)libof-iterator.mk,$(cpp-srcs-left)) endif diff --git a/Makerules b/Makerules index 7f0eef8096..15f46e5f3a 100644 --- a/Makerules +++ b/Makerules @@ -794,11 +794,9 @@ endif # The makefile may define $(modules-names) to build additional modules. # These are built with $(build-module), except any in $(modules-names-nobuild). ifdef modules-names -# extra-lib.mk is included once for each extra lib to define rules -# to build it, and to add its objects to the various variables. -# During its evaluation, $(lib) is set to the name of the library. -extra-modules-left := $(modules-names) -include $(patsubst %,$(..)extra-modules.mk,$(modules-names)) +cpp-srcs-left := $(modules-names) +lib = extramodules +include $(patsubst %,$(..)libof-iterator.mk,$(modules-names)) extra-modules-build := $(filter-out $(modules-names-nobuild),$(modules-names)) $(extra-modules-build:%=$(objpfx)%.so): $(objpfx)%.so: \ @@ -1327,7 +1325,7 @@ all-nonlib = $(strip $(tests) $(xtests) $(test-srcs) $(test-extras) $(others)) ifneq (,$(all-nonlib)) cpp-srcs-left = $(all-nonlib) lib := nonlib -include $(patsubst %,$(..)cppflags-iterator.mk,$(cpp-srcs-left)) +include $(patsubst %,$(..)libof-iterator.mk,$(cpp-srcs-left)) endif diff --git a/benchtests/Makefile b/benchtests/Makefile index a96e9533b3..7f5fda5ef4 100644 --- a/benchtests/Makefile +++ b/benchtests/Makefile @@ -123,7 +123,7 @@ endif cpp-srcs-left := $(binaries-benchset:=.c) $(binaries-bench:=.c) \ $(binaries-bench-malloc:=.c) lib := nonlib -include $(patsubst %,$(..)cppflags-iterator.mk,$(cpp-srcs-left)) +include $(patsubst %,$(..)libof-iterator.mk,$(cpp-srcs-left)) bench-extra-objs = json-lib.o diff --git a/elf/Makefile b/elf/Makefile index cc4aeb25b6..baf967821c 100644 --- a/elf/Makefile +++ b/elf/Makefile @@ -547,7 +547,7 @@ CFLAGS-rtld.c = $(SYSCONF-FLAGS) cpp-srcs-left := $(all-rtld-routines:=.os) lib := rtld -include $(patsubst %,$(..)cppflags-iterator.mk,$(cpp-srcs-left)) +include $(patsubst %,$(..)libof-iterator.mk,$(cpp-srcs-left)) test-modules = $(addprefix $(objpfx),$(addsuffix .so,$(strip $(modules-names)))) generated += $(addsuffix .so,$(strip $(modules-names))) diff --git a/elf/rtld-Rules b/elf/rtld-Rules index 99b5899f7d..2c7b99828c 100644 --- a/elf/rtld-Rules +++ b/elf/rtld-Rules @@ -142,7 +142,7 @@ endif # Set libof-* for each routine. cpp-srcs-left := $(rtld-modules:%.os=%) lib := rtld -include $(patsubst %,$(..)cppflags-iterator.mk,$(cpp-srcs-left)) +include $(patsubst %,$(..)libof-iterator.mk,$(cpp-srcs-left)) rtld-CFLAGS += $(no-stack-protector) diff --git a/iconv/Makefile b/iconv/Makefile index 2b2bc8e303..b2fead0479 100644 --- a/iconv/Makefile +++ b/iconv/Makefile @@ -55,7 +55,7 @@ CFLAGS-iconvconfig.c = -DGCONV_PATH='"$(gconvdir)"' -DGCONV_DIR='"$(gconvdir)"' # Set libof-* for each routine. cpp-srcs-left := $(iconv_prog-modules) $(iconvconfig-modules) lib := iconvprogs -include $(patsubst %,$(..)cppflags-iterator.mk,$(cpp-srcs-left)) +include $(patsubst %,$(..)libof-iterator.mk,$(cpp-srcs-left)) ifeq ($(run-built-tests),yes) xtests-special += $(objpfx)test-iconvconfig.out diff --git a/iconvdata/Makefile b/iconvdata/Makefile index e4845871f5..a0e297e4de 100644 --- a/iconvdata/Makefile +++ b/iconvdata/Makefile @@ -290,7 +290,7 @@ cpp-srcs-left := $(modules) $(generated-modules) $(libJIS-routines) \ $(libKSC-routines) $(libGB-routines) $(libCNS-routines) \ $(libISOIR165-routines) $(libJISX0213-routines) lib := iconvdata -include $(patsubst %,$(..)cppflags-iterator.mk,$(cpp-srcs-left)) +include $(patsubst %,$(..)libof-iterator.mk,$(cpp-srcs-left)) tst-loading-ENV = MALLOC_TRACE=$(objpfx)tst-loading.mtrace $(objpfx)mtrace-tst-loading.out: $(objpfx)tst-loading.out diff --git a/locale/Makefile b/locale/Makefile index 29a34b1f26..d9ef48fe93 100644 --- a/locale/Makefile +++ b/locale/Makefile @@ -101,4 +101,4 @@ CFLAGS-charmap-dir.c = -Wno-write-strings cpp-srcs-left := $(localedef-modules) $(localedef-aux) $(locale-modules) \ $(lib-modules) lib := locale-programs -include $(patsubst %,$(..)cppflags-iterator.mk,$(cpp-srcs-left)) +include $(patsubst %,$(..)libof-iterator.mk,$(cpp-srcs-left)) diff --git a/malloc/Makefile b/malloc/Makefile index e93b83b57d..e380e9629b 100644 --- a/malloc/Makefile +++ b/malloc/Makefile @@ -126,7 +126,7 @@ memusagestat-modules = memusagestat cpp-srcs-left := $(memusagestat-modules) lib := memusagestat -include $(patsubst %,$(..)cppflags-iterator.mk,$(cpp-srcs-left)) +include $(patsubst %,$(..)libof-iterator.mk,$(cpp-srcs-left)) $(objpfx)memusagestat: $(memusagestat-modules:%=$(objpfx)%.o) $(LINK.o) -o $@ $^ $(libgd-LDFLAGS) -lgd -lpng -lz -lm diff --git a/nscd/Makefile b/nscd/Makefile index 0385a370f0..4126996887 100644 --- a/nscd/Makefile +++ b/nscd/Makefile @@ -92,7 +92,7 @@ endif # Set libof-nscd. cpp-srcs-left := $(nscd-modules) lib := nscd -include $(patsubst %,$(..)cppflags-iterator.mk,$(cpp-srcs-left)) +include $(patsubst %,$(..)libof-iterator.mk,$(cpp-srcs-left)) $(objpfx)nscd: $(nscd-modules:%=$(objpfx)%.o) diff --git a/sunrpc/Makefile b/sunrpc/Makefile index a5177fffdc..08bb9c6822 100644 --- a/sunrpc/Makefile +++ b/sunrpc/Makefile @@ -183,7 +183,7 @@ $(objpfx)cross-rpcgen: $(cross-rpcgen-objs) # This makes sure -DIN_MODULE is passed for all these modules. cpp-srcs-left := $(rpcgen-objs:.o=.c) lib := nonlib -include $(patsubst %,$(..)cppflags-iterator.mk,$(cpp-srcs-left)) +include $(patsubst %,$(..)libof-iterator.mk,$(cpp-srcs-left)) # How we run rpcgen to generate sources and headers in the rules below. # Setting CPP tells it how to run the C preprocessor correctly. Note diff --git a/sysdeps/s390/Makefile b/sysdeps/s390/Makefile index d50836579f..8a54f88cd7 100644 --- a/sysdeps/s390/Makefile +++ b/sysdeps/s390/Makefile @@ -18,7 +18,7 @@ include extra-module.mk cpp-srcs-left := $(foreach mod,$(s390x-iconv-modules),$($(mod)-routines)) lib := iconvdata -include $(patsubst %,$(..)cppflags-iterator.mk,$(cpp-srcs-left)) +include $(patsubst %,$(..)libof-iterator.mk,$(cpp-srcs-left)) extra-objs += $(addsuffix .so, $(s390x-iconv-modules)) install-others += $(patsubst %, $(inst_gconvdir)/%.so, $(s390x-iconv-modules)) diff --git a/benchtests/strcoll-inputs/filelist#en_US.UTF-8 b/benchtests/strcoll-inputs/filelist#en_US.UTF-8 index b7b38017d8..6cee2374f4 100644 --- a/benchtests/strcoll-inputs/filelist#en_US.UTF-8 +++ b/benchtests/strcoll-inputs/filelist#en_US.UTF-8 @@ -9667,7 +9667,6 @@ hr.po libc.pot ko.po ru.po -extra-modules.mk intl tst-gettext4-fr.po tstcodeset.po @@ -13252,7 +13251,6 @@ sigset.h atomic.h utmp.h stdlib-bsearch.h -cppflags-iterator.mk iconvdata MISSING ibm1164.h @@ -15245,3 +15243,4 @@ w_exp2l.c bug-nexttoward.c w_dreml.c s_nanf.c +libof-iterator.mk