Re: [PATCH] perf tools: Fix check-headers.sh AND list path of execution

2018-08-13 Thread Alexander Kapshuk
On Mon, Aug 13, 2018 at 9:58 PM Arnaldo Carvalho de Melo wrote: > Thanks, applied the three patches to acme/perf/core. > > - Arnaldo Thanks.

Re: [PATCH] perf tools: Fix check-headers.sh AND list path of execution

2018-08-13 Thread Arnaldo Carvalho de Melo
Em Mon, Aug 13, 2018 at 02:19:10PM +0300, Alexander Kapshuk escreveu: > On Mon, Aug 13, 2018 at 2:16 PM Jiri Olsa wrote: > > > > On Sat, Aug 11, 2018 at 11:39:15AM +0300, Alexander Kapshuk wrote: > > > The '||' path of execution in the 'test' block of the check_2() function > > > may also be taken

Re: [PATCH] perf tools: Fix check-headers.sh AND list path of execution

2018-08-13 Thread Alexander Kapshuk
On Mon, Aug 13, 2018 at 2:16 PM Jiri Olsa wrote: > > On Sat, Aug 11, 2018 at 11:39:15AM +0300, Alexander Kapshuk wrote: > > The '||' path of execution in the 'test' block of the check_2() function > > may also be taken if file2 does not exist, in which case the warning > > message about the ABI he

Re: [PATCH] perf tools: Fix check-headers.sh AND list path of execution

2018-08-13 Thread Jiri Olsa
On Sat, Aug 11, 2018 at 11:39:15AM +0300, Alexander Kapshuk wrote: > The '||' path of execution in the 'test' block of the check_2() function > may also be taken if file2 does not exist, in which case the warning > message about the ABI headers being different would still be printed > where it shou

[PATCH] perf tools: Fix check-headers.sh AND list path of execution

2018-08-11 Thread Alexander Kapshuk
The '||' path of execution in the 'test' block of the check_2() function may also be taken if file2 does not exist, in which case the warning message about the ABI headers being different would still be printed where it should not be. See below. % file1=file1; file2=file2 % cmd="echo diff $file1