[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL v2 00/34] virtio, vhost, pc, pci: tests, document
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PULL v2 00/34] virtio, vhost, pc, pci: tests, documentation, fixes and cleanups |
Date: |
Fri, 18 Nov 2016 01:04:57 +0800 |
User-agent: |
Mutt/1.7.1 (2016-10-04) |
On Thu, 11/17 18:44, Michael S. Tsirkin wrote:
> On Thu, Nov 17, 2016 at 06:16:12AM -0800, address@hidden wrote:
> > ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
> > #75: FILE: contrib/libvhost-user/libvhost-user.c:9:
> > + * Marc-Andr Lureau <address@hidden>
> > ^
> >
> > ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
> > #1579: FILE: contrib/libvhost-user/libvhost-user.h:8:
> > + * Marc-Andr Lureau <address@hidden>
> > ^
> >
> > total: 2 errors, 0 warnings, 1950 lines checked
>
> Any idea what's going on here?
Because of how patchew works, the patch email in question went through "git am",
"git push" then "git fetch", and eventually it's the output of "git show
--format=email" that is fed to checkpatch.pl. Something went wrong with these
git commands but I don't know why. :(
To reproduce the last step (which looks the most suspicious to me, maybe you
have an idea), try this on your machine:
$ git fetch https://github.com/patchew-project/qemu \
patchew/address@hidden
From https://github.com/patchew-project/qemu
* tag patchew/address@hidden -> FETCH_HEAD
$ git show c19172474cd8da8375307b324344345ad7ad01ca --format=email \
| ./scripts/checkpatch.pl -
ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
#75: FILE: contrib/libvhost-user/libvhost-user.c:9:
+ * Marc-Andr� Lureau <address@hidden>
^
ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
#1579: FILE: contrib/libvhost-user/libvhost-user.h:8:
+ * Marc-Andr� Lureau <address@hidden>
^
total: 2 errors, 0 warnings, 1950 lines checked
Your patch has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
$
Fam
- [Qemu-devel] [PULL v2 30/34] nvdimm acpi: introduce NVDIMM_DSM_MEMORY_SIZE, (continued)
- [Qemu-devel] [PULL v2 30/34] nvdimm acpi: introduce NVDIMM_DSM_MEMORY_SIZE, Michael S. Tsirkin, 2016/11/11
- [Qemu-devel] [PULL v2 33/34] virtio: drop virtio_queue_get_ring_{size, addr}(), Michael S. Tsirkin, 2016/11/11
- [Qemu-devel] [PULL v2 31/34] vhost: adapt vhost_verify_ring_mappings() to virtio 1 ring layout, Michael S. Tsirkin, 2016/11/11
- [Qemu-devel] [PULL v2 32/34] vhost: drop legacy vring layout bits, Michael S. Tsirkin, 2016/11/11
- [Qemu-devel] [PULL v2 34/34] docs: add PCIe devices placement guidelines, Michael S. Tsirkin, 2016/11/11
- Re: [Qemu-devel] [PULL v2 00/34] virtio, vhost, pc, pci: tests, documentation, fixes and cleanups, Stefan Hajnoczi, 2016/11/14
- Re: [Qemu-devel] [PULL v2 00/34] virtio, vhost, pc, pci: tests, documentation, fixes and cleanups, no-reply, 2016/11/17
- Re: [Qemu-devel] [PULL v2 00/34] virtio, vhost, pc, pci: tests, documentation, fixes and cleanups, Michael S. Tsirkin, 2016/11/17
- Re: [Qemu-devel] [PULL v2 00/34] virtio, vhost, pc, pci: tests, documentation, fixes and cleanups,
Fam Zheng <=