On Wed, Aug 7, 2019 at 5:09 AM Martin Liška <mli...@suse.cz> wrote: > > There's one enhanced version where I added HAVE_FCNTL_H. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Ready to be installed?
I think a better name than fd_exists would be is_valid_fd. If that's OK with you, the libiberty part of this patch seems fine with that change. Thanks. (Although I'm not quite sure what problem this is trying to solve. What goes wrong if the driver just doesn't bother to check whether the file descriptors listed in --jobserver-auth are valid? Why would that ever happen, and why would it matter if it did? The ChangeLog talks about the linker using the same file descriptors, but in that case won't this test see the descriptors as valid?) Ian