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

Add a way to exit GAP immediately, with an exit code. #365

Closed
fingolfin opened this issue Nov 19, 2015 · 6 comments
Closed

Add a way to exit GAP immediately, with an exit code. #365

fingolfin opened this issue Nov 19, 2015 · 6 comments

Comments

@fingolfin
Copy link
Member

Currently, I have to use IO_exit for that, but as was explained in #364, this is not so nice for various reasons.

Such a feature is very important if one wants to properly script running GAP computations. E.g. when building the manual, or running tests, being able to set an exit code in order to signal the build system that there was a failure.

@stevelinton
Copy link
Contributor

Would it be enough to add a one-argument version of QUIT_GAP() and FORCE_QUIT_GAP()?

@ChrisJefferson
Copy link
Contributor

Or, what's wrong with QUIT_GAP and FORCE_QUIT_GAP already? :) (you may have missed them being added).

@olexandr-konovalov
Copy link
Member

@ChrisJefferson isnt' this something what you've done recently ?

@fingolfin
Copy link
Member Author

Great! But I wasn't aware of it. I searched my email archive, I found an email by @ChrisJefferson from October 2014 that he wants to modify QUIT_GAP in that way. But I saw no further discussion, nor any description of this new feature in dev/Updates or any GAP release notes, i.e. it is not mentioned in chapter 2 of the GAP manual.

In fact, it doesn't seem to be documented at all: I just re-run make manuals, and it neither appears in the index, nor does the built-in help system know anything about it. Though I do see it being mentioned in doc/ref/mloop.xml. Huh?

@fingolfin
Copy link
Member Author

Argh, never mind. For some really stupid reason, GAP picked up the manual from another, older GAP installation (a matter of a badly placed symlink). Fixed that, so now I see QUIT_GAP in all its glory in the manual.

Still think it would be nice to mention it in an dev/Updates note, though :)

@olexandr-konovalov
Copy link
Member

BTW, PR #150 is marked for GAP 4.8.0 milestone.

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

No branches or pull requests

4 participants