Comment 2 for bug 1175665

Revision history for this message
Thomi Richards (thomir-deactivatedaccount) wrote : Re: [Bug 1175665] Re: Autopilot xml dependancy should be optional

Yeah, we need to do something better than raise an ImportError. At least
print a nice error message.

However, since we're moving to subunit, this will go away pretty soon
anyway.

Cheers,

On Fri, Oct 11, 2013 at 8:55 AM, Corey Goldberg
<email address hidden>wrote:

> right now it only imports `junitxml` if you select xml output. so the
> package isn't needed (unless you want xml output obviously).
>
> you will get an ImportError. Is that good enough, or should it quit
> gracefully asking you to install `junitxml` to enable xml output?
>
>
> ----
>
> def get_output_format(format):
> ....
> elif format == "xml":
> from junitxml import JUnitXmlResult
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1175665
>
> Title:
> Autopilot xml dependancy should be optional
>
> Status in Autopilot:
> Triaged
>
> Bug description:
> We should make the Xml output optional - only offer it if the junitxml
> package is installed, to reduce the number of packages we need to
> install on the tablet.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/autopilot/+bug/1175665/+subscriptions
>

--
Thomi Richards
<email address hidden>