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

Reply via email to