RFC: [PATCH] Retry replacing in-use files in setup

[Igor Peshansky <pechtcha at cs dot nyu dot edu>]

Hi,

Here's a quick and dirty patch to retry replacing in-use files.  Unless
setup is running in unattended mode, it will show an Abort/Retry/Ignore
dialog warning the user that a particular file was in-use.  The user can
"Retry" indefinitely, though the second and subsequent messages are going
to be a bit more stringent.  "Ignore" will prevent any further pop-ups
(the logic being that if even one file was set to be replaced on reboot,
might as well do all of them).

This worked for me in some limited testing, but is obviously not complete
(e.g., "Abort" doesn't work -- do we even need it?).

I'd like some comments from people on both the behavior and my
assumptions.  The patch was intentionally done in whitespace-indifferent
mode to make the actual changes easier to see, so please don't mention the
sloppy indentation. :-)  Just in case this gets passed with flying colors
and accepted into the codebase, I've included the ChangeLog below.
	Igor
==============================================================================
2006-02-14  Igor Peshansky  <pechtcha@cs.nyu.edu>

	* install.cc (Installer::installOne): If file is in use, ask the user
	to stop processes and retry.

-- 
				http://cs.nyu.edu/~pechtcha/
      |\      _,,,---,,_	    pechtcha@cs.nyu.edu | igor@watson.ibm.com
ZZZzz /,`.-'`'    -.  ;-;;,_		Igor Peshansky, Ph.D. (name changed!)
     |,4-  ) )-,_. ,\ (  `'-'		old name: Igor Pechtchanski
    '---''(_/--'  `-'\_) fL	a.k.a JaguaR-R-R-r-r-r-.-.-.  Meow!

"Las! je suis sot... -Mais non, tu ne l'es pas, puisque tu t'en rends compte."
"But no -- you are no fool; you call yourself a fool, there's proof enough in
that!" -- Rostand, "Cyrano de Bergerac"

Attachment: setup-retry-replace.patch
Description: Text document