[PATCH] Don't show setup version warning message box in unattended mode

Corinna Vinschen corinna-cygwin@cygwin.com
Tue Nov 19 11:21:00 GMT 2013


Hi Jon,

On Nov 18 16:47, Jon TURNEY wrote:
> 
> As was pointed out on IRC a week or two ago, setup can show a "The current ini
> file is from a newer version of setup..." message box even in unattended mode.
> 
> Attached is a patch to fix.
> 
> There are other uses of MessageBox() which probably present a similar problem
> and still need fixing.
> 
> 2013-11-18  Jon TURNEY  <...>
> 
> 	* msg.cc (mbox): Generalize to a MessgeBox() wrapper which knows
                                         ^^^^^^^^^^^
                                         typo.  Also, just remove the "()",
                                         they are unnecessary.

> 	what to do in unattended mode.
> 	* msg.h: Add prototype.
> 	* ini.cc (IniParseFeedback): Use here rather than MessageBox().
                                                                    ^^
                                                                    same here.

Other than this minor ChangeLog nit, the patch looks good, please apply.
If you fix further MessageBox usages to work fine in attended and
unattended mode, consider them pre-approved.


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://cygwin.com/pipermail/cygwin-apps/attachments/20131119/c7ae8e90/attachment.sig>


More information about the Cygwin-apps mailing list