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

MPR#6890: provide use_ocamlfind as a tag #185

Closed
damiendoligez opened this issue Mar 1, 2017 · 4 comments
Closed

MPR#6890: provide use_ocamlfind as a tag #185

damiendoligez opened this issue Mar 1, 2017 · 4 comments

Comments

@damiendoligez
Copy link
Member

PR transferred from https://caml.inria.fr/mantis/view.php?id=6890
[original reporter: @bobzhang]

it would be nice that I can write

true: use_ocamlfind

so I don't need type ocamlbuild -use-ocamlfind everytime, thanks -- Hongbo

@bluddy
Copy link

bluddy commented Mar 1, 2017

Honestly, this should already be a default. Any package not using ocamlfind by default is not living in the modern Ocaml world. If something as small as this can't be fixed due to backwards compatibility, that's a real problem, and it suggests maybe forking is needed.

@dbuenzli
Copy link
Collaborator

dbuenzli commented Mar 1, 2017

If something as small as this can't be fixed due to backwards compatibility, that's a real problem, and it suggests maybe forking is needed.

So you are going to be the brave soul who maintains ocamlbuild ? If not please keep these kind of comments for yourself, they achieve absolutely nothing except annoy current maintainers.

I think this can be closed in favour of #48 which also has a link to a mantis issue where this was tried at some point (I don't remember exactly why it was reverted). @gasche's comment related to the semantics of _tags in MPR6890 is right in my opinion.

@whitequark
Copy link
Member

Triage: is it even possible to meaningfully implement that? _tags is read much later after the command-line options are parsed, and the global state in Options is long recorded elsewhere by that point. In my understanding this (i.e. true: use_ocamlfind) request is too complex to implement for its value.

@gasche
Copy link
Member

gasche commented Mar 3, 2017

Let's close this in favor of simply changing the default, as discussed (now) in #214.

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

No branches or pull requests

5 participants