Skip to content
New issue

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

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

Already on GitHub? # to your account

Use dune-build-info to manage version information #1783

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

smorimoto
Copy link
Member

@smorimoto smorimoto commented Jan 1, 2025

The purpose of using dune-build-info is to simplify managing the version of packages, but I still need a pointer because I don't know where to ideally place the version retrieval file.

Ref: https://dune.readthedocs.io/en/stable/dune-libs.html#dune-build-info-library

Signed-off-by: Sora Morimoto <sora@morimoto.io>
@@ -4,10 +4,9 @@
(package js_of_ocaml-compiler)
(libraries
jsoo_cmdline
js_of_ocaml-compiler
Copy link
Member

Choose a reason for hiding this comment

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

js_of_ocaml-compiler is a (direct) dependency

@@ -2,7 +2,7 @@
(name jsoo_minify)
(public_name jsoo_minify)
(package js_of_ocaml-compiler)
(libraries jsoo_cmdline js_of_ocaml-compiler cmdliner compiler-libs.common)
Copy link
Member

Choose a reason for hiding this comment

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

js_of_ocaml-compiler is a direct dependency

@hhugo
Copy link
Member

hhugo commented Jan 6, 2025

  • The version is no longer reported in the toplevel
  • The version is only available after installation, I suppose this doesn't play well with mono-repos.
  • The version reported is incorrect because the tag of latest version 5.9.1 is not an ancestor of master. With this PR, I get 5.9.0-617-g3b9f656 but I would expect a 5.9.1 prefix.

@hhugo hhugo marked this pull request as draft January 16, 2025 22:52
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants