This is the mail archive of the cygwin-apps@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [setup PATCH] Add some log msgs to installOneSource


Robert Collins wrote:
> On Wed, 2003-07-16 at 18:44, Max Bowsher wrote:
>> 2003-07-16  Max Bowsher  <maxb@ukf.net>
>>
>>         * install.cc (Installer::installOneSource): Add some log
messages,
>> as
>>         a stopgap until we add some proper error handling here.
>>
>>
>> --- install.cc 25 Mar 2003 20:57:13 -0000 2.61
>> +++ install.cc 16 Jul 2003 08:21:39 -0000
>> @@ -227,7 +227,14 @@ Installer::installOneSource (packagemeta
>>        if (tmp2)
>>   thefile = archive::extract (tmp2);
>>        else
>> - thefile = archive::extract (tmp);
>> +        {
>> +          log(LOG_PLAIN, String("Failed to decompress: ") +
>> source.Cached() );
>> +          thefile = archive::extract (tmp);
>
> This isn't an error. It's simply the automagic compressed archive
> detection.

The message could be clearer, but it is valuable info to know whether the
docompress or the dearchive failed. How about:

if (tmp) {
/* Existing code */
} else {
if (!tmp2) { log ("Decompression failed. Maybe source not compressed?"; }
else { log ("De-archiving failed.");
}

Is that ok?

>> +        }
>> +    }
>> +  else
>> +    {
>> +      log(LOG_PLAIN, String("Failed to open: ") + source.Cached() );
>>      }
>
> this is redundant: the if (thefile) check covers this case, as thefile
> is null on error.

Yes, but the additional message shows you *how* it failed.

Max.



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]