reopen 663417 tags 663417 - patch thnanks On Saturday 02 June 2012 11:12:10 Thijs Kinkhorst wrote: > On Tue, March 13, 2012 21:22, Stefan Fritsch wrote: > > On Sunday 11 March 2012, you wrote: > >> diffindex-download calls url_not_found when looking for the index > >> file and when downloading the Contents file, resulting in double > >> requests. > >> > >> You could just pass --fail to curl and look for the exit code 22. > >> The attached patch does that. It is a bit ugly, though. > > > > According to the curl docs, using --fail treats all 4xx errors the > > same. The idea of the double request was to treat 404 differently. > > Other 4xx errors should not be ignored. > > Closing this bug then, as it was designed this way.
Er, I beg to differ. If the patch isn't appropriate then so be it, but the bug still exists: it uselessly sends two requests. One way to do it would be to pass -s and parse the status line, or use LWP. Regards, -- Raphael Geissert - Debian Developer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
