Skip to content

Conversation

@smorimoto
Copy link
Member

No description provided.

Signed-off-by: Sora Morimoto <[email protected]>
@hhugo
Copy link
Member

hhugo commented Oct 25, 2025

I don't understand this PR. Isn't jsoo already compatible with cmdliner v2 ?

@dbuenzli
Copy link
Contributor

Also the changes have nothing to do with cmdliner. They propagate the exit code of main to let exit pick them up (which apparently was not done before).

At that point you could also replace the exit calls in the error branches by the code number so that the program has a single place where it exits.

@hhugo
Copy link
Member

hhugo commented Oct 28, 2025

exit code fixed in #2121

@hhugo hhugo closed this Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants