From: Wei Liu <wei.liu2@citrix.com>
To: Michael Young <m.a.young@durham.ac.uk>
Cc: Wei Liu <wei.liu2@citrix.com>,
John Thomson <git@johnthomson.fastmail.com.au>,
Marcello Seri <marcello.seri@citrix.com>,
Christian Lindig <christian.lindig@citrix.com>,
David Scott <dave@recoil.org>,
xen-devel@lists.xenproject.org
Subject: Re: [PATCH 1/2] make xen ocaml safe-strings compliant
Date: Wed, 7 Feb 2018 10:31:42 +0000 [thread overview]
Message-ID: <20180207103142.b7lbgqw7nwdtkiss@citrix.com> (raw)
In-Reply-To: <alpine.LFD.2.21.1802062144430.3326@austen3.home>
On Tue, Feb 06, 2018 at 09:56:14PM +0000, Michael Young wrote:
> On Tue, 6 Feb 2018, Wei Liu wrote:
>
> > On Tue, Jan 30, 2018 at 10:55:47PM +0000, Michael Young wrote:
> > > Xen built with ocaml 4.06 gives errors such as
> > > Error: This expression has type bytes but an expression was
> > > expected of type string
> > > as Byte and safe-strings which were introduced in 4.02 are the
> > > default in 4.06.
> > > This patch which is partly by Richard W.M. Jones of Red Hat
> > > from https://bugzilla.redhat.com/show_bug.cgi?id=1526703
> > > fixes these issues.
> > >
> > > Signed-off-by: Michael Young <m.a.young@durham.ac.uk>
> > > Reviewed-by: Christian Lindig <christian.lindig@citrix.com>
> >
> > Strangely this doesn't apply to staging. And after examining the
> > downloaded patch I'm not sure if my mail client is acting up. Do you
> > have a git branch that I can pull from?
>
> The patch needed to be reduced as one of the sections being patched was
> removed by a recent patch. I am attaching the revised patch as a file in
> case there was also an email formatting issue.
>
Thanks, I will try this today.
Wei.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
next prev parent reply other threads:[~2018-02-07 10:31 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-30 22:55 [PATCH 1/2] make xen ocaml safe-strings compliant Michael Young
2018-02-06 16:49 ` Wei Liu
2018-02-06 21:56 ` Michael Young
2018-02-07 10:31 ` Wei Liu [this message]
2018-02-08 17:49 ` Dario Faggioli
2018-02-08 18:03 ` Wei Liu
2018-02-08 18:24 ` Wei Liu
2018-02-09 9:20 ` Christian Lindig
2018-02-09 10:06 ` Dario Faggioli
2018-02-12 14:55 ` Wei Liu
2018-03-09 22:57 ` Michael Young
2018-03-09 23:47 ` Christian Lindig
2018-03-10 0:36 ` Michael Young
2018-03-12 11:29 ` Christian Lindig
2018-03-12 19:35 ` Michael Young
2018-03-13 9:29 ` Christian Lindig
2018-03-13 14:49 ` Wei Liu
2018-02-13 0:35 ` Michael Young
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180207103142.b7lbgqw7nwdtkiss@citrix.com \
--to=wei.liu2@citrix.com \
--cc=christian.lindig@citrix.com \
--cc=dave@recoil.org \
--cc=git@johnthomson.fastmail.com.au \
--cc=m.a.young@durham.ac.uk \
--cc=marcello.seri@citrix.com \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).