Re: [patch libiberty]: pex-win32 make sure that read handle has no inherit flag set, too

Re: [patch libiberty]: pex-win32 make sure that read handle has no inherit flag set, too

[Kai Tietz]

gcc-patches-owner@gcc.gnu.org wrote on 27.05.2009 15:03:39:

> Hello
> 
> This patch ensures, that the file descriptor has set inherit to false 
for 
> read open, too.
> 
> ChangeLog
> 
> 2009-05-27  Kai Tietz  <kai.tietz@onevision.com>
> 
>         * pex-win32.c (pex_win32_fdopenr): Set INHERIT to false.
> 
> Tested for i686-pc-cygwin, i386-pc-mingw32, and for x86_64-pc-mingw32. 
Ok 
> for apply?
> 
> Kai
> 
> |  (\_/)  This is Bunny. Copy and paste Bunny
> | (='.'=) into your signature to help him gain
> | (")_(") world domination.
> [attachment "pex_w32_inherit.diff" deleted by Kai Tietz/Onevision] 

Missed to add binutils, too.

Kai

|  (\_/)  This is Bunny. Copy and paste Bunny
| (='.'=) into your signature to help him gain
| (")_(") world domination.

Re: [patch libiberty]: pex-win32 make sure that read handle has no inherit flag set, too

[DJ Delorie]

> > 2009-05-27  Kai Tietz  <kai.tietz@onevision.com>
> > 
> >         * pex-win32.c (pex_win32_fdopenr): Set INHERIT to false.

Kai, you're a windows/cygwin/mingw maintainer.  You don't need our
approval to fix bugs in files specific to your targets.

Re: [patch libiberty]: pex-win32 make sure that read handle has no inherit flag set, too

[Kai Tietz]

2009/5/29 DJ Delorie <dj@redhat.com>:
>
>> > 2009-05-27  Kai Tietz  <kai.tietz@onevision.com>
>> >
>> >         * pex-win32.c (pex_win32_fdopenr): Set INHERIT to false.
>
> Kai, you're a windows/cygwin/mingw maintainer.  You don't need our
> approval to fix bugs in files specific to your targets.
>

Thanks for clarification. I assumed - as libiberty is under separate
maintainer-ship - an approval is still necessary.

Cheers,
Kai

-- 
|  (\_/) This is Bunny. Copy and paste
| (='.'=) Bunny into your signature to help
| (")_(") him gain world domination

Re: [patch libiberty]: pex-win32 make sure that read handle has no inherit flag set, too

[Ian Lance Taylor]

Kai Tietz <ktietz70@googlemail.com> writes:

> 2009/5/29 DJ Delorie <dj@redhat.com>:
>>
>>> > 2009-05-27  Kai Tietz  <kai.tietz@onevision.com>
>>> >
>>> >         * pex-win32.c (pex_win32_fdopenr): Set INHERIT to false.
>>
>> Kai, you're a windows/cygwin/mingw maintainer.  You don't need our
>> approval to fix bugs in files specific to your targets.
>>
>
> Thanks for clarification. I assumed - as libiberty is under separate
> maintainer-ship - an approval is still necessary.

I think we can reasonably describe pex-win32.c as a Windows specific
file.  Note that DJ is a libiberty maintainer (as am I), so his
deferring to you implies approval.

Ian