Thanks for the patch

> @@ -154,7 +154,10 @@ def parse_url(url, **urlopen_args):
>     `bitfrost.util.urlrange.urlopen`."""
>     import bitfrost.util.urlrange as urlrange
>     with urlrange.urlopen(url, **urlopen_args) as f:
> -        return parse_html(f.read(), f.url)
> +       try:
> +            return parse_html(f.read(), f.url)
> +       except urllib2.HTTPError:
> +            return None, None, []

Whats the rationale here?
If an invalid activity group URL is passed, olpc-os-builder should
exit with error.
I don't see the value in silently continuing and producing a build
with missing activities.

Daniel
_______________________________________________
Devel mailing list
Devel@lists.laptop.org
http://lists.laptop.org/listinfo/devel

Reply via email to