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

Fix --help output for options without a short flag (again). #1

Open
wants to merge 1 commit into
base: cheriot-submodule
Choose a base branch
from

Conversation

rmn30
Copy link
Collaborator

@rmn30 rmn30 commented Jun 5, 2024

The simulator --help output container unprintable characters for options without a short flag.
This was previously fixed in ffa591d but broken again in 037c835.
This reverts the relevant lines to the upstream version.

Since this branch is regularly rebased we could optionally merge the fix with the latter commit and remove the diff from upstream entirely.

@rmn30 rmn30 requested a review from nwf June 5, 2024 10:43
@rmn30
Copy link
Collaborator Author

rmn30 commented Jun 5, 2024

Apparently on this branch the relevant upstream commit is 34a32a2 not sure why it has a different hash. Update: most likely due to committer info etc. being added when it was merged upstream (I was looking at the commit from the PR).

The simulator --help output container unprintable characters for
options without a short flag.  This was previously fixed in
34a32a2 but broken again in
037c835.  This reverts the relevant
lines to the upstream version.
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