On Tue, Feb 11, 2020 at 03:34:48PM -0500, Alexander Bulekov wrote:
> 524b4c2c5c moves vl.c into softmmu/ , breaking the checkpatch
524b4c2c5c is a local git sha1. That commit will have a different sha1
when applied to qemu.git/master. Saying "The next patch" instead would
be fine.
However, this patch leaves the tree in a state where checkpatch.pl will
fail because softmmu/ doesn't exist yet! Please squash this patch into
the next commit instead.
I guess you kept it separate because changing checkpatch.pl can be
thought of as a separate change. However, they two need to happen in a
single step in order for checkpatch.pl to function correctly at each
commit. Therefore it's appropriate to combine them into a single
commit.
> top-of-kernel-tree check. Replace with checks for softmmu and linux-user
>
> Signed-off-by: Alexander Bulekov <alxndr@bu.edu>
> ---
> scripts/checkpatch.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index ce43a306f8..2e2273b8a3 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -462,7 +462,7 @@ sub top_of_kernel_tree {
> my @tree_check = (
> "COPYING", "MAINTAINERS", "Makefile",
> "README.rst", "docs", "VERSION",
> - "vl.c"
> + "softmmu", "linux-user"
> );
>
> foreach my $check (@tree_check) {
> --
> 2.25.0
>