Skip to content
This repository has been archived by the owner on Aug 25, 2022. It is now read-only.

Command exit master #180

Merged
merged 2 commits into from
Oct 14, 2019
Merged

Command exit master #180

merged 2 commits into from
Oct 14, 2019

Conversation

hannesm
Copy link
Member

@hannesm hannesm commented Oct 14, 2019

that's #179 but targeting master instead of 2.2!

…line

processor):
- on parse failure, 64
- if help or version was requested, 63

reasoning is to design reasonable restart-on-failure semantics.
@hannesm hannesm force-pushed the command-exit-master branch from 6b9a97e to f33aa7a Compare October 14, 2019 18:33
@hannesm
Copy link
Member Author

hannesm commented Oct 14, 2019

merging to stay consistent with released 2.2 series

@hannesm hannesm merged commit 5b6b208 into mirage:master Oct 14, 2019
@hannesm hannesm deleted the command-exit-master branch October 14, 2019 19:36
samoht added a commit to samoht/opam-repository that referenced this pull request Oct 22, 2019
CHANGES:

* Use `dune` to compile `config.ml` into an executable and run it.
  This replaces the use of `ocamlbuild` and dynlinking of `config.ml`
  (mirage/functoria#176, @samoht)
  The new compilation scheme:
  - generates `dune`, `dune.config` and `dune.build` with sensible
    configuration values. Each file can be overwritten by the user,
    in that case functoria will detect it and will not remove during
    the clean step;
  - by default, `dune` just includes `dune.config` and `dune.build`;
  - by default, `dune.config` contains the rules to build `config.ml`
    into `config.exe`;
  - by default, `dune.build` is empty -- functoria users such as
    `mirage` can just overwrite that file with the rigth build rules.
* Invoke `opam list` with `--color=never` (mirage/functoria#177, @ehmry)
* Use different exit codes in `Functoria_runtime.with_argv` (mirage/functoria#180, @hannesm)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant