This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH v3 4/7] manual: Refactor errno @comments.


On Wed, May 17, 2017 at 12:44 AM, Rical Jasan <ricaljasan@pacific.net> >
> I was unaware of these dependencies---thanks for pointing them out.
> Nothing I've encountered indicated their existence, though "DO NOT
> REMOVE" makes a little more sense.
>
> After hacking on this a bit, I think this patch should be rescinded.
> Fixing the errlist.awks is easy, but it does nothing to help the screwy,
> Texinfo @comment-based dependency that is just asking for trouble.  I'd
> like to propose an @errno{} macro to handle these, and then update the
> errlist.awks accordingly.  Something like:
>
>   @errno{EPERM, 1, Operation not permitted}
>
> should make the Awk scripts practically benign...

This seems like a reasonable plan to me.  Note that
https://sourceware.org/ml/libc-alpha/2017-05/msg00198.html also makes
changes in this area.

I think the error *numbers* shouldn't be embedded in the manual.  Only
Hurd uses these, and so it's not obvious that this part of the manual
has ABI implications but only for Hurd.  It would be better if those
came from a sysdeps/ file - sysdeps/mach/hurd/bits/errno.h.in maybe?
It can't just be sysdeps/mach/hurd/bits/errno.h because the Hurd
errnos.awk pulls additional error numbers from several other places
(unfortunately, all of them are Mach headers that aren't in tree,
IIRC).

zw


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]