Re: [PATCH] Fix createUser

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Aug 15, 2011 at 09:53:06AM +0200, Martin Gracik wrote:
> On Fri, 2011-08-12 at 16:54 -0700, Brian C. Lane wrote:
> > From: "Brian C. Lane" <bcl@xxxxxxxxxx>
> > 
> > name and username are possible kwargs so the required argument needs
> > to be named something different to prevent a conflict with the kwargs.
> 
> Why are we supporting 3 different argument names for the same thing?
> 

We aren't. The problem is due to the switch to using kwargs conflicting
with the required argument. We could switch to just using kwargs, but
then the requirement for a user name isn't as clear. Or you could remove
the username from the kwargs, but that requires the caller to know the
name of the argument in use and is also not clean.

-- 
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)

Attachment: pgp6zZllFStFu.pgp
Description: PGP signature

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux