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

Re: [Xen-devel] [PATCH] oxenstored: allow compilation prior to OCaml 3.12.0



On Wed, Mar 30, 2016 at 03:47:34PM +0000, Jonathan Davies wrote:
> Commit 363ae55c8 used an OCaml feature called record field punning. This broke
> the build on compilers prior to OCaml 3.12.0.
> 
> This patch makes no semantic change but now uses backwards-compatible syntax.
> 
> Signed-off-by: Jonathan Davies <jonathan.davies@xxxxxxxxxx>
> Reviewed-by: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>

This should be
  Tested-by: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>

Other than that I don't really have comment on this patch. :-)

> ---
>  tools/ocaml/xenstored/process.ml |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/ocaml/xenstored/process.ml 
> b/tools/ocaml/xenstored/process.ml
> index fb5fdaf..7b60376 100644
> --- a/tools/ocaml/xenstored/process.ml
> +++ b/tools/ocaml/xenstored/process.ml
> @@ -484,7 +484,7 @@ let do_input store cons doms con =
>       if newpacket then (
>               let packet = Connection.pop_in con in
>               let tid, rid, ty, data = Xenbus.Xb.Packet.unpack packet in
> -             let req = {Packet.tid; Packet.rid; Packet.ty; Packet.data} in
> +             let req = {Packet.tid=tid; Packet.rid=rid; Packet.ty=ty; 
> Packet.data=data} in
>  
>               (* As we don't log IO, do not call an unnecessary sanitize_data 
>                  info "[%s] -> [%d] %s \"%s\""
> -- 
> 1.7.10.4
> 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.