[Yum-devel] urlgrabber: minor bug in latest cvs

seth vidal skvidal at phy.duke.edu
Mon Feb 21 05:06:31 UTC 2005


urlgrabber/grabber.py starting at line 908:

it reads:
            try:
                new = self.fo.read(readamount)
            except socket.error:
                raise URLGrabError(4, _('Socket Error: %s') % (e, ))

but it needs to read:
            try:
                new = self.fo.read(readamount)
            except socket.error, e:
                raise URLGrabError(4, _('Socket Error: %s') % (e, ))

the ', e' needs to be there so the socket error can be expressed. I
fixed it before in yum's pull of urlgrabber but I think this should be
fixed in urlgrabber cvs.

can one of y'all patch that in?

diff for what I did before is here:
http://devel.linux.duke.edu/cgi-bin/viewcvs.cgi/yum/urlgrabber/grabber.py?r1=1.7&r2=1.8&makepatch=1&diff_format=h

it's utterly trivial.

-sv







More information about the Yum-devel mailing list