Thanks for the help on the missing '-p1' for the acl and xattr patches. Things are configuring and making properly now. However, I couldn't help noticing this during the configure: checking whether to support ACLs... ACLs requested -- running tests checking for acl_get_file in -lacl... no checking for ACL support... yes checking ACL test results... Using posix ACLs checking for acl_get_perm_np... yes checking attr/xattr.h usability... no checking attr/xattr.h presence... no checking for attr/xattr.h... no checking whether to support extended attributes... Xattrs requested but not linux. Good luck Note the last line ... if my goal is to have OSX clients rsyncing their extended attributes files on to my FreeBSD 6.1 UFS2 filesystem ... is that a reasonable thing to be attempting ?
One last detail... On Wed, 1 Nov 2006, Ensel Sharon wrote:> checking whether to support ACLs... ACLs requested -- running tests > checking for acl_get_file in -lacl... no > checking for ACL support... yes > checking ACL test results... Using posix ACLs > checking for acl_get_perm_np... yes > checking attr/xattr.h usability... no > checking attr/xattr.h presence... no > checking for attr/xattr.h... no > checking whether to support extended attributes... Xattrs requested but > not linux. Good luck > > > Note the last line ... if my goal is to have OSX clients rsyncing their > extended attributes files on to my FreeBSD 6.1 UFS2 filesystem ... is that > a reasonable thing to be attempting ?During the 'make' I also saw this notice: gcc -I. -I. -g -O2 -DHAVE_CONFIG_H -Wall -W -I./popt -c acls.c -o acls.o acls.c: In function `change_sacl_perms': acls.c:735: warning: large integer implicitly truncated to unsigned type acls.c: In function `set_acl': acls.c:1163: warning: comparison is always false due to limited range of data type gcc -I. -I. -g -O2 -DHAVE_CONFIG_H -Wall -W -I./popt -c xattr.c -o xattr.o Comments ?
On Nov 1, 2006, at 9:57 PM, Ensel Sharon wrote:> I couldn't help noticing this during the configure: > > checking whether to support extended attributes... Xattrs requested > but > not linux. Good luckFrom what I've just been reading about FreeBSD extended attributes, it's not going to work for you, as is. I'm working on a patch for FreeBSD using this page as a reference: http://www.freebsd.org/cgi/man.cgi?query=extattr&format=html Be aware, however, that this stuff is extremely alpha!> Note the last line ... if my goal is to have OSX clients rsyncing > their > extended attributes files on to my FreeBSD 6.1 UFS2 filesystem ... > is that > a reasonable thing to be attempting ?Attempting? Sure. Succeeding? You'll need to help out! =D> The xattrs code has been ported to work with OS X, so it shouldn't be > hard to add the code necessary to get it to work with FreeBSD.Unfortunately, it appears the API is completely different. Give me a half hour! :-) According to wikipedia [1], linux, freebsd, solaris, and osx support extended attributes. It seems a worthy (and achievable) goal to get them working on all of these platforms, if we have people who can test the support. Wayne, I think it was a mistake to add an OS% infront of extended attributes from MacOS. I think we should just leave them unescaped. I also think the extended attribute for fake-super should just be "RSync.stat" [2]. I really think this should be fixed BEFORE 2.6.9! Otherwise people who use the patch as it is right now will have made bad backups if it's changed later. [1] http://en.wikipedia.org/wiki/Extended_Attributes [2] http://www.freedesktop.org/wiki/CommonExtendedAttributes