Created attachment 6273 [details]
Patch

-#if defined(POSIX_ACL_NEEDS_MASK)
+#if defined(POSIX_ACL_NEEDS_MASK) || defined (__TANDEM) /* or in configure */

Is that hunk still needed, given that you define POSIX_ACL_NEEDS_MASK in configure?

Volker

It is an alternative. Sorry, should have mentioned this...

Bye, Jojo

Can you fix this to not use a system-specific 

if defined(__TANDEM)

please ? It took a long time to move away from system-specific ifdefs and I *really* don't want to move back..

Jeremy.

Hmm, should be possible, it would mean a couple new checks in configure thoguh, for:

- sys/acl.h (vs. sys/aclv.h)
- hpux_acl_call_present()
- hpux_aclsort_call_present()

Plus the corresponding changes to vfs_hpuxacl.c (e.g. #ifdef HAVE_SYS_ACL_H, #ifdef HPUX_ACL_CALL_ALWAYS_PRESENT, #ifdef HPUX_ACLSORT_CALL ALWAYS_PRESENT

I'm not fluent enough in the configure/autoconf/automake syntax for this, unfortunatly.

bye, Jojo

On Wed, Apr 06, 2011 at 08:21:37AM +0000, samba-bugs@samba.org wrote:
> --- Comment #5 from Joachim Schmitz <schmitz@hp.com> 2011-04-06 08:21:37 UTC ---
> Hmm, should be possible, it would mean a couple new checks in configure thoguh,
> for:
> 
> - sys/acl.h (vs. sys/aclv.h)
> - hpux_acl_call_present()
> - hpux_aclsort_call_present()
> 
> Plus the corresponding changes to vfs_hpuxacl.c (e.g. #ifdef HAVE_SYS_ACL_H,
> #ifdef HPUX_ACL_CALL_ALWAYS_PRESENT, #ifdef HPUX_ACLSORT_CALL ALWAYS_PRESENT
> 
> I'm not fluent enough in the configure/autoconf/automake syntax for this,
> unfortunatly.

For this kind of change access to such a box would be
extremely handy.

Volker

Access to such a system has been set up...

Any chance of moving this forward somehow?