Installed inkscape on the travis-ci vm. #1869

Merged
merged 1 commit into from Mar 30, 2013

Conversation

Projects
None yet
3 participants
@pelson
Member

pelson commented Mar 29, 2013

This change makes the travis-ci run SVG tests, which currently are skipped.

I'm not sure if there was a reason for not including this in the first place (the 180Mb download maybe?), but it seems pretty important to me. Happy to be dissuaded if there was a good reason though 馃槈

@mdboom

This comment has been minimized.

Show comment Hide comment
@mdboom

mdboom Mar 29, 2013

Owner

I think when we first tried this, the version of Inkscape had problems, but clearly it's working now! Great! I see no reason not to do this until the Travis folks tell us our tests are too heavy ;)

Owner

mdboom commented Mar 29, 2013

I think when we first tried this, the version of Inkscape had problems, but clearly it's working now! Great! I see no reason not to do this until the Travis folks tell us our tests are too heavy ;)

@dmcdougall

This comment has been minimized.

Show comment Hide comment
@dmcdougall

dmcdougall Mar 29, 2013

Member

In my opinion, this is a must-have for our tests.

Member

dmcdougall commented Mar 29, 2013

In my opinion, this is a must-have for our tests.

dmcdougall added a commit that referenced this pull request Mar 30, 2013

Merge pull request #1869 from pelson/svg_testing
Installed inkscape on the travis-ci vm.

@dmcdougall dmcdougall merged commit b062b50 into matplotlib:v1.2.x Mar 30, 2013

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment