Re: [libvirt] [PATCH] util : fix compile error on fedora14 Add src/util/virnetdevbandwidth.h, src/util/virnetdevveth.h, src/util/virnetdevvportprofile.h to private symbols.

2011-11-16 Thread Daniel P. Berrange
On Wed, Nov 16, 2011 at 09:36:06AM +0800, ta...@linux.vnet.ibm.com wrote:
> From: Eli Qiao 
> 
> 
> Signed-off-by: Eli Qiao 
> ---
>  src/libvirt_private.syms |   19 +++
>  1 files changed, 19 insertions(+), 0 deletions(-)
> 
> diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
> index 7b96a3c..f178662 100644
> --- a/src/libvirt_private.syms
> +++ b/src/libvirt_private.syms
> @@ -1201,9 +1201,19 @@ virNetDevSetIPv4Address;
>  virNetDevSetMAC;
>  virNetDevSetMTU;
>  virNetDevSetMTUFromDevice;
> +virNetDevSetName;
> +virNetDevSetNamespace;
>  virNetDevSetOnline;
>  
>  
> +# virnetdevbandwidth.h
> +virNetDevBandwidthClear;
> +virNetDevBandwidthCopy;
> +virNetDevBandwidthEqual;
> +virNetDevBandwidthFree;
> +virNetDevBandwidthSet;
> +
> +
>  # virnetdevbridge.h
>  virNetDevBridgeAddPort;
>  virNetDevBridgeCreate;
> @@ -1221,6 +1231,15 @@ virNetDevTapCreateInBridgePort;
>  virNetDevTapDelete;
>  
>  
> +# virnetdevveth.h
> +virNetDevVethCreate;
> +virNetDevVethDelete;
> +
> +
> +# virnetdevvportprofile.h 
> +virNetDevVPortProfileEqual;
> +
> +
>  # virnetmessage.h
>  virNetMessageClear;
>  virNetMessageDecodeNumFDs;

ACK

Daniel
-- 
|: http://berrange.com  -o-http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org  -o- http://virt-manager.org :|
|: http://autobuild.org   -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org   -o-   http://live.gnome.org/gtk-vnc :|

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


Re: [libvirt] [PATCH] util : fix compile error on fedora14 Add src/util/virnetdevbandwidth.h, src/util/virnetdevveth.h, src/util/virnetdevvportprofile.h to private symbols.

2011-11-15 Thread Eli Qiao



于 2011年11月16日 11:02, Osier Yang 写道:

于 2011年11月16日 09:36, ta...@linux.vnet.ibm.com 写道:

From: Eli Qiao


Signed-off-by: Eli Qiao
---
  src/libvirt_private.syms |   19 +++
  1 files changed, 19 insertions(+), 0 deletions(-)

diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
index 7b96a3c..f178662 100644
--- a/src/libvirt_private.syms
+++ b/src/libvirt_private.syms
@@ -1201,9 +1201,19 @@ virNetDevSetIPv4Address;
  virNetDevSetMAC;
  virNetDevSetMTU;
  virNetDevSetMTUFromDevice;
+virNetDevSetName;
+virNetDevSetNamespace;
  virNetDevSetOnline;


+# virnetdevbandwidth.h
+virNetDevBandwidthClear;
+virNetDevBandwidthCopy;
+virNetDevBandwidthEqual;
+virNetDevBandwidthFree;
+virNetDevBandwidthSet;
+
+
  # virnetdevbridge.h
  virNetDevBridgeAddPort;
  virNetDevBridgeCreate;
@@ -1221,6 +1231,15 @@ virNetDevTapCreateInBridgePort;
  virNetDevTapDelete;


+# virnetdevveth.h
+virNetDevVethCreate;
+virNetDevVethDelete;
+
+
+# virnetdevvportprofile.h
+virNetDevVPortProfileEqual;
+
+
  # virnetmessage.h
  virNetMessageClear;
  virNetMessageDecodeNumFDs;


Straightforward, ACK, but there is a trailing whitespace and the
patch subject is too long, pushed with the trainling whitespace
removed and patch subject shortened as:


thanks Osier.

util: Add netdev helper functions to private symbols

Thanks
Osier



--
best regards
eli

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

Re: [libvirt] [PATCH] util : fix compile error on fedora14 Add src/util/virnetdevbandwidth.h, src/util/virnetdevveth.h, src/util/virnetdevvportprofile.h to private symbols.

2011-11-15 Thread Osier Yang

于 2011年11月16日 09:36, ta...@linux.vnet.ibm.com 写道:

From: Eli Qiao


Signed-off-by: Eli Qiao
---
  src/libvirt_private.syms |   19 +++
  1 files changed, 19 insertions(+), 0 deletions(-)

diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
index 7b96a3c..f178662 100644
--- a/src/libvirt_private.syms
+++ b/src/libvirt_private.syms
@@ -1201,9 +1201,19 @@ virNetDevSetIPv4Address;
  virNetDevSetMAC;
  virNetDevSetMTU;
  virNetDevSetMTUFromDevice;
