Re: [PATCH 1/8] Small fixes on rpc generator scripts

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

 



On 07/13/2011 04:19 AM, Osier Yang wrote:
> 1. Fix typos caused by the renaming
> 2. It should substitute "remote/remote_protocol.h" to "remote_protocol.h".
> ---
>  src/rpc/gendispatch.pl |    6 +++---
>  src/rpc/genprotocol.pl |    2 +-
>  2 files changed, 4 insertions(+), 4 deletions(-)

ACK.

> +++ b/src/rpc/genprotocol.pl
> @@ -59,7 +59,7 @@ while (<RPCGEN>) {
>      s/xdr_u_quad_t/xdr_uint64_t/g;
>      s/xdr_quad_t/xdr_int64_t/g;
>      s/(?<!IXDR_GET_INT32 )IXDR_GET_LONG/IXDR_GET_INT32/g;
> -    s,#include "\./remote/remote_protocol\.h",#include "remote_protocol.h",;
> +    s,#include "remote/remote_protocol\.h",#include "remote_protocol.h",;

I wonder if we could even go one step further in using:

s,#include ".*remote/remote_protocol\.h",#include "remote_protocol.h",;

but as written, this was okay, and things compiled fine both before and
after this change.

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
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]