Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

opam package? #4

Open
vzaliva opened this issue Jul 14, 2019 · 6 comments
Open

opam package? #4

vzaliva opened this issue Jul 14, 2019 · 6 comments

Comments

@vzaliva
Copy link

vzaliva commented Jul 14, 2019

It would be nice if this library is installable via OPAM

@fccm
Copy link
Owner

fccm commented Jul 19, 2019

Why?

@vzaliva
Copy link
Author

vzaliva commented Jul 21, 2019

convenience mostly. I wanted to use it in a small project. But it looks like it is out of date and does not compile with recent versions of OCaml. In OPAM it could depend on older OCaml version which would make it possible to install it.

@fccm
Copy link
Owner

fccm commented Jul 21, 2019

Have you tryed to compile this fork from 2017:
#2
?
The description says: "Fixes for ocaml >= 4.04"
So it would maybe resolve your problem to install it with recent version of OCaml.

@fccm
Copy link
Owner

fccm commented Dec 1, 2020

Hi @vzaliva,
I've just added an opam file!
So now you can install this code repository with:

opam install .

@vzaliva
Copy link
Author

vzaliva commented Dec 7, 2020

Thanks. You may also consider submitting it to OPAM repository.

@fccm2
Copy link

fccm2 commented Oct 13, 2024

If I submit it to the opam-repository, I'm worry that some people play with it, and don't realise that they can crash their x-server with it, and loose the files that they haven't saved yet. (I forgot to add a warning about it.)

This package still compiles on my computer with one of the last ocaml4 versions. The API of ocaml5 is maybe different, I don't know, because I haven't try to use 5 yet, but if you use opam, you can easily use a switch with one of the last 4 version.

I compiled this bindings yesterday, it still does compile, with some warnings. I had to add "[@@boxed]" at the end of "xKeymapEvent_contents" and "xVisibilityEvent_contents" type definitions to avoid some of the warnings. I haven't started to learn the new api of ocaml5, so I don't know the differences yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants