-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Michael Brown wrote:
| Hi,
|
| samba-2.2.3a-10mdk was somehow built without ACL support, even though the
| SPEC file should include it.
|
| Proof:
|
|   1. rpm --requires samba does not list libacl as a requirement.

Seems to be so ...

|   2. ldd /usr/sbin/smbd does not list libacl as a linked library

Which is the reason for 1)

|   3. I cannot get Samba to display or modify ACLs created with setfacl on
|      an XFS filesystem.

This may not solve the actual problem, but it should help you:

There are updated RPMs of 2.2.5a for 8.2 (and 8.1 and 8.0, but no acl
support on 8.0 by default) with working acl support on ftp.samba.org and
~ other versions both on my site
(http://ranger.dnsalias.com/mandrake/samba) and Sylvestre's
(http://people.mandrakesoft.com/~staburet/samba)

|
|   4. Rebuilding samba-2.2.3a-10mdk gives an smbd that does link against
|      libacl (according to ldd).

Could you try rebuilding the SRPM without libattr-devel? (I don't have
an 8.2 box handy to try on, I could try on cooker ...)

|
| This is cc'ed to QA since there is nothing wrong with the source RPM file
| (no missing BuildRequires for libacl-devel etc.) and yet somehow the
| package managed to get built and distributed without ACL support.

I think we will have to take a look at this, since it recently happened
also with 2.2.5a and acl 2.0.9.

Geoffrey Lee reckoned it was due to the samba configure script not
checking for libattr. And the samba rpm doesn't buildrequires
libattr-devel, and the libacl-devel (which samba does buildrequire)
doesn't require libattr.

But I am quite sure I have built samba RPMs with acl support without
libattr-devel.

It does link without libattr-devel:

[bgmilne@bgmilne source]$ rpm -q libattr1-devel
package libattr1-devel is not installed
[bgmilne@bgmilne source]$ rpm -q libacl1-devel
libacl1-devel-2.0.11-1mdk
[bgmilne@bgmilne source]$ rm -f bin/smbd
[bgmilne@bgmilne source]$ make >/dev/null
[bgmilne@bgmilne source]$ ldd bin/smbd |grep acl
~        libacl.so.1 => /lib/libacl.so.1 (0x40024000)


But also against libattr
[bgmilne@bgmilne source]$ ldd bin/smbd |grep attr
~        libattr.so.1 => /lib/libattr.so.1 (0x40288000)

So, it seems like a packaging problem on libacl, libacl-devel should
buildrequires libattr-devel

It's actually been quite a mission getting acl support in samba (it's
been broken so many times ...), but it is worth it.

Buchan


- --
|----------------Registered Linux User #182071-----------------|
Buchan Milne                Mechanical Engineer, Network Manager
Cellphone * Work            +27 82 472 2231 * +27 21 8828820x121
Stellenbosch Automotive Engineering         http://www.cae.co.za
GPG Key                   http://ranger.dnsalias.com/bgmilne.asc
1024D/60D204A7 2919 E232 5610 A038 87B1 72D6 AC92 BA50 60D2 04A7
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQE9SoUarJK6UGDSBKcRAs41AJwOEeQB6mEyS8H8Cn/tm1kCa3ih5gCeK7NA
kLLtKST4/5fdXLcZcJRxslw=
=k6nX
-----END PGP SIGNATURE-----


Reply via email to