Skip to content

Commit

Permalink
Removes the IPC for launching the app from the updater.
Browse files Browse the repository at this point in the history
First, the updater don't have to push information to the app process (as far as I can see).
Second, this causes major trouble with multiple updates with "update chains" (multiple restarting updates without existing the application between the updates). This comes because the first update process still blocks the named pipe making it impossible for the second update to start the cold update process.
  • Loading branch information
phillipp committed Aug 23, 2012
1 parent c571e51 commit acfbb69
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions src/NAppUpdate.Updater/AppStart.cs
Original file line number Diff line number Diff line change
Expand Up @@ -179,9 +179,11 @@ private static void Main()
FileName = appPath,
};

var p = NauIpc.LaunchProcessAndSendDto(dto, info, syncProcessName);
if (p == null)
throw new UpdateProcessFailedException("Unable to relaunch application");
try {
Process.Start(info);
} catch(Exception) {
throw new UpdateProcessFailedException("Unable to relaunch application");
}
}

Log("All done");
Expand Down

0 comments on commit acfbb69

Please sign in to comment.