[BRLTTY] Patch: support for OCaml-4.04

Jaroslav Skarvada jskarvad at redhat.com
Tue Nov 8 12:12:05 EST 2016



----- Original Message -----
> Jaroslav Skarvada (2016/11/08 11:19 -0500):
> > 
> > 
> > ----- Original Message -----
> > > [quoted lines by Jaroslav Skarvada on 2016/11/08 at 07:58 -0500]
> > > 
> > > >it seems that brltty-5.4 doesn't compile with OCaml-4.04.
> > > >Tthe code uses CAMLlocal without calling CAMLparam which is against
> > > >current OCaml documentation. More details in Fedora bug [1]. Attached
> > > >patch should fix the problem
> > > 
> > > This gives me the following two warnings. How can they be resolved?
> > > 
> > > ./brlapi_stubs.c: In function ‘raise_brlapi_error’:
> > > ./brlapi_stubs.c:147:29: warning: unused variable ‘caml__frame’
> > > [-Wunused-variable]
> > > ./brlapi_stubs.c: In function ‘raise_brlapi_exception’:
> > > ./brlapi_stubs.c:161:29: warning: unused variable ‘caml__frame’
> > > [-Wunused-variable]
> > > 
> > 
> > Which version of OCaml? I haven't encountered these warnings with
> > OCaml-4.04.
> > To be honest I don't know much about OCaml, but this looks like OCaml bug,
> > I can forward it to OCaml maintainer
> 
> No no it's not a bug. See https://github.com/brltty/brltty/pull/81
> 
Shérab: Sorry, could you explain it more?

Dave: which version of gcc are you using?


More information about the BRLTTY mailing list