Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Lwt client new #958

Merged
merged 2 commits into from Jan 9, 2013

Conversation

Projects
None yet
2 participants
Owner

jonludlam commented Jan 5, 2013

Functorise the autogenned client module, and a fix for the new jsonrpc module

jonludlam added some commits Jan 5, 2013

Functorise the client module so it can be used with Lwt
Signed-off-by: Jon Ludlam <jonathan.ludlam@eu.citrix.com>
Fix jsonrpc so that it marshalls the reponse in json rather than xml
Signed-off-by: Jon Ludlam <jonathan.ludlam@eu.citrix.com>
Collaborator

djs55 commented Jan 5, 2013

Good stuff...

On Saturday, January 5, 2013, Jon Ludlam wrote:

Functorise the autogenned client module, and a fix for the new jsonrpc

module

You can merge this Pull Request by running

git pull https://github.com/jonludlam/xen-api lwt-client-new

Or view, comment on, or merge it at:

xen-org#958
Commit Summary

  • Functorise the client module so it can be used with Lwt
  • Fix jsonrpc so that it marshalls the reponse in json rather than xml

File Changes

  • M ocaml/idl/ocaml_backend/gen_api.ml (5)
  • M ocaml/idl/ocaml_backend/gen_client.ml (16)
  • M ocaml/xapi/api_server.ml (2)

Patch Links:

Dave Scott

jonludlam added a commit that referenced this pull request Jan 9, 2013

@jonludlam jonludlam merged commit 5c282a1 into xapi-project:master Jan 9, 2013

1 check passed

default Merged build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment