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
[email protected]
https://www.redhat.com/mailman/listinfo/fedora-livecd-list


--
Fedora-livecd-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/fedora-livecd-list

Reply via email to