Re: [PATCH] strace: update to 4.7

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Thu, 2012-06-14 at 11:57 -0700, Saul Wold wrote:
> On 06/12/2012 12:59 AM, Bogdan Marinescu wrote:
> > Removed the x32 specific patches, since they seem to be fully
> > integrated into 4.7. Also removed the sigmask patch, since the
> > new version doesn't seem to use sigmask anymore.
> >
> > Signed-off-by: Bogdan Marinescu<bogdan.a.marinescu@intel.com>
> > ---
> >   .../strace/strace-4.6/sigmask.patch                |   23 -
> >   .../strace-4.6/strace-4.6_add_x32_support.patch    | 1242 --------------------
> >   meta/recipes-devtools/strace/strace_4.6.bb         |   23 -
> >   meta/recipes-devtools/strace/strace_4.7.bb         |   20 +
> >   4 files changed, 20 insertions(+), 1288 deletions(-)
> >   delete mode 100644 meta/recipes-devtools/strace/strace-4.6/sigmask.patch
> >   delete mode 100644 meta/recipes-devtools/strace/strace-4.6/strace-4.6_add_x32_support.patch
> >   delete mode 100644 meta/recipes-devtools/strace/strace_4.6.bb
> >   create mode 100644 meta/recipes-devtools/strace/strace_4.7.bb
> >
> Did you do a git mv?  by removing and creating, the git history gets 
> lost, can you respin this?

This is an urban myth. git does not track renames, ever. Some of the
tools can detect and display things with "rename detection" though.

So the patch was just sent without rename detection, it will make no
difference to the history whatsoever if merged. It can be harder to
review though. Rename detection doesn't always work in the cases of
large delta, the exact threshold can be configured.

Cheers,

Richard