Patchwork QEMU: Update .gitignore

(2010/06/21 17:19), Avi Kivity wrote:
> On 06/21/2010 08:24 AM, Hidetoshi Seto wrote:
>> I think some people have noticed that:
>>
>>> $ ./configure
>>> $ make
>>> $ git status
>>> # On branch master
>>> # Untracked files:
>>> #   (use "git add <file>..." to include in what will be committed)
>>> #
>>> #       QMP/qmp-commands.txt
>>> #       libdis-user/
>>> #       libdis/
>>> #       pc-bios/optionrom/vapic.bin
>>> nothing added to commit but untracked files present (use "git add" to track)
>>>     
>> Please consider applying this patch to qemu-kvm.git.
> 
> This is equally applicable to qemu.git, so please sent it to the qemu
> mailing list, qemu-devel@nongnu.org.

Thanks for your advice, Avi.

Now this mail is sent to qemu ML, w/ above quotes as short history.
Could someone pick this up?

Thanks,
H.Seto

-- 1.7.0

On Mon, Jun 21, 2010 at 06:14:17PM +0900, Hidetoshi Seto wrote:
> (2010/06/21 17:19), Avi Kivity wrote:
> > On 06/21/2010 08:24 AM, Hidetoshi Seto wrote:
> >> I think some people have noticed that:
> >>
> >>> $ ./configure
> >>> $ make
> >>> $ git status
> >>> # On branch master
> >>> # Untracked files:
> >>> #   (use "git add <file>..." to include in what will be committed)
> >>> #
> >>> #       QMP/qmp-commands.txt
> >>> #       libdis-user/
> >>> #       libdis/
> >>> #       pc-bios/optionrom/vapic.bin
> >>> nothing added to commit but untracked files present (use "git add" to track)
> >>>     
> >> Please consider applying this patch to qemu-kvm.git.
> > 
> > This is equally applicable to qemu.git, so please sent it to the qemu
> > mailing list, qemu-devel@nongnu.org.
> 
> Thanks for your advice, Avi.
> 
> Now this mail is sent to qemu ML, w/ above quotes as short history.
> Could someone pick this up?
> 
> Thanks,
> H.Seto
> 
> =====

This patch looks good, but doesn't apply anymore, as the libdis/ and
libdis-user/ part has already been applied from another patch. Care
to resend it?

> Subject: [PATCH] QEMU: Update .gitignore
> 
> Add some files/directories to .gitignore
> 
>   - vapic.bin
>       A generated binary file.
>   - libdis/ and libdis-user/
>       These are directories generated by ./configure.
>   - QMP/qmp-commands.txt
>       A generated text.
> 
> Signed-off-by: Hidetoshi Seto <seto.hidetoshi@jp.fujitsu.com>
> ---
>  .gitignore |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/.gitignore b/.gitignore
> index 2d7f439..fa4f241 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -9,6 +9,8 @@ config-target.*
>  libhw32
>  libhw64
>  libuser
> +libdis
> +libdis-user
>  qemu-doc.html
>  qemu-tech.html
>  qemu-doc.info
> @@ -26,6 +28,7 @@ qemu-img-cmds.texi
>  qemu-img-cmds.h
>  qemu-io
>  qemu-monitor.texi
> +QMP/qmp-commands.txt
>  .gdbinit
>  *.a
>  *.aux
> @@ -50,4 +53,5 @@ pc-bios/optionrom/linuxboot.bin
>  pc-bios/optionrom/multiboot.bin
>  pc-bios/optionrom/multiboot.raw
>  pc-bios/optionrom/extboot.bin
> +pc-bios/optionrom/vapic.bin
>  .stgit-*
> -- 1.7.0 
> 
> 
>

(2010/07/01 6:33), Aurelien Jarno wrote:
> On Mon, Jun 21, 2010 at 06:14:17PM +0900, Hidetoshi Seto wrote:
>> (2010/06/21 17:19), Avi Kivity wrote:
>>> On 06/21/2010 08:24 AM, Hidetoshi Seto wrote:
>>>> I think some people have noticed that:
>>>>
>>>>> $ ./configure
>>>>> $ make
>>>>> $ git status
>>>>> # On branch master
>>>>> # Untracked files:
>>>>> #   (use "git add <file>..." to include in what will be committed)
>>>>> #
>>>>> #       QMP/qmp-commands.txt
>>>>> #       libdis-user/
>>>>> #       libdis/
>>>>> #       pc-bios/optionrom/vapic.bin
>>>>> nothing added to commit but untracked files present (use "git add" to track)
>>>>>     
>>>> Please consider applying this patch to qemu-kvm.git.
>>>
>>> This is equally applicable to qemu.git, so please sent it to the qemu
>>> mailing list, qemu-devel@nongnu.org.
>>
>> Thanks for your advice, Avi.
>>
>> Now this mail is sent to qemu ML, w/ above quotes as short history.
>> Could someone pick this up?
>>
>> Thanks,
>> H.Seto
>>
>> =====
> 
> This patch looks good, but doesn't apply anymore, as the libdis/ and
> libdis-user/ part has already been applied from another patch. Care
> to resend it?

I've just posted.  Please check it.

Thanks.
H.Seto