Re: [PATCH v1 1/1] srcu-cbmc: Use /usr/bin/awk instead of /bin/awk

From: Paul E. McKenney
Date: Sun Apr 23 2017 - 23:33:59 EST


On Sun, Apr 23, 2017 at 08:24:11PM -0700, Lance Roy wrote:
> This is fine with me. I didn't notice that /bin/awk was just a symlink on the
> distro I was using when I wrote the script.

Very good! May I please have your Acked-by?

Thanx, Paul

> Thanks,
> Lance
>
> On Fri, 21 Apr 2017 10:50:42 -0700
> "Paul E. McKenney" <paulmck@xxxxxxxxxxxxxxxxxx> wrote:
>
> > On Sat, Apr 22, 2017 at 10:17:11AM -0700, priyalee.kushwaha@xxxxxxxxx wrote:
> > > From: Priyalee Kushwaha <priyalee.kushwaha@xxxxxxxxx>
> > >
> > > Most OS distribution have awk in /usr/bin not in /bin
> > > Without this patch, kernel-devsrc fails to build as
> > > runtime dependency for srcu-cbmc script /bin/awk is
> > > not found.
> >
> > Adding Lance on CC.
> >
> > Thanx, Paul
> >
> > > Signed-off-by: Kushwaha, Priyalee <priyalee.kushwaha@xxxxxxxxx>
> > > ---
> > > tools/testing/selftests/rcutorture/formal/srcu-cbmc/modify_srcu.awk | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git
> > > a/tools/testing/selftests/rcutorture/formal/srcu-cbmc/modify_srcu.awk
> > > b/tools/testing/selftests/rcutorture/formal/srcu-cbmc/modify_srcu.awk index
> > > 8ff8904..c9e8bc5 100755 ---
> > > a/tools/testing/selftests/rcutorture/formal/srcu-cbmc/modify_srcu.awk +++
> > > b/tools/testing/selftests/rcutorture/formal/srcu-cbmc/modify_srcu.awk @@
> > > -1,4 +1,4 @@ -#!/bin/awk -f +#!/usr/bin/awk -f
> > >
> > > # Modify SRCU for formal verification. The first argument should be srcu.h
> > > and # the second should be srcu.c. Outputs modified srcu.h and srcu.c into
> > > the --
> > > 2.10.0
> > >
> >
>