[PATCH] Move abilist files into sysdep dirs.

Roland McGrath roland@hack.frob.com
Fri Apr 27 19:46:00 GMT 2012


> So far the abilist contained expressions like 
> "x86_64-.*-linux.*/thread", so including the usage of nptl. I think we 
> had no abilists without nptl but the tools supported it.
> 
> I see no more way to generate abilists without thread.
> 
> I think it's fine that nptl is required (if my understanding of the 
> "thread" keyword in abilists is correct), just wanted to point this out 
> so we can think whether it makes a difference.

We could put the files into sysdeps/.../nptl/ subdirectories so they'll
apply in an nptl configuration and be missing entirely in a no-nptl
configuration (which should make such a configuration fail make check,
which seems fine).

But as Dave and Joseph say, we can also just let the next person to build
such a configuration worry about the subject.

I guess my pedantic preference would be to use the nptl/ subdirs.
But I won't argue strenuously if the people doing the actual work
have the contrary preference.

(I do think it's wiser to use sysdeps/.../nptl/ rather than
nptl/sysdeps/... for any new thing.)


Thanks,
Roland



More information about the Libc-alpha mailing list