2008/7/11 Denys Vlasenko <[EMAIL PROTECTED]>:
> On Friday 11 July 2008 17:25, Alex Owen wrote:
>> My parsing code can therefore look for:
>> "server returned error: HTTP/1.1 404"
>
> Or better "HTTP/nums_and_dots 404"

Yes that would be better thanks.

> Wait a second. What is "debian installer"?
Debian-installer is the code that runs to install a debian system.
I some times tinker with it and submit patches.

> Are you saying
> there is some part of Debian distribution
> which works with normal wget but fails with busybox wget?
_No_ not as far as I'm aware.

I'm interested in a new feature in the debian-installer the initial
implementation tracked here: http://bugs.debian.org/422088

The core of the idea is that we can tell the difference between 3
outcomes when getting a file:
1) the file is fetched ok
2) the server says 404
3) any other error (perhaps network failure)

I have submitted a bug to normal wget asking for them to implement
exit status error codes to identify 404 errors.... this is tracked
here:
 https://savannah.gnu.org/bugs/index.php?20333
time frame wget ~1.13
If/when they do implement exit status I will then ask busybox to do the same!


In the mean time we parse busybox-wget output to find 404 errors.
the version of busybox-wget used by debian-installer must have changed
since the implementation..

In version 1.1.3 of busybox-wget  the output came from a line looking like:
  close_delete_and_die("server returned error %d: %s", atoi(s), buf)

$ busybox --help | grep BusyBox
BusyBox v1.1.3 (Debian 1:1.1.3-4) multi-call binary
$ busybox wget http://www.space.com/bogondsfas
wget: server returned error 404: HTTP/1.1 404 Not Found


In version 1.10.2 of busybox-wget  the output comes from a line looking like:
  bb_error_msg_and_die("server returned error: %s", buf);

$ busybox --help | grep BusyBox
BusyBox v1.9.2 (Debian 1:1.9.2-3) multi-call binary
$ busybox wget http://www.space.com/bogondsfas
wget: server returned error: HTTP/1.1 404 Not Found



The status code before the ":" was easy to test for and the change
means we need to update our match pattern.

Anyway... this is getting a bit off topic for the busybox mailing list...
Sorry for that.. but thanks for you help and good ideas!

Alex Owen
_______________________________________________
busybox mailing list
[email protected]
http://busybox.net/cgi-bin/mailman/listinfo/busybox

Reply via email to