search

synhershko / NAppUpdate

A simple framework for providing auto-update support to .NET applications
http://www.code972.com/blog/2010/08/nappupdate-application-auto-update-framework-for-dotnet/
Apache License 2.0
525 stars 163 forks source link

#35 Added delay in case there is a lock on files that are to be updated #86

Closed robinwassen closed 8 years ago

robinwassen commented 8 years ago

This should fix issue #35 , one of my colleagues will code review and QA test this before merging.

FWennerdahl commented 8 years ago

I have tested and code-reviewed this. Applying this change fixes #35 in a consistent reproduction case. Code changes look solid.

The comment at http://stackoverflow.com/a/937558 that explains that the solution wouldn't work if the file is reopened after the check shouldn't apply in this case since it's not expected that the application under update will be reopened.