Is it a bug of mock?

[Clark Williams]

On Tue, 17 Jan 2012 12:25:28 +0000
Kirby Zhou <kirbyzhou at sogou-inc.com> wrote:

> 
> /usr/lib/python2.6/site-packages/mockbuild/uid.py
> mock-1.1.18-1.el6.noarch
> 
> ccache plugin calls changeOwner to take the ownership of ccache directory,
> but why set gid to uid?
> 
> 70     decorate(traceLog())
> 71     def changeOwner(self, path, uid=None, gid=None):
> 72         self._elevatePrivs()
> 73         if uid is None:
> 74             uid = self.unprivUid
> 75         if gid is None:
> 76             gid = uid
> 77         os.chown(path, uid, gid)
> 
> Regards
> Kirby Zhou
> +86 (10) 6272 8261
> 

Kirby,

I'm not sure why the gid was set to the uid (I'm not the original
author of the uid.py code).

Seems more logical to set it to -1 so that it will be unchanged...

Clark