Skip to content

Resequence exit codes in jparse/util #1193

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

Merged
merged 2 commits into from
Feb 28, 2025
Merged

Resequence exit codes in jparse/util #1193

merged 2 commits into from
Feb 28, 2025

Conversation

xexyl
Copy link
Contributor

@xexyl xexyl commented Feb 28, 2025

It appears that this was not done or something went wrong when doing so
(as running make seqcexit updated the exit codes and this comes from
after running it in jparse/ and committing and then syncing from
jparse to jparse/).

Also as an update to the previous commit with the minor fix and
improvement to mkiocccentry -y and -Y warnings, I meant to update the
MKIOCCCENTRY_VERSION in the CHANGES.md; it is now "1.2.36 2025-02-28".

The code did not explicitly check for -Y; it just used the implicit -y from
-Y. That is fixed and now if not -q show a simple summary but always show
the longer warning to be sure that they know. As for -Y it really ought to be
used only for the test script but even -y should be used with EXTREME
caution.

It is hoped this is the last update to mkiocccentry(1) prior to the soft code
freeze today.

TODO: finish work on the chkentry_test.sh script.
It appears that this was not done or something went wrong when doing so
(as running make seqcexit updated the exit codes and this comes from
after running it in jparse/ and committing and then syncing from
jparse to jparse/).

Also as an update to the previous commit with the minor fix and
improvement to mkiocccentry -y and -Y warnings, I meant to update the
MKIOCCCENTRY_VERSION in the CHANGES.md; it is now "1.2.36 2025-02-28".
@xexyl
Copy link
Contributor Author

xexyl commented Feb 28, 2025

Ugh. I accidentally committed work on chkentry_test.sh here .... well it's not being run by ioccc_test.sh yet so it's okay. I'll continue my work on it anyway.

Copy link
Contributor

@lcn2 lcn2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@lcn2 lcn2 merged commit 01b8689 into ioccc-src:master Feb 28, 2025
3 checks passed
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

Successfully merging this pull request may close these issues.

2 participants