Skip to content

convert ocaml-googlemaps into a dune project - #30

Open
Thibaut-Gudin wants to merge 6 commits into
besport:masterfrom
Thibaut-Gudin:dune_converstion
Open

convert ocaml-googlemaps into a dune project#30
Thibaut-Gudin wants to merge 6 commits into
besport:masterfrom
Thibaut-Gudin:dune_converstion

Conversation

@Thibaut-Gudin

Copy link
Copy Markdown

No description provided.

Comment thread ocaml-googlemaps.opam Outdated
homepage: "https://github.com/besport/ocaml-googlemaps"
bug-reports: "https://github.com/besport/ocaml-googlemaps/issues"
license: "undefined"
description: "Binding to the `Googlemaps` plugin ussing gen_js_api"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

using

@Thibaut-Gudin
Thibaut-Gudin force-pushed the dune_converstion branch 2 times, most recently from ed5a39c to 3fb684a Compare September 21, 2021 15:58
@Thibaut-Gudin

Thibaut-Gudin commented Sep 22, 2021

Copy link
Copy Markdown
Author

Cela fait maintenant des heures que je tente en vain de corriger le déploiement de la conversion de la librairie en projet "dune": j'ai beau essayer je n'arrive pas à supprimer les "Warning" qui indiquent que le déploiement échoue sur tout les OS.

Il y a une dizaine de warining" de type: Error (warning 22): Heuristic for automatic binding is deprecated; please add the '@js.call' attribute.`

J'ai essayé divers approche mais je me retrouve toujours avec les mêmes warnings demandant des modifications absurdes telles que d'ajouter une balise "[@js.call]" à un constructeur ou mettre un "[@js.set" à une fonction pourtant explicitement nommée "set_X".

Je n'ai aucune piste pour remédier à cette situations.

@Thibaut-Gudin
Thibaut-Gudin force-pushed the dune_converstion branch 2 times, most recently from 7852de6 to afd05b1 Compare September 22, 2021 14:00
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

Successfully merging this pull request may close these issues.

2 participants