[glibc] s390: Enabling lint-makefiles
Stefan Liebler via Glibc-cvs <[email protected]> Tue, 26 May 2026 09:06:41 +0000 (GMT)
| Newsgroups | gmane.comp.lib.glibc.cvs |
|---|---|
| Message-ID | <[email protected]> |
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=c1b589b21cfee55ae3702f2967553d66ee257b51 commit c1b589b21cfee55ae3702f2967553d66ee257b51 Author: Stefan Liebler <[email protected]> Date: Thu May 21 14:07:24 2026 +0200 s390: Enabling lint-makefiles The s390 specific Makefiles were adjusted to match the required format for scripts/lint-makefiles.sh / scripts/sort-makefile-lines.py. Afterwards the lines were sorted by those scripts. And the testcase lint-makefiles is passing. Reviewed-by: Adhemerval Zanella <[email protected]> Diff: --- sysdeps/s390/Makefile | 213 +++++++++++++++++++++++++--------- sysdeps/s390/multiarch/Makefile | 4 +- sysdeps/s390/nptl/Makefile | 4 +- sysdeps/unix/sysv/linux/s390/Makefile | 33 ++++-- 4 files changed, 192 insertions(+), 62 deletions(-) diff --git a/sysdeps/s390/Makefile b/sysdeps/s390/Makefile index 85446ca0b3..481e834792 100644 --- a/sysdeps/s390/Makefile +++ b/sysdeps/s390/Makefile @@ -1,5 +1,7 @@ ifeq ($(subdir),gmon) -sysdep_routines += s390x-mcount +sysdep_routines += \ + s390x-mcount \ + # sysdep_routines endif ifeq ($(subdir),iconvdata) @@ -15,7 +17,12 @@ UTF16_UTF32_Z9-map := gconv.map UTF8_UTF16_Z9-routines := utf8-utf16-z9 UTF8_UTF16_Z9-map := gconv.map -s390x-iconv-modules = ISO-8859-1_CP037_Z900 UTF8_UTF16_Z9 UTF16_UTF32_Z9 UTF8_UTF32_Z9 +s390x-iconv-modules = \ + ISO-8859-1_CP037_Z900 \ + UTF16_UTF32_Z9 \ + UTF8_UTF16_Z9 \ + UTF8_UTF32_Z9 \ + # s390x-iconv-modules extra-modules-left += $(s390x-iconv-modules) include extra-module.mk @@ -34,7 +41,9 @@ $(inst_gconvdir)/%.so: $(objpfx)%.so $(+force) $(do-install-program) ifdef objpfx -generated += gconv-modules.d/gconv-modules-s390.conf +generated += \ + gconv-modules.d/gconv-modules-s390.conf \ + # generated endif $(inst_gconvdir)/gconv-modules.d/gconv-modules-s390.conf: \ @@ -126,15 +135,22 @@ $(objpfx)glibc-hwcaps/z17/libmarkermod6.so: $(objpfx)libmarkermod6-6.so ifeq (no,$(build-hardcoded-path-in-tests)) # This is an ld.so.cache test, and RPATH/RUNPATH in the executable # interferes with its test objectives. -tests-container += tst-glibc-hwcaps-cache +tests-container += \ + tst-glibc-hwcaps-cache \ + # tests-container endif ifeq ($(build-shared),yes) -tests += tst-dl-runtime-resolve-noaudit tst-dl-runtime-resolve-audit \ - tst-dl-runtime-profile-noaudit +tests += \ + tst-dl-runtime-profile-noaudit \ + tst-dl-runtime-resolve-audit \ + tst-dl-runtime-resolve-noaudit \ + # tests # FIXME: When both LD_PROFILE and LD_AUDIT are set, a segfault occurs. #tests += tst-dl-runtime-profile-audit -modules-names += tst-dl-runtime-mod +modules-names += \ + tst-dl-runtime-mod \ + # modules-names $(objpfx)tst-dl-runtime-resolve-noaudit: $(objpfx)tst-dl-runtime-mod.so $(objpfx)tst-dl-runtime-resolve-audit: $(objpfx)tst-dl-runtime-mod.so $(objpfx)tst-dl-runtime-profile-noaudit: $(objpfx)tst-dl-runtime-mod.so @@ -152,32 +168,85 @@ endif endif ifeq ($(subdir),string) -sysdep_routines += memset memset-z900 \ - memcmp memcmp-z900 \ - mempcpy memcpy memcpy-z900 \ - memmove memmove-c \ - strstr strstr-arch13 strstr-vx strstr-c \ - memmem memmem-arch13 memmem-vx memmem-c \ - strlen strlen-vx strlen-c \ - strnlen strnlen-vx strnlen-c \ - strcpy strcpy-vx strcpy-z900 \ - stpcpy stpcpy-vx stpcpy-c \ - strncpy strncpy-vx strncpy-z900 \ - stpncpy stpncpy-vx stpncpy-c \ - strcat strcat-vx strcat-c \ - strncat strncat-vx strncat-c \ - strcmp strcmp-vx strcmp-z900 \ - strncmp strncmp-vx strncmp-c \ - strchr strchr-vx strchr-c \ - strchrnul strchrnul-vx strchrnul-c \ - strrchr strrchr-vx strrchr-c \ - strspn strspn-vx strspn-c \ - strpbrk strpbrk-vx strpbrk-c \ - strcspn strcspn-vx strcspn-c \ - memchr memchr-vx memchr-z900 \ - rawmemchr rawmemchr-vx rawmemchr-c \ - memccpy memccpy-vx memccpy-c \ - memrchr memrchr-vx memrchr-c +sysdep_routines += \ + memccpy \ + memccpy-c \ + memccpy-vx \ + memchr \ + memchr-vx \ + memchr-z900 \ + memcmp \ + memcmp-z900 \ + memcpy \ + memcpy-z900 \ + memmem \ + memmem-arch13 \ + memmem-c \ + memmem-vx \ + memmove \ + memmove-c \ + mempcpy \ + memrchr \ + memrchr-c \ + memrchr-vx \ + memset \ + memset-z900 \ + rawmemchr \ + rawmemchr-c \ + rawmemchr-vx \ + stpcpy \ + stpcpy-c \ + stpcpy-vx \ + stpncpy \ + stpncpy-c \ + stpncpy-vx \ + strcat \ + strcat-c \ + strcat-vx \ + strchr \ + strchr-c \ + strchr-vx \ + strchrnul \ + strchrnul-c \ + strchrnul-vx \ + strcmp \ + strcmp-vx \ + strcmp-z900 \ + strcpy \ + strcpy-vx \ + strcpy-z900 \ + strcspn \ + strcspn-c \ + strcspn-vx \ + strlen \ + strlen-c \ + strlen-vx \ + strncat \ + strncat-c \ + strncat-vx \ + strncmp \ + strncmp-c \ + strncmp-vx \ + strncpy \ + strncpy-vx \ + strncpy-z900 \ + strnlen \ + strnlen-c \ + strnlen-vx \ + strpbrk \ + strpbrk-c \ + strpbrk-vx \ + strrchr \ + strrchr-c \ + strrchr-vx \ + strspn \ + strspn-c \ + strspn-vx \ + strstr \ + strstr-arch13 \ + strstr-c \ + strstr-vx \ + # sysdep_routines # Exclude fortified routines from being built with _FORTIFY_SOURCE routines_no_fortify += \ @@ -190,25 +259,65 @@ routines_no_fortify += \ endif ifeq ($(subdir),wcsmbs) -sysdep_routines += wcslen wcslen-vx wcslen-c \ - wcsnlen wcsnlen-vx wcsnlen-c \ - wcscpy wcscpy-vx wcscpy-c \ - wcpcpy wcpcpy-vx wcpcpy-c \ - wcsncpy wcsncpy-vx wcsncpy-c \ - wcpncpy wcpncpy-vx wcpncpy-c \ - wcscat wcscat-vx wcscat-c \ - wcsncat wcsncat-vx wcsncat-c \ - wcscmp wcscmp-vx wcscmp-c \ - wcsncmp wcsncmp-vx wcsncmp-c \ - wcschr wcschr-vx wcschr-c \ - wcschrnul wcschrnul-vx wcschrnul-c \ - wcsrchr wcsrchr-vx wcsrchr-c \ - wcsspn wcsspn-vx wcsspn-c \ - wcspbrk wcspbrk-vx wcspbrk-c \ - wcscspn wcscspn-vx wcscspn-c \ - wmemchr wmemchr-vx wmemchr-c \ - wmemset wmemset-vx wmemset-c \ - wmemcmp wmemcmp-vx wmemcmp-c +sysdep_routines += \ + wcpcpy \ + wcpcpy-c \ + wcpcpy-vx \ + wcpncpy \ + wcpncpy-c \ + wcpncpy-vx \ + wcscat \ + wcscat-c \ + wcscat-vx \ + wcschr \ + wcschr-c \ + wcschr-vx \ + wcschrnul \ + wcschrnul-c \ + wcschrnul-vx \ + wcscmp \ + wcscmp-c \ + wcscmp-vx \ + wcscpy \ + wcscpy-c \ + wcscpy-vx \ + wcscspn \ + wcscspn-c \ + wcscspn-vx \ + wcslen \ + wcslen-c \ + wcslen-vx \ + wcsncat \ + wcsncat-c \ + wcsncat-vx \ + wcsncmp \ + wcsncmp-c \ + wcsncmp-vx \ + wcsncpy \ + wcsncpy-c \ + wcsncpy-vx \ + wcsnlen \ + wcsnlen-c \ + wcsnlen-vx \ + wcspbrk \ + wcspbrk-c \ + wcspbrk-vx \ + wcsrchr \ + wcsrchr-c \ + wcsrchr-vx \ + wcsspn \ + wcsspn-c \ + wcsspn-vx \ + wmemchr \ + wmemchr-c \ + wmemchr-vx \ + wmemcmp \ + wmemcmp-c \ + wmemcmp-vx \ + wmemset \ + wmemset-c \ + wmemset-vx \ + # sysdep_routines # Exclude fortified routines from being built with _FORTIFY_SOURCE routines_no_fortify += \ diff --git a/sysdeps/s390/multiarch/Makefile b/sysdeps/s390/multiarch/Makefile index fec3615304..9cc70fd49e 100644 --- a/sysdeps/s390/multiarch/Makefile +++ b/sysdeps/s390/multiarch/Makefile @@ -9,5 +9,7 @@ endef endif ifeq ($(subdir),iconv) -sysdep_routines += gconv_simple +sysdep_routines += \ + gconv_simple \ + # sysdep_routines endif diff --git a/sysdeps/s390/nptl/Makefile b/sysdeps/s390/nptl/Makefile index 94b759df1f..ef845854d9 100644 --- a/sysdeps/s390/nptl/Makefile +++ b/sysdeps/s390/nptl/Makefile @@ -16,5 +16,7 @@ # <https://www.gnu.org/licenses/>. ifeq ($(subdir),csu) -gen-as-const-headers += tcb-offsets.sym +gen-as-const-headers += \ + tcb-offsets.sym \ + # gen-as-const-headers endif diff --git a/sysdeps/unix/sysv/linux/s390/Makefile b/sysdeps/unix/sysv/linux/s390/Makefile index e9ef5c51ac..3033290998 100644 --- a/sysdeps/unix/sysv/linux/s390/Makefile +++ b/sysdeps/unix/sysv/linux/s390/Makefile @@ -1,22 +1,39 @@ ifeq ($(subdir),elf) ifeq (yes,$(build-shared)) # This is needed to support g++ v2 and v3. -sysdep_routines += framestate -shared-only-routines += framestate +sysdep_routines += \ + framestate \ + # sysdep_routines +shared-only-routines += \ + framestate \ + # shared-only-routines endif endif ifeq ($(subdir),rt) -librt-routines += rt-sysdep -librt-shared-only-routines += rt-sysdep +librt-routines += \ + rt-sysdep \ + # librt-routines + +librt-shared-only-routines += \ + rt-sysdep \ + # librt-shared-only-routines endif ifeq ($(subdir),stdlib) -gen-as-const-headers += ucontext_i.sym -sysdep_routines += __makecontext_ret +gen-as-const-headers += \ + ucontext_i.sym \ + # gen-as-const-headers +sysdep_routines += \ + __makecontext_ret \ + # sysdep_routines endif ifeq ($(subdir),misc) -tests += tst-ptrace-singleblock -sysdep_headers += sys/elf.h +tests += \ + tst-ptrace-singleblock \ + # tests +sysdep_headers += \ + sys/elf.h \ + # sysdep_headers endif