Autopilot xml dependancy should be optional

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

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>