[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 0/6] Initial support for keycodemapdb GIT sub
From: |
Programmingkid |
Subject: |
Re: [Qemu-devel] [PATCH v7 0/6] Initial support for keycodemapdb GIT submodule |
Date: |
Thu, 28 Sep 2017 20:20:09 -0400 |
> On Sep 28, 2017, at 12:53 PM, Daniel P. Berrange <address@hidden> 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.
- Re: [Qemu-devel] [PATCH v7 2/6] docker: don't rely on submodules existing in the main checkout, (continued)
- [Qemu-devel] [PATCH v7 3/6] ui: add keycodemapdb repository as a GIT submodule, Daniel P. Berrange, 2017/09/28
- [Qemu-devel] [PATCH v7 5/6] ui: convert key events to QKeyCodes immediately, Daniel P. Berrange, 2017/09/28
- [Qemu-devel] [PATCH v7 6/6] ui: don't export qemu_input_event_new_key, Daniel P. Berrange, 2017/09/28
- [Qemu-devel] [PATCH v7 4/6] ui: convert common input code to keycodemapdb, Daniel P. Berrange, 2017/09/28
- Re: [Qemu-devel] [PATCH v7 0/6] Initial support for keycodemapdb GIT submodule, no-reply, 2017/09/28
- [Qemu-devel] [PATCH v7 0/6] Initial support for keycodemapdb GIT submodule, Programmingkid, 2017/09/28