Follow-up Comment #1, bug #27809 (project make):
Thank you for your contribution. I have a few questions for the patches you
sent:
1. What versions of GCC and of MinGW runtime did you use to build Make?
2. Which header defines pid_t as appropriate for both 32-bit and 64-bit
Windows? What is the definition for each one of them?
3. Why did you need casts in assignments, like this:
- *pid_p = (int) hProcess;
+ *pid_p = (pid_t) hProcess;
Didn't it work without a cast? (There are quite a few such casts; please see
which ones are absolutely necessary.)
4. This change:
- pipedes[0] = _open_osfhandle((long) hChildOutRd, O_RDONLY);
+ pipedes[0] = _open_osfhandle((intptr_t) hChildOutRd, O_RDONLY);
assumes that _open_osfhandle accepts an intptr_t type as its first argument.
But the prototype I have on my machine (in io.h) says the first argument is a
`long'. Which version of MinGW changed that?
5. Finally, could you please see if the build_w32.bat script works for a
64-bit MinGW GCC? If you see problems there, please report them.
Thanks.
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/bugs/?27809>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/
_______________________________________________
Bug-make mailing list
Bug-make@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-make
