From patchwork Sun Jan 28 22:56:51 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Michael Forney X-Patchwork-Id: 10188317 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 0517860383 for ; Sun, 28 Jan 2018 22:58:26 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id EA6EB2858A for ; Sun, 28 Jan 2018 22:58:25 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id DE6C2286D7; Sun, 28 Jan 2018 22:58:25 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,T_DKIM_INVALID autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id D09292858A for ; Sun, 28 Jan 2018 22:58:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932115AbeA1W6X (ORCPT ); Sun, 28 Jan 2018 17:58:23 -0500 Received: from mail-pf0-f174.google.com ([209.85.192.174]:41433 "EHLO mail-pf0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932751AbeA1W6V (ORCPT ); Sun, 28 Jan 2018 17:58:21 -0500 Received: by mail-pf0-f174.google.com with SMTP id c6so3231721pfi.8 for ; Sun, 28 Jan 2018 14:58:21 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mforney-org.20150623.gappssmtp.com; s=20150623; h=from:to:subject:date:message-id; bh=paOST6zfdR5vJq997deL8i9QxTn6jQGT9egCIRxlmsE=; b=XfOZFd1Va4yBlFdd+jmap5xat8lXkkPwryxFDHSZqAJAz09/2x4QCgH/BDLOIBLAby ApR1fEYxs7viEWBYmmB15OyKd7x54GwhQjOe3nXNJHKrZTktNW32Q8z78hadQKQ6tek9 +IsY2LHfWMEQPupf+jlSBszbvhQNuHok6MgXpVonezteFlnRPvM6BZXcJc3/y7P3BFFb iwo5jepEwlpEYb0xbPbgsd8V2pm+gMWMQPpmBhHSFzena1K+/q16auB5ySfjUFieFgPR OMKwSIGbbDf1dQjkLmO1g38XTvX+F2kv0NivEtjYIiNf5cIZ9KAOPBu6y7OG9HG2unHZ tqvQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id; bh=paOST6zfdR5vJq997deL8i9QxTn6jQGT9egCIRxlmsE=; b=XjpFI2P3HeMsD0y3wtghRnQDQoBoS2xLME7RZXJo2T2Dus/bgZZfzUqcE9qbuAnKUB ckMbX81Pxa6SgkiQjaErXee91FC5yJmxzlc1Omgg+PWgjqAC56RF/QGNSmnBt5iCHknZ vDE1ClAXeMTBnk14RGett9W7XQ5hB25AXG/fttk0E3zy0BEJVQNxxYxYouUoZISYb/13 xagFScsJQTv4EI33Jbuc2eQk6TP/xjXfvpj3wFZdYn4jZd9IVWXF0gPKECrs0t/hF4ao qn2ojf5I/90sAyI74MPJ6LK/neVGkaLJe0gDABSDUJ1n6dD2O2Ma6ZZ9Fd7atwXg2PP7 uYxg== X-Gm-Message-State: AKwxytfg4PfMjCEf+glJJzGJ96A7eO8C86uv4NVnUzLa2OwH/uGhJmSi P6zXP1TbomYOspJCJYgaIIz7ofQvYoM= X-Google-Smtp-Source: AH8x227X6hGuaED0TIqXk+mhKU6mjqtIatg5Ztwe8QU2Mhq59ZbrirMI0pLv3H96oVN50PDGJfMmkw== X-Received: by 10.98.30.1 with SMTP id e1mr25197700pfe.37.1517180300194; Sun, 28 Jan 2018 14:58:20 -0800 (PST) Received: from localhost ([2601:647:5100:1da4::70af]) by smtp.gmail.com with ESMTPSA id v4sm15645153pgq.23.2018.01.28.14.58.19 for (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Sun, 28 Jan 2018 14:58:19 -0800 (PST) From: Michael Forney To: linux-kbuild@vger.kernel.org Subject: [PATCH] kbuild: Improve portability of some sed invocations Date: Sun, 28 Jan 2018 14:56:51 -0800 Message-Id: <20180128225651.1134-1-mforney@mforney.org> X-Mailer: git-send-email 2.15.1 Sender: linux-kbuild-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kbuild@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Michael Forney * Use BREs where EREs aren't necessary. * Pass -E instead of -r to use EREs. This will be standardized in the next POSIX revision[0]. GNU sed supports this since 4.2 (May 2009), and busybox since 1.22.0 (Jan 2014). * Use a literal instead of \t in bracket expressions. In bracket expressions, POSIX says that loses its special meaning, so a conforming implementation cannot expand \t to [1]. * In BREs, use interval expressions (\{n,m\}) instead of non-standard features like \+ and \?. * Use a loop instead of -s flag. There are still plenty of other cases of non-standard sed invocations (use of ERE features in BREs, in-place editing), but this fixes some core ones. [0] http://austingroupbugs.net/view.php?id=528 [1] http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap09.html#tag_09_03_05 Signed-off-by: Michael Forney --- scripts/Kbuild.include | 2 +- scripts/Makefile.build | 2 +- scripts/adjust_autoksyms.sh | 4 +++- scripts/gen_initramfs_list.sh | 2 +- scripts/headers_install.sh | 10 +++++----- 5 files changed, 11 insertions(+), 9 deletions(-) diff --git a/scripts/Kbuild.include b/scripts/Kbuild.include index 065324a8046f..34cbd81024b0 100644 --- a/scripts/Kbuild.include +++ b/scripts/Kbuild.include @@ -368,7 +368,7 @@ ksym_dep_filter = \ $(CPP) $(call flags_nodeps,a_flags) -D__KSYM_DEPS__ $< ;; \ boot*|build*|cpp_its_S|*cpp_lds_S|dtc|host*|vdso*) : ;; \ *) echo "Don't know how to preprocess $(1)" >&2; false ;; \ - esac | tr ";" "\n" | sed -rn 's/^.*=== __KSYM_(.*) ===.*$$/KSYM_\1/p' + esac | tr ";" "\n" | sed -n 's/^.*=== __KSYM_\(.*\) ===.*$$/KSYM_\1/p' cmd_and_fixdep = \ $(echo-cmd) $(cmd_$(1)); \ diff --git a/scripts/Makefile.build b/scripts/Makefile.build index cb8997ed0149..524000a4ccda 100644 --- a/scripts/Makefile.build +++ b/scripts/Makefile.build @@ -300,7 +300,7 @@ endef # List module undefined symbols (or empty line if not enabled) ifdef CONFIG_TRIM_UNUSED_KSYMS -cmd_undef_syms = $(NM) $@ | sed -n 's/^ \+U //p' | xargs echo +cmd_undef_syms = $(NM) $@ | sed -n 's/^ *U //p' | xargs echo else cmd_undef_syms = echo endif diff --git a/scripts/adjust_autoksyms.sh b/scripts/adjust_autoksyms.sh index 513da1a4a2da..a162258ab606 100755 --- a/scripts/adjust_autoksyms.sh +++ b/scripts/adjust_autoksyms.sh @@ -60,7 +60,9 @@ cat > "$new_ksyms_file" << EOT EOT [ "$(ls -A "$MODVERDIR")" ] && -sed -ns -e '3{s/ /\n/g;/^$/!p;}' "$MODVERDIR"/*.mod | sort -u | +for mod in "$MODVERDIR"/*.mod; do + sed -n -e '3{s/ /\n/g;/^$/!p;}' "$mod" +done | sort -u | while read sym; do if [ -n "$CONFIG_HAVE_UNDERSCORE_SYMBOL_PREFIX" ]; then sym="${sym#_}" diff --git a/scripts/gen_initramfs_list.sh b/scripts/gen_initramfs_list.sh index 86a3c0e5cfbc..10e528b3a08f 100755 --- a/scripts/gen_initramfs_list.sh +++ b/scripts/gen_initramfs_list.sh @@ -194,7 +194,7 @@ input_file() { source="$1" if [ -f "$1" ]; then ${dep_list}header "$1" - is_cpio="$(echo "$1" | sed 's/^.*\.cpio\(\..*\)\?/cpio/')" + is_cpio="$(echo "$1" | sed 's/^.*\.cpio\(\..*\)\{0,1\}/cpio/')" if [ $2 -eq 0 -a ${is_cpio} = "cpio" ]; then cpio_file=$1 echo "$1" | grep -q '^.*\.cpio\..*' && is_cpio_compressed="compressed" diff --git a/scripts/headers_install.sh b/scripts/headers_install.sh index a18bca720995..7328394cf9e8 100755 --- a/scripts/headers_install.sh +++ b/scripts/headers_install.sh @@ -31,13 +31,13 @@ trap 'rm -f "$OUTDIR/$FILE" "$OUTDIR/$FILE.sed"' EXIT for i in "$@" do FILE="$(basename "$i")" - sed -r \ - -e 's/([ \t(])(__user|__force|__iomem)[ \t]/\1/g' \ - -e 's/__attribute_const__([ \t]|$)/\1/g' \ + sed -E \ + -e 's/([ (])(__user|__force|__iomem)[ ]/\1/g' \ + -e 's/__attribute_const__([ ]|$)/\1/g' \ -e 's@^#include @@' \ -e 's/(^|[^a-zA-Z0-9])__packed([^a-zA-Z0-9_]|$)/\1__attribute__((packed))\2/g' \ - -e 's/(^|[ \t(])(inline|asm|volatile)([ \t(]|$)/\1__\2__\3/g' \ - -e 's@#(ifndef|define|endif[ \t]*/[*])[ \t]*_UAPI@#\1 @' \ + -e 's/(^|[ (])(inline|asm|volatile)([ (]|$)/\1__\2__\3/g' \ + -e 's@#(ifndef|define|endif[ ]*/[*])[ ]*_UAPI@#\1 @' \ "$SRCDIR/$i" > "$OUTDIR/$FILE.sed" || exit 1 scripts/unifdef -U__KERNEL__ -D__EXPORTED_HEADERS__ "$OUTDIR/$FILE.sed" \ > "$OUTDIR/$FILE"