-
Notifications
You must be signed in to change notification settings - Fork 397
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
sync toolchains with main #10532
Closed
Closed
sync toolchains with main #10532
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
* chore: update to what wil be melange v4 Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com> * fix: broken tests Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com> --------- Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
Make it the same as file://. Both of these constructors mean the same thing, so let's keep one of them. Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
Opam's "compiler" flag is used too broadly to accurately identify packages containing ocaml compilers. For example this flag is set for compiler options packages such as ocaml-option-flambda which configure the ocaml-variants package, effectively preventing the use of compiler options packages, as ocaml-variants also has the "compiler" flag set, and dune only permits a single compiler package in a solution. This change fixes this problem by using the presence of the "ocaml-core-compiler" conflict class to identify compiler packages. Signed-off-by: Stephen Sherratt <stephen@sherra.tt>
Signed-off-by: Etienne Millon <me@emillon.org>
* doc: add `merlin_reader` to the `(dialect ...)` documentation Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com> * Update doc/reference/dune-project/dialect.rst Co-authored-by: Andrey Popp <8mayday@gmail.com> Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com> * fix: grammar Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com> * fix: use seealso and a stable tag Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com> --------- Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com> Co-authored-by: Andrey Popp <8mayday@gmail.com> Co-authored-by: Etienne Millon <me@emillon.org>
* test(melange): parse lint but show it not working in melange.emit Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com> * feat(melange): support lint in `melange.emit` Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com> --------- Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
….) (#10525) Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
Do not use OpamFile.Dot_config.read as it relies on some stubs to resolve paths in a special way we don't need. Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
Some options are interpreted by `bootstrap.ml`, other ones by `duneboot.ml`. They need to be parsed, but can be ignored instead of setting a dummy reference. Signed-off-by: Etienne Millon <me@emillon.org>
This options adds `-ccopt -static` to the link flags (this requires a libc that supports static linking). The motivation is `nix build .#dune-static`, which otherwise requires patching sources. Signed-off-by: Etienne Millon <me@emillon.org>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
merlin_reader
to the(dialect ...)
documentation (doc: addmerlin_reader
to the(dialect ...)
documentation #10483)melange.emit
(feat(melange): support lint inmelange.emit
#10494)