[PATCH] inform user if any postinstall script failed to run

[Jon TURNEY]

On 27/08/2010 19:33, Christopher Faylor wrote:
> On Fri, Aug 27, 2010 at 06:15:38PM +0100, Jon TURNEY wrote:
>> On 29/07/2010 17:28, Jon TURNEY wrote:
>>> On 28/07/2010 15:58, Christopher Faylor wrote:
>>>> On Wed, Jul 28, 2010 at 03:25:17PM +0100, Jon TURNEY wrote:
>>>>> Anyhow, here's another attempt, which unfortunately changes rather more than I
>>>>> wanted to. It adds a new page, which is displayed if any script failed, and
>>>>> reports which packages and scripts failed.
>>>>
>>>> That is great. Please check in (with a ChangeLog of course).
>>
>> Due to the way I tested this change, I'd failed to notice that when a package
>> is installed with a failing postinstall script, this will list the failing
>> script twice, once with the package name and once as 'no package'.
>>
>> Attached is a patch to remedy that.
>
> Do we realy need a separate for loop for this?  Couldn't we just piggy
> back on the previous for loop?

Sure.  I'm not sure if it's any more elegant, though :-)



-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: setup_report_postinstall_failure_once_v2.patch
URL: <http://cygwin.com/pipermail/cygwin-apps/attachments/20100828/f5d49432/attachment.ksh>