> On Sep 28, 2017, at 12:53 PM, Daniel P. Berrange <[email protected]> wrote: > > On Thu, Sep 28, 2017 at 12:47:51PM -0400, Programmingkid wrote: >> >>> This patch series is an update to: >>> >>> v1: https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg02047.html >>> v2: https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg02471.html >>> v3: https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg02517.html >>> v4: https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg02708.html >>> v5: https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg02950.html >>> v6: https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg07673.html >> >> <snip> >> >> These error messages appears when compiling version 7 of this patch series: >> >> /bin/sh: ./scripts/git-submodule.sh: Permission denied > > The git diff sets execute permission: > > diff --git a/scripts/git-submodule.sh b/scripts/git-submodule.sh > new file mode 100755 > index 0000000000..f8e7d44ce6 > --- /dev/null > +++ b/scripts/git-submodule.sh > > did you use "git am" to apply this series or just plain 'patch', because > the latter won't deal with permissions correctly and so would cause that > error message above.
I used the patch command. After trying again using 'git am' I was able to built QEMU. Even thought the build did succeed, this error message still appeared: config-host.mak is out-of-date, running configure GIT ui/keycodemapdb dtc make: *** No rule to make target `ui/input-keymap-linux-to-qcode.c', needed by `Makefile'. Stop. make: *** Waiting for unfinished jobs.... I suggest adding a note to patch 0/7 that states this patch set is best applied using 'git am' instead of the patch command. That should help someone with testing out this patch set.
