[PATCH] TRAC Tickets #843 [20140425]
- First Post
- Replies
- Stats
-
Go to
- ----- 2024 -----
- April
- March
- February
- January
- ----- 2023 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2022 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2021 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2020 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2019 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2018 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2017 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2016 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2015 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2014 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2013 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2012 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2011 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2010 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2009 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2008 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
This patch addresses the following re-opened ticket:
* PKI TRAC Ticket #843 - Incorrect CLI argument parsing
<https://fedorahosted.org/pki/ticket/843>
I re-organized the code to make it more uniform and moved code around to
achieve the desired results.
This patch was tested on a Fedora 20 machine via the shell script called
/*pki/base/scripts/pkiclihelp*/.
Please review this patch.
Thanks,
-- Matt
Attachments:
- 20140425-Fixed-issue-by-streamlining-code-to-be-more-consiste.patch (text/x-patch — 227.9 KB)
- attachment.html (text/html — 718 bytes)
On 4/25/2014 10:06 PM, Matthew Harmsen wrote:
This patch addresses the following re-opened ticket:
* PKI TRAC Ticket #843 - Incorrect CLI argument parsing
<https://fedorahosted.org/pki/ticket/843>
I re-organized the code to make it more uniform and moved code around to
achieve the desired results.
This patch was tested on a Fedora 20 machine via the shell script called
/*pki/base/scripts/pkiclihelp*/.
Please review this patch.
Just to confirm, so all required options will now be listed in the usage
line? We probably should update the man pages too. This can be done in a
separate ticket.
If the number of arguments doesn't match exactly the required number,
some commands will show messages like these:
Error: Insufficient number of arguments specified.
Error: No <...> specified.
If the user specifies more than the required number of arguments the
above messages wouldn't be accurate. Probably the following generic
message will work better?
Error: Incorrect number of arguments specified.
Other than that it's ACKed.
--
Endi S. Dewata
3643
days inactive
3646
days old
1 comments
2 participants
tags (0)
participants (2)
-
Endi Sukma Dewata -
Matthew Harmsen