[Fedora-livecd-list] [PATCH] print out what urls are being downloaded, more verbose yum
Tim Wood
tim.wood at datawranglers.com
Wed Aug 8 17:26:18 UTC 2007
FWIW, this would really help troubleshooting.
Tim
On Aug 8, 2007, at 10:36 AM, Colin Walters wrote:
> On Wed, 2007-08-08 at 12:17 -0400, Jeremy Katz wrote:
>> On Tue, 2007-08-07 at 17:44 -0400, Colin Walters wrote:
>>> +class TextProgress(object):
>>> + def start(self, filename, url, *args, **kwargs):
>>> + print "Retrieving %s" % (url,)
>>> + self.url = url
>>> + def update(self, *args):
>>> + pass
>>> + def end(self, *args):
>>> + print "Retrieved %s OK" % (self.url,)
>>
>> Why two lines for every package? Seems like we should be able to
>> just
>> have the retrieving when it starts (and no newline) and then "OK"
>> when
>> it completes.
>
> Mainly because that creates the assumption that it will be the only
> function showing output during that time.
>
>>> @@ -189,7 +198,7 @@ class LiveCDYum(yum.YumBase):
>>> conf += "installroot=%s\n" % installroot
>>> conf += "cachedir=/var/cache/yum\n"
>>> conf += "plugins=0\n"
>>> - conf += "debuglevel=2\n"
>>> + conf += "debuglevel=3\n"
>>> conf += "reposdir=\n"
>>
>> What does debuglevel of 3 vs 2 give you? 2 is the default with
>> yum, and
>> I don't really know that making it higher for livecd-tools makes
>> sense.
>> It probably does, though, make sense to think about a --quiet and
>> --verbose as we get more things which output more.
>
> This change should probably not be included; I don't remember now
> why I
> did that.
>
> Anyways, this patch isn't really important; the only one I actually
> care
> about is the caching patch.
>
>
> --
> Fedora-livecd-list mailing list
> Fedora-livecd-list at redhat.com
> https://www.redhat.com/mailman/listinfo/fedora-livecd-list
>
More information about the Fedora-livecd-list
mailing list