Date: Tue, 14 Jul 2009 09:02:35 -0700 From: Tim Kientzle <kientzle@freebsd.org> To: Jilles Tjoelker <jilles@stack.nl> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Edward Tomasz Napierala <trasz@freebsd.org> Subject: Re: svn commit: r195458 - in head: include lib/libc/sys sys/bsm sys/compat/freebsd32 sys/compat/svr4 sys/i386/ibcs2 sys/kern sys/sys Message-ID: <4A5CAC1B.4070006@freebsd.org> In-Reply-To: <20090709173046.GA48155@stack.nl> References: <200907081523.n68FNIpi055036@svn.freebsd.org> <20090709173046.GA48155@stack.nl>
next in thread | previous in thread | raw e-mail | index | archive | help
Jilles Tjoelker wrote: > On Wed, Jul 08, 2009 at 03:23:18PM +0000, Edward Tomasz Napierala wrote: >> Author: trasz >> Date: Wed Jul 8 15:23:18 2009 >> New Revision: 195458 >> URL: http://svn.freebsd.org/changeset/base/195458 > >> Log: >> There is an optimization in chmod(1), that makes it not to call chmod(2) >> if the new file mode is the same as it was before; however, this >> optimization must be disabled for filesystems that support NFSv4 ACLs. >> Chmod uses pathconf(2) to determine whether this is the case - however, >> pathconf(2) always follows symbolic links, while the 'chmod -h' doesn't. > >> This change adds lpathconf(3) to make it possible to solve that problem >> in a clean way. > > I think pathconfat() would be nicer than adding a function that will not > appear in standards and lacks a useful feature. Compare futimens() and > utimensat() in the 2008 POSIX spec, and notice that there is no such > thing as lutimes() in POSIX (it can be done with utimensat()). Another data point: I've started looking at the *at() functions for libarchive's deep directory handling. The current code uses chdir() to manipulate files with very long pathnames, but that causes complications for libarchive clients that I'd like to eliminate. Using *at() functions to manage a "virtual current directory" looks a lot more robust. Unfortunately, there are still a few operations that are unavailable with this paradigm: * ACL functions * extattr functions * link() * symlink()
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4A5CAC1B.4070006>