This is the mail archive of the
mailing list for the Cygwin project.
Re: [PATCH] inform user if any postinstall script failed to run
On 12 August 2010 19:13, Christopher Faylor wrote:
>>> I think that wording will still cause consternation.
>>> Maybe we need something like:
>>> "This does not necessarily mean that the affected package will fail to
>>> function properly but if you do notice problems please check
>>> /var/log/setup.log.full ."
>>How about removing the popup box and amending the postinstall results
>>page along those lines?
> I think removing the MessageBox is a good idea but I still think we
> should be slightly more reassuring about the errors. ÂThis might just
> be a temporary thing as package maintainers fix the problems in their
> scripts; assuming that they actually do that. ÂSo far the response has
> not been overwhelming.
> But assuming that the problems are all fixed then we can drop the
> assurance in a couple of months.
RCS file: /cvs/cygwin-apps/setup/res.rc,v
retrieving revision 2.88
diff -u -r2.88 res.rc
--- res.rc 6 Aug 2010 18:56:12 -0000 2.88
+++ res.rc 12 Aug 2010 19:27:22 -0000
@@ -433,7 +433,9 @@
LTEXT "Postinstall script errors",IDC_STATIC_HEADER_TITLE
- LTEXT "The following errors occured executing
+ LTEXT "These don't necessarily mean that affected packages will "
+ "fail to function properly, but if you do notice problems "
+ "please check /var/log/setup.log.full.",
EDITTEXT IDC_POSTINSTALL_EDIT,7,41,303,124,WS_VSCROLL | WS_HSCROLL |
ES_LEFT | ES_MULTILINE | ES_READONLY | ES_AUTOHSCROLL |
RCS file: /cvs/cygwin-apps/setup/postinstallresults.cc,v
retrieving revision 1.1
diff -u -r1.1 postinstallresults.cc
--- postinstallresults.cc 29 Jul 2010 13:09:04 -0000 1.1
+++ postinstallresults.cc 12 Aug 2010 19:27:22 -0000
@@ -54,15 +54,6 @@
- // one or more postinstall scripts failed to run successfully
- // installation may be broken
- MessageBox (NULL,
- "You will need to investigate and correct these errors "
- "before your Cygwin installation will function properly.\n"
- "Check setup.log for details.",
- "ERROR - postinstall scripts failed",
- MB_OK | MB_ICONERROR | MB_SETFOREGROUND | MB_TOPMOST);
(Turns out a line break is inserted automatically into the message, so
no need for \n.)