From mboxrd@z Thu Jan 1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
Received: from dpdk.org (dpdk.org [92.243.14.124])
by inbox.dpdk.org (Postfix) with ESMTP id 6E0DFA058A;
Fri, 17 Apr 2020 14:50:37 +0200 (CEST)
Received: from [92.243.14.124] (localhost [127.0.0.1])
by dpdk.org (Postfix) with ESMTP id AE4B11E567;
Fri, 17 Apr 2020 14:50:35 +0200 (CEST)
Received: from mail-io1-f68.google.com (mail-io1-f68.google.com
[209.85.166.68]) by dpdk.org (Postfix) with ESMTP id A81601E551
for <dev@dpdk.org>; Fri, 17 Apr 2020 14:50:34 +0200 (CEST)
Received: by mail-io1-f68.google.com with SMTP id o127so2181915iof.0
for <dev@dpdk.org>; Fri, 17 Apr 2020 05:50:34 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
d=emumba-com.20150623.gappssmtp.com; s=20150623;
h=mime-version:references:in-reply-to:from:date:message-id:subject:to
:cc; bh=KE/UuKkl5YqFAAHfHIQV5efCfA9GpJiIAm8T1FdLDIs=;
b=eIjrXLRnyqvf1H883opGC/a51cSaRcScugy/M7/0ASmP76RaL0QqBmH/+IwImM2HlH
MXMNtU4F+DX2GnoJdPITWAXN25HfFnyyw9JVwt6R8575l4WKscN7k8WNXB1z1WpySUYC
R9bRGuAvtqhIbJxesUN2ucTEIhyvZ4uf9FzS46HFSo+H/AVqpJud4B1mYAH7NFHm7Xiv
pHng6qjcK5OUGPrRpmoJr+rnVuZ6+jLjAf828L9EgoDgeZYo/mgShKnopJhYv9JRx2u3
1RKwWSz5dBIMczTZScHDPfou4EDpudyfJgMDyGzZKcOJnF5u1zMrAzwwI1zSpdd6V4wB
LrEQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
d=1e100.net; s=20161025;
h=x-gm-message-state:mime-version:references:in-reply-to:from:date
:message-id:subject:to:cc;
bh=KE/UuKkl5YqFAAHfHIQV5efCfA9GpJiIAm8T1FdLDIs=;
b=r9M5yZAgnhdryzU3PiiU7H6C4xV636qHs9JdyG+DjvxhhABs4LjwbNxrD/I3NwboQe
FC5nXUopU+pNRyy+iU+FC1ufGglE8OitGgajiTMSDJ8G13u/y+8DVF6k0vnH8iJoxA3G
yxUd+xLUkxGdauRFzxwrIaPQZTDcshTE6/7/1ot1MdfWs/cQcAIw36tkBP5ZUOgbg2Tr
sgqO1OPoLp0j7t7lh5Lc+N4+Pi0lWRdB+j3YFOZveefm7b+4yWsCzWXu7pNxFEIBpi68
tCt5sqWidOLtHI6/YXXCPhRPfzX9vhpZSpR41Q1jMnaHdrvjVUC09UWPSam7WKBinNRZ
Kclw==
X-Gm-Message-State: AGi0PuZGhHEmBBDcZeGBNwgtBKLrZlQ+q2nQDsN5GPqvma+tg256k/4e
1VoidUIiSqKhagzTlHuVB2ECQiZD1jRUAQGU3jnFDduo
X-Google-Smtp-Source: APiQypIaMouCnXH8dbFOkS2bYWXo1mysXrXrDA6r11QDL9eMruen9S04tLx9NsAID5R2zV79V13rX2Wskp9VWJWW6Hg=
X-Received: by 2002:a02:2a4a:: with SMTP id w71mr3121820jaw.75.1587127833851;
Fri, 17 Apr 2020 05:50:33 -0700 (PDT)
MIME-Version: 1.0
References: <20200320065002.5410-1-sarosh.arif@emumba.com>
In-Reply-To: <20200320065002.5410-1-sarosh.arif@emumba.com>
From: Sarosh Arif <sarosh.arif@emumba.com>
Date: Fri, 17 Apr 2020 17:50:22 +0500
Message-ID: <CABoZmYOjxNzu02QdLrmet8Xt3cmXrMHRbbO=M3sA=0m9sVucbw@mail.gmail.com>
To: dev@dpdk.org
Cc: stable@dpdk.org
Content-Type: text/plain; charset="UTF-8"
X-Content-Filtered-By: Mailman/MimeDel 2.1.15
Subject: Re: [dpdk-dev] [PATCH v2] usertools/dpdk-setup.sh: fix dpdk-setup's
behaviour on non-alphanumeric inputs
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
<mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
<mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>
Hello,
If there is no objection on this patch then it can perhaps be applied.
On Fri, Mar 20, 2020 at 11:50 AM Sarosh Arif <sarosh.arif@emumba.com> wrote:
> Bugzilla ID: 419
> Cc: stable@dpdk.org
> Signed-off-by: Sarosh Arif <sarosh.arif@emumba.com>
> ---
> usertools/dpdk-setup.sh | 14 ++++++++++----
> 1 file changed, 10 insertions(+), 4 deletions(-)
>
> diff --git a/usertools/dpdk-setup.sh b/usertools/dpdk-setup.sh
> index e5bbe9fee..c27f89c5f 100755
> --- a/usertools/dpdk-setup.sh
> +++ b/usertools/dpdk-setup.sh
> @@ -592,11 +592,17 @@ while [ "$QUIT" == "0" ]; do
> echo "[$OPTION_NUM] Exit Script"
> OPTIONS[$OPTION_NUM]="quit"
> echo ""
> - echo -n "Option: "
> - read our_entry
> + read -p "Option: " our_entry
> + [ $? -eq 0 ] || exit 0
> +
> echo ""
> - ${OPTIONS[our_entry]} ${our_entry}
> -
> + numeric="^[[:digit:]]+$"
> + if [[ "$our_entry" =~ $numeric ]]; then
> + ${OPTIONS[our_entry]} ${our_entry}
> + else
> + echo "Please enter a numeric value"
> + fi
> +
> if [ "$QUIT" == "0" ] ; then
> echo
> echo -n "Press enter to continue ..."; read
> --
> 2.17.1
>
>