X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/647b8e379ed0f470f0f643fe816ba872be41541d..66010e8d0d04c180c51203f9ed7e2c26c78eff25:/src/unix/utilsunx.cpp diff --git a/src/unix/utilsunx.cpp b/src/unix/utilsunx.cpp index a9a3c1156f..d0cb4c4ae2 100644 --- a/src/unix/utilsunx.cpp +++ b/src/unix/utilsunx.cpp @@ -544,7 +544,7 @@ long wxExecute(wxChar **argv, wxChar **mb_argv = argv; #endif // Unicode/ANSI -#if wxUSE_GUI +#if wxUSE_GUI && !(defined(__DARWIN__) && defined(__WXMAC__)) // create pipes wxPipe pipeEndProcDetect; if ( !pipeEndProcDetect.Create() ) @@ -555,7 +555,7 @@ long wxExecute(wxChar **argv, return ERROR_RETURN_CODE; } -#endif // wxUSE_GUI +#endif // wxUSE_GUI && !(defined(__DARWIN__) && defined(__WXMAC__)) // pipes for inter process communication wxPipe pipeIn, // stdin @@ -606,9 +606,9 @@ long wxExecute(wxChar **argv, if ( fd == pipeIn[wxPipe::Read] || fd == pipeOut[wxPipe::Write] || fd == pipeErr[wxPipe::Write] -#if wxUSE_GUI +#if wxUSE_GUI && !(defined(__DARWIN__) && defined(__WXMAC__)) || fd == pipeEndProcDetect[wxPipe::Write] -#endif // wxUSE_GUI +#endif // wxUSE_GUI && !(defined(__DARWIN__) && defined(__WXMAC__)) ) { // don't close this one, we still need it @@ -630,12 +630,12 @@ long wxExecute(wxChar **argv, } #endif // !__VMS -#if wxUSE_GUI +#if wxUSE_GUI && !(defined(__DARWIN__) && defined(__WXMAC__)) // reading side can be safely closed but we should keep the write one // opened pipeEndProcDetect.Detach(wxPipe::Write); pipeEndProcDetect.Close(); -#endif // wxUSE_GUI +#endif // wxUSE_GUI && !(defined(__DARWIN__) && defined(__WXMAC__)) // redirect stdin, stdout and stderr if ( pipeIn.IsOk() ) @@ -654,6 +654,11 @@ long wxExecute(wxChar **argv, execvp (*mb_argv, mb_argv); + fprintf(stderr, "execvp("); + for ( char **ppc = mb_argv; *ppc; ppc++ ) + fprintf(stderr, "%s%s", ppc == mb_argv ? "" : ", ", *ppc); + fprintf(stderr, ") failed with error %d!\n", errno); + // there is no return after successful exec() _exit(-1); @@ -709,6 +714,31 @@ long wxExecute(wxChar **argv, #if wxUSE_GUI && !defined(__WXMICROWIN__) wxEndProcessData *data = new wxEndProcessData; + // wxAddProcessCallback is now (with DARWIN) allowed to call the + // callback function directly if the process terminates before + // the callback can be added to the run loop. Set up the data. + if ( flags & wxEXEC_SYNC ) + { + // we may have process for capturing the program output, but it's + // not used in wxEndProcessData in the case of sync execution + data->process = NULL; + + // sync execution: indicate it by negating the pid + data->pid = -pid; + } + else + { + // async execution, nothing special to do - caller will be + // notified about the process termination if process != NULL, data + // will be deleted in GTK_EndProcessDetector + data->process = process; + data->pid = pid; + } + + +#if defined(__DARWIN__) && defined(__WXMAC__) + data->tag = wxAddProcessCallbackForPid(data,pid); +#else data->tag = wxAddProcessCallback ( data, @@ -716,16 +746,10 @@ long wxExecute(wxChar **argv, ); pipeEndProcDetect.Close(); +#endif // defined(__DARWIN__) && defined(__WXMAC__) if ( flags & wxEXEC_SYNC ) { - // we may have process for capturing the program output, but it's - // not used in wxEndProcessData in the case of sync execution - data->process = NULL; - - // sync execution: indicate it by negating the pid - data->pid = -pid; - wxBusyCursor bc; wxWindowDisabler wd; @@ -751,12 +775,6 @@ long wxExecute(wxChar **argv, } else // async execution { - // async execution, nothing special to do - caller will be - // notified about the process termination if process != NULL, data - // will be deleted in GTK_EndProcessDetector - data->process = process; - data->pid = pid; - return pid; } #else // !wxUSE_GUI