[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/18] hw: include hw header files with full pat
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 09/18] hw: include hw header files with full paths |
Date: |
Tue, 12 Mar 2013 09:22:58 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130219 Thunderbird/17.0.3 |
Il 12/03/2013 09:19, KONRAD Frédéric ha scritto:
> On 01/03/2013 14:33, Paolo Bonzini wrote:
>> Done with this script:
>>
>> cd hw
>> for i in `find . -name '*.h' | sed 's/^..//'`; do
>> echo '\,^#.*include.*["<]'$i'[">], s,'$i',hw/&,'
>> done | sed -i -f - `find . -type f`
>>
>> This is so that paths remain valid as files are moved.
>>
>> Instead, files in hw/dataplane are referenced with the relative path.
>> We know they are not going to move to include/, and they are the only
>> include files that are in subdirectories _and_ move.
>>
>> Signed-off-by: Paolo Bonzini <address@hidden>
> Hi Paolo,
>> diff --git a/hw/virtio-blk.c b/hw/virtio-blk.c
>> index 34913ee..248a966 100644
>> --- a/hw/virtio-blk.c
>> +++ b/hw/virtio-blk.c
>> @@ -16,11 +16,11 @@
>> #include "trace.h"
>> #include "hw/block-common.h"
>> #include "sysemu/blockdev.h"
>> -#include "virtio-blk.h"
>> +#include "hw/virtio-blk.h"
>> #ifdef CONFIG_VIRTIO_BLK_DATA_PLANE
>> -#include "hw/dataplane/virtio-blk.h"
>> +#include "dataplane/virtio-blk.h"
> Is that normal? ^^
> Or I missed something?
> I though you include hw header with full paths?
See the commit message: "files in hw/dataplane are referenced with the
relative path. We know they are not going to move to include/, and they
are the only include files that are in subdirectories _and_ move".
Paolo
- [Qemu-devel] [PATCH 13/18] arm: move files referencing CPU to hw/arm/, (continued)
- [Qemu-devel] [PATCH 13/18] arm: move files referencing CPU to hw/arm/, Paolo Bonzini, 2013/03/01
- [Qemu-devel] [PATCH 14/18] i386: move files referencing CPU to hw/i386/, Paolo Bonzini, 2013/03/01
- [Qemu-devel] [PATCH 12/18] hw: move boards and other isolated files to hw/ARCH, Paolo Bonzini, 2013/03/01
- [Qemu-devel] [PATCH 15/18] m68k: move files referencing CPU to hw/m68k/, Paolo Bonzini, 2013/03/01
- [Qemu-devel] [PATCH 16/18] ppc: move files referencing CPU to hw/ppc/, Paolo Bonzini, 2013/03/01
- [Qemu-devel] [PATCH 17/18] ppc: move more files to hw/ppc, Paolo Bonzini, 2013/03/01
- [Qemu-devel] [PATCH 18/18] sh: move files referencing CPU to hw/sh4/, Paolo Bonzini, 2013/03/01
- [Qemu-devel] [PATCH 09/18] hw: include hw header files with full paths, Paolo Bonzini, 2013/03/01