All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laurent Vivier <laurent@vivier.eu>
To: Wei Yang <richardw.yang@linux.intel.com>
Cc: "Philippe Mathieu-Daudé" <philmd@redhat.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	qemu-trivial@nongnu.org, qemu-devel@nongnu.org,
	imammedo@redhat.com
Subject: Re: [Qemu-devel] [PATCH v3 0/3] Trivial cleanup in hw/acpi
Date: Thu, 14 Feb 2019 09:10:14 +0100	[thread overview]
Message-ID: <d69dd1c9-aa58-4b03-6b9c-1cde7bed83cb@vivier.eu> (raw)
In-Reply-To: <3dd545f2-5346-2b01-8725-5f35cbaa0ae5@vivier.eu>

On 14/02/2019 08:57, Laurent Vivier wrote:
> On 14/02/2019 01:03, Wei Yang wrote:
>> On Wed, Feb 13, 2019 at 02:45:01PM +0100, Laurent Vivier wrote:
>>> On 13/02/2019 09:56, Philippe Mathieu-Daudé wrote:
>>>> Hi Laurent,
>>>>
>>>> On 2/13/19 9:49 AM, Laurent Vivier wrote:
>>>>> On 12/02/2019 06:34, Michael S. Tsirkin wrote:
>>>>>> On Tue, Feb 12, 2019 at 01:22:24PM +0800, Wei Yang wrote:
>>>>>>> On Wed, Jan 30, 2019 at 08:06:50AM +0800, Wei Yang wrote:
>>>>>>>> There are several functions/variable which are not used anymore.
>>>>>>>>
>>>>>>>> This serials just remove those without functional change.
>>>>>>>>
>>>>>>>> v3: add ack and repost in a new thread
>>>>>>>> v2: change commit log from "is now used in no place" to "in not used anymore"
>>>>>>>
>>>>>>> Michael,
>>>>>>>
>>>>>>> Looks this serials is not merged yet.
>>>>>>>
>>>>>>> Is there any problem I need to fix?
>>>>>>
>>>>>> Yes pls repost with fixed reviewed-by tags.
>>>>>
>>>>> Or I can take the series through the trivial branch and update manually
>>>>> Philippe's lastname?
>>>>
>>>> I appreciate the help with manual update, however Wei is not the only
>>>> one having this problem (other developpers has non-ASCII in their name,
>>>> and use such tags).
>>>> I'd rather prefer we find a proper configuration setup to help the
>>>> community.
>>>
>>> I'm wondering if it can be a problem with cut'n'paste rather than with
>>> git-sendemail.
>>>
>>> I think the following tags should cover Latin and Chinese characters:
>>>
>>> Content-Type: text/plain; charset=UTF-8
>>> Content-Transfer-Encoding: 8bit
>>>
>>
>> Looks not work.
>>
>> I have tried this and also with 
>>
>> Content-Language: en-US
> 
> Could  you send me a gzipped version of the 0001-XXX.patch in attachment
> so I could see the real content before git-send-email?

OK, I've got it.

What I see is it is already corrupted, thus the problem is not with
git-send-email.

So either it is corrupted by git-format-patch (which is very unlikely,
we could see that if you push it to a public git repo) or it has been
corrupted when you have added it to the commit message.

Thanks,
Laurent

  reply	other threads:[~2019-02-14  8:10 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-30  0:06 [Qemu-devel] [PATCH v3 0/3] Trivial cleanup in hw/acpi Wei Yang
2019-01-30  0:06 ` [Qemu-devel] [PATCH v3 1/3] hw/i386/pc.c: remove unused function pc_acpi_init() Wei Yang
2019-01-30  8:10   ` Igor Mammedov
2019-01-30  9:50   ` Thomas Huth
2019-01-30 11:07     ` Philippe Mathieu-Daudé
2019-01-30 12:58       ` Wei Yang
2019-01-30  0:06 ` [Qemu-devel] [PATCH v3 2/3] hw/acpi: remove unused function acpi_table_add_builtin() Wei Yang
2019-01-30  8:11   ` Igor Mammedov
2019-01-30  0:06 ` [Qemu-devel] [PATCH v3 3/3] hw/acpi: remove unnecessary variable acpi_table_builtin Wei Yang
2019-01-30  8:11   ` Igor Mammedov
2019-02-12  5:22 ` [Qemu-devel] [PATCH v3 0/3] Trivial cleanup in hw/acpi Wei Yang
2019-02-12  5:34   ` Michael S. Tsirkin
2019-02-13  6:26     ` Wei Yang
2019-02-13  6:32       ` Philippe Mathieu-Daudé
2019-02-13  7:01         ` Wei Yang
2019-02-13  8:49     ` Laurent Vivier
2019-02-13  8:54       ` Wei Yang
2019-02-13  8:56       ` Philippe Mathieu-Daudé
2019-02-13 13:31         ` Wei Yang
2019-02-13 13:45         ` Laurent Vivier
2019-02-14  0:03           ` Wei Yang
2019-02-14  7:57             ` Laurent Vivier
2019-02-14  8:10               ` Laurent Vivier [this message]
2019-02-14  8:18                 ` Wei Yang
2019-02-14  8:23                   ` Laurent Vivier
2019-02-14  8:39                     ` Wei Yang
2019-02-14  8:51                     ` Wei Yang
2019-02-14  9:11                       ` Laurent Vivier

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=d69dd1c9-aa58-4b03-6b9c-1cde7bed83cb@vivier.eu \
    --to=laurent@vivier.eu \
    --cc=imammedo@redhat.com \
    --cc=mst@redhat.com \
    --cc=philmd@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-trivial@nongnu.org \
    --cc=richardw.yang@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.