[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RISU PATCH v4 09/10] new: record_traces.sh helper scri
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [RISU PATCH v4 09/10] new: record_traces.sh helper script |
Date: |
Tue, 06 Jun 2017 15:25:24 +0100 |
User-agent: |
mu4e 0.9.19; emacs 25.2.50.3 |
Peter Maydell <address@hidden> writes:
> On 2 June 2017 at 17:08, Alex Bennée <address@hidden> wrote:
>> A simple script to run through a bunch of binaries and generate their
>> trace files.
>>
>> Signed-off-by: Alex Bennée <address@hidden>
>>
>> ---
>> v3
>> - allow overriding of RISU binary
>> ---
>> record_traces.sh | 20 ++++++++++++++++++++
>> 1 file changed, 20 insertions(+)
>> create mode 100755 record_traces.sh
>>
>> diff --git a/record_traces.sh b/record_traces.sh
>> new file mode 100755
>> index 0000000..78ce47f
>> --- /dev/null
>> +++ b/record_traces.sh
>> @@ -0,0 +1,20 @@
>> +#!/bin/sh
>> +#
>> +# Record traces
>> +#
>> +# A risu helper script to batch process a bunch of binaries and record
>> their outputs
>> +#
>
> Can we have a license statement, copyright line and a
> usage summary, please? (Ditto patch 10). I'm not sure if
> these scripts are intended for testing purposes or as
> something that risu users might want to use -- if the
> latter then also documenting them in README would be
> a good idea.
They are more useful scripts I used for bulk generating stuff. Maybe I
should move them into contrib?
I'll update the usage and copyright lines.
>
>> +set -e
>> +
>> +if test -z "$RISU"; then
>> + script_dir=$(CDPATH= cd -- "$(dirname -- "$0")" && pwd -P)
>> + RISU=${script_dir}/risu
>> +fi
>> +
>> +for f in $@; do
>> + echo "Running risu against $f"
>> + t="$f.trace"
>> + ${RISU} --master $f -t $t
>> + echo "Checking trace file OK"
>> + ${RISU} $f -t $t
>> +done
>> --
>> 2.13.0
>
> thanks
> -- PMM
--
Alex Bennée