Proposed fixes to gspawn-win32-helper.c for newer Microsoft CRTs (was: glib-mkenums in glib 2)
- From: Fan Chun-wei <fanc999 yahoo com tw>
- To: John Emmas <johne53 tiscali co uk>
- Cc: gtk-devel-list <gtk-devel-list gnome org>
- Subject: Proposed fixes to gspawn-win32-helper.c for newer Microsoft CRTs (was: glib-mkenums in glib 2)
- Date: Wed, 20 Feb 2013 14:50:41 +0800
Hi John,
I'm sorry if this sounds a bit off-topic, but can you check your Visual
Studio 2005 build (if I recalled correctly, that's the version of Visual
Studio you are using) of
gspawn-win32-helper.exe/gspawn-win32-helper-console.exe whether programs
using them are able to complete successfully without being
terminated/aborted by Windows. One good example to check here is using
the glib-compile-resources utility program in gio/, the other ones would
be the spawn-singlethread* and spawn-multithreaded* in
$(srcroot)/glib/tests.
With blessings, thank you!
-Fan, Chun-wei
---
Hi list,
I have posted a bug report with a proposed patch at
https://bugzilla.gnome.org/show_bug.cgi?id=693646, as newer Microsoft
CRTs are much more picky on the parameters passed into close() and
_get_osfhandle(), so it would be greatly appreciated on my side if
people building and using GLib on Windows can check that for me (so far,
no feedback yet), especially whether:
-People using Visual Studio 2005 or later to build and use GLib using
msvcr80.dll or later does not run into the bug
-This does not break GLib builds of people using MinGW to build and use
GLib (or those using MSVCRT.dll)
With blessings, thank you!
-Fan, Chun-wei
---
*You will need to edit the spawn-singlethread.c and
spawn-multithreaded.c and append ".exe" to the various parts referring
to test-spawn-echo, and you will need to build test-spawn-echo.exe
before running the spawn tests
You can comment out this line:g_test_add_func ("/gthread/spawn-script",
test_spawn_script);
as that test involves running the test scripts in a shell.
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]