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

adding a "Lwt" module into Screen_orientation #1

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Thibaut-Gudin
Copy link
Contributor

No description provided.

@Thibaut-Gudin Thibaut-Gudin added the enhancement New feature or request label Jul 22, 2021
@Thibaut-Gudin Thibaut-Gudin self-assigned this Jul 22, 2021
@Thibaut-Gudin Thibaut-Gudin requested review from ilankri and jrochel July 23, 2021 10:46
@Thibaut-Gudin Thibaut-Gudin marked this pull request as draft July 23, 2021 10:46
@ilankri ilankri removed their request for review July 30, 2021 11:15
@Thibaut-Gudin
Copy link
Contributor Author

J'ai tenté de me renseigner sur https://github.com/mnxn/promise_jsoo mais je n'ai pas réussi à m'en servir.
Non seulement je ne suis pas sûr de voir où vous voulez en venir avec l'utilisation de cette librairie-ci (je n'ai pas compris quel étais le soucis avec ce que j'avais fait avant et pourquoi "Lwt" en lui même est insuffisant), mais je n'ai pas non plus réussi à installer le module opam "promise_jsoo_lwt" qui me semble tout adapté à la situation.

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.

@Thibaut-Gudin Thibaut-Gudin marked this pull request as ready for review September 13, 2021 10:16
@ilankri
Copy link
Member

ilankri commented Sep 14, 2021

J'ai tenté de me renseigner sur https://github.com/mnxn/promise_jsoo mais je n'ai pas réussi à m'en servir.
Non seulement je ne suis pas sûr de voir où vous voulez en venir avec l'utilisation de cette librairie-ci (je n'ai pas compris quel étais le soucis avec ce que j'avais fait avant et pourquoi "Lwt" en lui même est insuffisant), mais je n'ai pas non plus réussi à installer le module opam "promise_jsoo_lwt" qui me semble tout adapté à la situation.

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 promise_jsoo_lwt n'a pas encore été publiée sur Opam… Donc si on veut l'utiliser, il va falloir l'épingler dans le fichier Opam, voir https://opam.ocaml.org/blog/opam-20-tips/#Pinnings.

@Thibaut-Gudin
Copy link
Contributor Author

J'ai tenté de me renseigner sur https://github.com/mnxn/promise_jsoo mais je n'ai pas réussi à m'en servir.
Non seulement je ne suis pas sûr de voir où vous voulez en venir avec l'utilisation de cette librairie-ci (je n'ai pas compris quel étais le soucis avec ce que j'avais fait avant et pourquoi "Lwt" en lui même est insuffisant), mais je n'ai pas non plus réussi à installer le module opam "promise_jsoo_lwt" qui me semble tout adapté à la situation.
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 promise_jsoo_lwt n'a pas encore été publiée sur Opam… Donc si on veut l'utiliser, il va falloir l'épingler dans le fichier Opam, voir https://opam.ocaml.org/blog/opam-20-tips/#Pinnings.

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.

"promise_jsoo_lwt"
]
pin-depends: [
[ "promise_jsoo_lwt" "git+https://github.com/mnxn/promise_jsoo"]
Copy link
Member

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:

Suggested change
[ "promise_jsoo_lwt" "git+https://github.com/mnxn/promise_jsoo"]
[ "promise_jsoo_lwt.dev" "git+https://github.com/mnxn/promise_jsoo"]

]
"lwt_ppx" {>= "2.0.1"}
"promise_jsoo"
"promise_jsoo-ppx"
Copy link
Member

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:

Suggested change
"promise_jsoo-ppx"

@Thibaut-Gudin
Copy link
Contributor Author

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...

@Thibaut-Gudin
Copy link
Contributor Author

Thibaut-Gudin commented Sep 15, 2021

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.

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

Successfully merging this pull request may close these issues.

2 participants