On Mon, Feb 8, 2016 at 4:10 PM, Corinna Vinschen <corinna-cyg...@cygwin.com> wrote: > On Feb 8 15:31, Ismail Donmez wrote: >> On Mon, Feb 8, 2016 at 3:18 PM, Corinna Vinschen >> <corinna-cyg...@cygwin.com> wrote: >> > On Feb 8 12:01, Ismail Donmez wrote: >> >> Hi, >> >> >> >> cdrtools has some code to detect Solaris style ACLs: >> >> >> >> #if defined(HAVE_ACL) && defined(HAVE_FACL) && \ >> >> defined(HAVE_ACLFROMTEXT) && defined(HAVE_ACLTOTEXT) >> >> # define HAVE_SUN_ACL 1 /* Sun UFS ACL's present */ >> >> #endif >> >> >> >> Since cygwin still seems to be defining aclfromtext() and acltotext() >> >> functions (which are not defined in POSIX) cdrtools thinks this a >> >> Solaris-style system and get up getting a compile error later on. >> > >> > Probably due to including sys/acl.h. Does swtiching to cygwin/acl.h >> > help? Or changing the above check to prefer POSIX ACLs over Solaris >> > ACLs? >> >> This is a generic code so I don't want to add a cygwin specific >> dependency there. Is there a preprocessor definition for cygwin >> version? I could use that to disable HAVE_SUN_ACL for cygwin 2.5+ > > If you include cygwin/version.h you could use the version definitions. > > Alternatively we could allow to use the Solaris ACL functions even if > only including sys/acl.h, given some macro: > > sys/acl.h: > > #ifdef __USE_OLD_SOLARIS_ACL_FUNCTIONS > # include <cygwin/acl.h> > #else > [...POSIX definitions...] > #endif > > Would that help?
That should help, I cook a patch and send to cdrecord maintainer. Thanks a lot. ismail -- Problem reports: http://cygwin.com/problems.html FAQ: http://cygwin.com/faq/ Documentation: http://cygwin.com/docs.html Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple
