C / C++ / MFC

ForNow wrote:

I believe this has to do with sharing window handles between threads

sdancer75 wrote:

As you can see ... What ever I tried I had no success.

void CMainFrame::OnRunExternalApp() {


	SHELLEXECUTEINFO	lpExecInfo;
	PROCESS_INFORMATION processInfo;
	DWORD				dwExitCode;
	HANDLE				hProcess = 0;
	BOOL				bResult;
	LPTSTR				strCmd;





	strCmd = _T("");

	//bResult = StartupApplicationWithShell (_T("LiveUpdate.exe"), (LPTSTR)strCmd,  GetSafeHwnd(), &hProcess);
	bResult = StartupApplicationWithProcess (_T("C:\\Windows\\notepad.exe"), (LPTSTR)strCmd,  &processInfo);

	EnableWindow(FALSE);



	if(bResult) { //(UINT)result > HINSTANCE_ERROR)
	
		//ShowWindow( SW_HIDE );
		//ShowWindow( SW_SHOWNOACTIVATE );
		
	
	    //WaitForSingleObject( hProcess, INFINITE );
		WaitForSingleObject(processInfo.hProcess, INFINITE);
		//WaitForMultipleObjects(1, &processInfo.hThread, TRUE, INFINITE );

	

		   
		//if (!GetExitCodeProcess(hProcess, &dwExitCode)) {
		//	   	AMLOGINFO(_T("LiveUpdate is not terminated normally.")); 
		//}
		if (!GetExitCodeProcess(processInfo.hProcess, &dwExitCode)) {
			   	//failed
		}
		CloseHandle( processInfo.hProcess );
        CloseHandle( processInfo.hThread );
		//CloseHandle(hProcess);
 

	} else {
		
		// failed
	}

	BringWindowToTop();
	EnableWindow(TRUE);
	
	//UpdateWindow();

	//::SetWindowPos( GetSafeHwnd(), HWND_TOP, 0, 0, 0, 0, SWP_NOMOVE | SWP_NOSIZE | SWP_SHOWWINDOW );
}

EnableWindow
SetForegroundWindow
SetActiveWindow
BringWindowToTop

WaitForMultipleObjects

• Ask the user if the update should be installed
• Perform closing tasks that are time consuming or require user interaction like closing open documents
• Start the update process
• Terminate your application
• The update process should check if there is a running instance of your application (after a short wait time) and terminate with an error if so
• The update process might restart your application

Quote:

When the external update process should also update the executable of your main application, that must be terminated before the new exe file can be copyied. So there is no need to get the focus back.

Quote:

In another post you mentioned that you are using WaitForSingleObject to wait for the update process to be terminated. If you call that from within your main thread, your application's message loop is blocked so that paint events are not processed.

Quote:

If you don't want to update the executable itself and must wait for the external process to be finished start the process from within a worker thread where you can call WaitForSingleObject without blocking the message loop. Use a global state variable indicating that the update is executing to disallow specific tasks of your application or just show a modal dialog that is closed automatically when the update has finished.