Re: [Ipmitool-devel] Code Review - ID: 3595612 - ask for password once only if used

[Ales L. <ale...@re...>]

----- Original Message -----
> From: "Zdenek Styblik" <zde...@gm...>
> To: "Ales Ledvinka" <ale...@re...>
> Cc: "ipmitool-devel" <ipm...@li...>
> Sent: Tuesday, January 15, 2013 3:03:28 PM
> Subject: Re: Code Review - ID: 3595612 - ask for password once only if used
> 
> On Tue, Jan 15, 2013 at 2:54 PM, Ales Ledvinka <ale...@re...>
> wrote:
> [...]
> >
> > Sure. For the specific corner cases that I overlooked. Unlikely
> > to be used by anyone. Rather then dropping the compatibility
> > for almost anyone by reworking and failing on invalid argument
> > combinations - your proposal.
> >
> 
> Automation is not a corner case. But yes. And software develops,
> evolves, things do change.
> 
> [...]
> > For the -a -f filename example it is:
> > Same variable.
> > Interactive input was dropped. Now not asked for at all.
> > Last occurred option used as before.
> >
> 
> Oh, you finally got it?

QA perspective? All the time. User perspective? Not yet.

> > Claims to be annoyance removal. Since we do not share the same idea
> > of the bug in the option parsing there is no common resolution of
> > what
> > can be proper fix.
> >
> 
> That's right. And I actually gave it an extra thought and I think
> this
> is not something that needs and has to be fixed. Because you're not
> going to police all arguments, which were passed once or N times, in
> which order, are you? So what makes '-a' and '-Y' special? Nothing.
> It's just a mere deficiency of getopt() or perhaps it's misuse or
> perhaps just a lack of thought in ipmitool's design. Only shadow
> knows.
> Please, close the ticket as ``Won't fix'' since this is not something
> that needs fixing.
> 
> Thanks!
> Z.
>