Page MenuHomeFreeBSD

mfh: Make argument parsing a bit more robust
ClosedPublic

Authored by 0mp on Aug 5 2020, 2:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 28, 2:59 PM
Unknown Object (File)
Sun, Jun 28, 4:26 AM
Unknown Object (File)
Sun, Jun 28, 3:09 AM
Unknown Object (File)
Fri, Jun 26, 7:52 PM
Unknown Object (File)
Fri, Jun 26, 7:39 AM
Unknown Object (File)
May 14 2026, 12:31 PM
Unknown Object (File)
May 6 2026, 10:30 PM
Unknown Object (File)
May 1 2026, 9:43 AM
Subscribers

Details

Reviewers
None
Group Reviewers
portmgr
Commits
rP544225: Make argument parsing a bit more robust
Summary
Make argument parsing a bit more robust

Currently, when running mfh with a flag like --help the user gets a confusing
error message from expr(1) like the following:

> expr: illegal option -- -
> expr: usage: expr [-e] expression
>
> revision "--help" should be a number

This is not helpful. Instead, make sure that any arguments specified by the
user are handled properly by expr(1). This results in a much cleaner error
message:

> revision "--help" should be a number

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

0mp requested review of this revision.Aug 5 2020, 2:19 PM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 5 2020, 3:03 PM
This revision was automatically updated to reflect the committed changes.