[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Acl-devel] [PATCH] attr: refer to ENODATA instead of ENOATTR in man

From: Mike Frysinger
Subject: Re: [Acl-devel] [PATCH] attr: refer to ENODATA instead of ENOATTR in man pages
Date: Thu, 27 Mar 2014 11:56:42 -0400
User-agent: KMail/4.12.3 (Linux/3.13.0; KDE/4.12.3; x86_64; ; )

On Thu 27 Mar 2014 13:23:09 Florian Weimer wrote:
> On 03/27/2014 07:19 AM, Mike Frysinger wrote:
> > On Tue 21 Jan 2014 14:27:33 Kamil Dudka wrote:
> >> The ENOATTR alias is only provided if <attr/xattr.h> is included, not
> >> with the more standard <sys/xattr.h> header file.  ENODATA is correct
> >> in both cases.
> > 
> > well, that's the case if the OS doesn't support ENOATTR, so the library
> > uses ENODATA in its place.  if the OS does have its own, then we return
> > ENOATTR everywhere, so checking ENODATA won't help.
> > 
> > it might be the case on Linux that ENOATTR == ENODATA, but i think the
> > goal is for libattr to be more than Linux.
> My concern is that it's confusing on Linux that you don't always get
> ENOATTR, so if you write code based on this manual page, it does not work.

sure, your compliant is perfectly reasonable :)

> Hmm, perhaps we should just make sure that glibc header defines ENOATTR,
> too?  That would resolve this issues as well.

i'm not sure how well that'd go over

the man-pages project has dealt with this too:

and their fix was:

 .I errno
 is set to
+is defined to be a synonym for
+.IR <attr/xattr.h> .)

wdyt of that solution ?

Attachment: signature.asc
Description: This is a digitally signed message part.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]