+virNetDevSetName;
+virNetDevSetNamespace;
  virNetDevSetOnline;


+# virnetdevbandwidth.h
+virNetDevBandwidthClear;
+virNetDevBandwidthCopy;
+virNetDevBandwidthEqual;
+virNetDevBandwidthFree;
+virNetDevBandwidthSet;
+
+
  # virnetdevbridge.h
  virNetDevBridgeAddPort;
  virNetDevBridgeCreate;
@@ -1221,6 +1231,15 @@ virNetDevTapCreateInBridgePort;
  virNetDevTapDelete;


+# virnetdevveth.h
+virNetDevVethCreate;
+virNetDevVethDelete;
+
+
+# virnetdevvportprofile.h
+virNetDevVPortProfileEqual;
+
+
  # virnetmessage.h
  virNetMessageClear;
  virNetMessageDecodeNumFDs;


Straightforward, ACK, but there is a trailing whitespace and the
patch subject is too long, pushed with the trainling whitespace
removed and patch subject shortened as:

util: Add netdev helper functions to private symbols

Thanks
Osier

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

Re: [libvirt] [PATCH] util : fix compile error on fedora14 Add src/util/virnetdevbandwidth.h, src/util/virnetdevveth.h, src/util/virnetdevvportprofile.h to private symbols.

2011-11-15 Thread Hong Xiang
I got the compile error too on RHEL workstation 6.1 with latest git
source, and this patch fixes it.

On 11/16/2011 09:36 AM, ta...@linux.vnet.ibm.com wrote:
> From: Eli Qiao 
> 
> 
> Signed-off-by: Eli Qiao 
> ---
>  src/libvirt_private.syms |   19 +++
>  1 files changed, 19 insertions(+), 0 deletions(-)
> 
> diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
> index 7b96a3c..f178662 100644
> --- a/src/libvirt_private.syms
> +++ b/src/libvirt_private.syms
> @@ -1201,9 +1201,19 @@ virNetDevSetIPv4Address;
>  virNetDevSetMAC;
>  virNetDevSetMTU;
>  virNetDevSetMTUFromDevice;
> +virNetDevSetName;
> +virNetDevSetNamespace;
>  virNetDevSetOnline;
> 
> 
> +# virnetdevbandwidth.h
> +virNetDevBandwidthClear;
> +virNetDevBandwidthCopy;
> +virNetDevBandwidthEqual;
> +virNetDevBandwidthFree;
> +virNetDevBandwidthSet;
> +
> +
>  # virnetdevbridge.h
>  virNetDevBridgeAddPort;
>  virNetDevBridgeCreate;
> @@ -1221,6 +1231,15 @@ virNetDevTapCreateInBridgePort;
>  virNetDevTapDelete;
> 
> 
> +# virnetdevveth.h
> +virNetDevVethCreate;
> +virNetDevVethDelete;
> +
> +
> +# virnetdevvportprofile.h 
> +virNetDevVPortProfileEqual;
> +
> +
>  # virnetmessage.h
>  virNetMessageClear;
>  virNetMessageDecodeNumFDs;

-- 
Thanks.
Hong Xiang

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


[libvirt] [PATCH] util : fix compile error on fedora14 Add src/util/virnetdevbandwidth.h, src/util/virnetdevveth.h, src/util/virnetdevvportprofile.h to private symbols.

2011-11-15 Thread taget
From: Eli Qiao 


Signed-off-by: Eli Qiao 
---
 src/libvirt_private.syms |   19 +++
 1 files changed, 19 insertions(+), 0 deletions(-)

diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
index 7b96a3c..f178662 100644
--- a/src/libvirt_private.syms
+++ b/src/libvirt_private.syms
@@ -1201,9 +1201,19 @@ virNetDevSetIPv4Address;
 virNetDevSetMAC;
 virNetDevSetMTU;
 virNetDevSetMTUFromDevice;
+virNetDevSetName;
+virNetDevSetNamespace;
 virNetDevSetOnline;
 
 
+# virnetdevbandwidth.h
+virNetDevBandwidthClear;
+virNetDevBandwidthCopy;
+virNetDevBandwidthEqual;
+virNetDevBandwidthFree;
+virNetDevBandwidthSet;
+
+
 # virnetdevbridge.h
 virNetDevBridgeAddPort;
 virNetDevBridgeCreate;
@@ -1221,6 +1231,15 @@ virNetDevTapCreateInBridgePort;
 virNetDevTapDelete;
 
 
+# virnetdevveth.h
+virNetDevVethCreate;
+virNetDevVethDelete;
+
+
+# virnetdevvportprofile.h 
+virNetDevVPortProfileEqual;
+
+
 # virnetmessage.h
 virNetMessageClear;
 virNetMessageDecodeNumFDs;
-- 
1.7.4.4

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