[OE-core] [PATCH] package.bbclass: Remove redundant chmod/chown operations

Phil Blundell pb at pbcl.net
Fri May 3 14:25:01 UTC 2013


On Thu, 2013-05-02 at 08:41 -0500, Mark Hatle wrote:
> On 5/2/13 8:20 AM, Phil Blundell wrote:
> > These were introduced in 6021e309e69d823e1467648aee12a32182945569.  The
> > code currently reads:
> >
> >                  os.link(file, fpath)
> >                  fstat = cpath.stat(file)
> >                  os.chmod(fpath, fstat.st_mode)
> >                  os.chown(fpath, fstat.st_uid, fstat.st_gid)
> >
> > which can have no useful effect since, if "fpath" is a hard link to
> > "file", it will (by definition) have the same mode, uid and gid.
> 
> I thought there were filesystems where a link operation can result in different 
> file permissions, owners and groups.  It's rare, but umask and effective uid/gid 
> could play a part in this. 

I can't think of any POSIX-conformant systems where this would be true.
1003.1 §3 is fairly clear that file permissions are a property of the
file, not of the link(s) to it, which would prohibit different links
from having different modes or owners.

Can you give a concrete example of such a scenario?

p.






More information about the Openembedded-core mailing list