Re: [PATCHv2 8/8] Remove bogus includes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Nov 23, 2010 at 04:50:01PM -0700, Eric Blake wrote:
> From: Daniel P. Berrange <berrange@xxxxxxxxxx>
> 
> ---
> 
> v2: rearrange to later in the series; no other change.  Passes
> for me with macvtap compilation enabled, so I'm not sure if it
> still suffers from the same problem as the v1 complaint:
> https://www.redhat.com/archives/libvir-list/2010-November/msg00834.html
> 
>  src/conf/domain_conf.c |    1 -
>  src/util/hooks.c       |    1 -
>  2 files changed, 0 insertions(+), 2 deletions(-)

ACK

The problem I hit was actually with removing

diff --git a/src/util/macvtap.c b/src/util/macvtap.c
index 5dcc9e1..eb4ea8f 100644
--- a/src/util/macvtap.c
+++ b/src/util/macvtap.c
@@ -49,7 +49,6 @@
 # include "logging.h"
 # include "macvtap.h"
 # include "interface.h"
-# include "conf/domain_conf.h"


Because the 'util' directory must never depend on the 'conf' directory.

Daniel

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]