-
Notifications
You must be signed in to change notification settings - Fork 0
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
adding a "Lwt" module into Screen_orientation #1
base: master
Are you sure you want to change the base?
Conversation
ef27baf
to
e4ab5e2
Compare
a529cd1
to
cf9c181
Compare
J'ai tenté de me renseigner sur https://github.com/mnxn/promise_jsoo mais je n'ai pas réussi à m'en servir. J'ai l'impression d'avoir réussi à installer sans soucis "promise_jsoo" avec un "opam install promise_jsoo" mais cela n'a pas marché pour "promise_jsoo_lwt" qui est pourtant définit dans un ".opam" à part entière, et de toute manière je n'ai pas réussit à utiliser la moindre fonction de "promise_jsoo", j'ai simplement laissé mes expérimentations en commentaire. |
Visiblement, la bibliothèque |
J'ai tenté d'effectuer le pin, mais ça ne change rien au fait que tenter d'ajouter "promise_jsoo_lwt" au fichier "dune" ne passe pas la compilation: j'ai l'impresion que ma machine n'a toujours aucune librairie du nom de "promise_jsoo_lwt" d'installée dans son environnement local. |
screen-orientation.opam
Outdated
"promise_jsoo_lwt" | ||
] | ||
pin-depends: [ | ||
[ "promise_jsoo_lwt" "git+https://github.com/mnxn/promise_jsoo"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems that we need to explicitly specify a version:
[ "promise_jsoo_lwt" "git+https://github.com/mnxn/promise_jsoo"] | |
[ "promise_jsoo_lwt.dev" "git+https://github.com/mnxn/promise_jsoo"] |
screen-orientation.opam
Outdated
] | ||
"lwt_ppx" {>= "2.0.1"} | ||
"promise_jsoo" | ||
"promise_jsoo-ppx" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This package does not exist:
"promise_jsoo-ppx" |
Je réussit à installer la librairie sur mon PC en effectuant un "pin" manuellement, j'espère que cela suffira. J'ai essayé d'obtenir un résultat qui passe la compilation sur mon environnement personnel, je ne suis pas spécialement fière de ce que j'ai fais, j'ignore honnêtement si c'est vraiment ce qui étais attendu de l'implémentation des fonctions "then_" du module lwt... |
PS: je peux confirmer de part une mésaventure de compilation avec les pin, il semblerait que "promise_jsoo_lwt" est incompatible avec "bs", je ne peux apparament pas compiler "make byte" quand cette librairie est installée sur mon ordinateur. J'espère que ça ne posera pas de soucis dans l’utilisation du module avec l'application. |
No description provided.