Page MenuHomePhabricator

Upgrade librsvg to 2.14 for better SVG support
Closed, ResolvedPublic

Description

Author: broberg

Description:
The librsvg version used on Wikimedia's servers seems to be pretty outdated.
According to
http://meta.wikimedia.org/wiki/SVG_image_support#Current_implementation the
version used is
pre-Cairo. The first stable librsvg version using Cairo backend was released
2005-09-15 and it works just fine.

The outdated version holds back some SVGs that could replace current PNGs.
Problems include lacking support for dashed lines, lack of support for viewboxes
and some various rendering anomalities. Here are some examples:
http://en.wikipedia.org/wiki/Image:DDC_field.svg vs.
http://en.wikipedia.org/wiki/Image:DDC_field.png

http://en.wikipedia.org/wiki/Image:Cartesian_coordinates_3D.svg vs.
http://en.wikipedia.org/wiki/Image:Cartesian_coordinates_3D.png


Version: unspecified
Severity: enhancement
URL: http://librsvg.sourceforge.net

Details

Reference
bz4976

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 9:05 PM
bzimport set Reference to bz4976.
bzimport added a subscriber: Unknown Object (MLST).

2.3.93 seems to misposition the text on [[Image:Mitochondrie.svg]].
Have to do some more testing...

On the plus side, it seems to be marking a white background color so the
transparent rendered images look ok in IE 6.0.

Filed upstream as:
http://bugzilla.gnome.org/show_bug.cgi?id=330956

Seems to be regression in 2.3.93; 2.3.92 is ok, and reverting a patch there looks ok.

Installed 2.14.0 and forcing all thumbs to regenerate (for this and various other fixed
image bugs). Referenced images look good.

fcartegnie wrote:

Sorry, this still doesn't work for Gradients.
http://bugzilla.wikimedia.org/show_bug.cgi?id=4593

Needs more rsvg lib fixes :(

reopening.

Martin.vGagern wrote:

I just compiled librsvg-2.14.0 locally. The fencing image gets rendered
correctly here. So I'd assume the error is not in librsvg this time but rather
in some underlying lib, maybe cairo or something.

Some information on the libs I have installed here:
cairo-1.0.2: libcairo.so.2.2.3
glib-2.8.6: libg{lib,module,object,thread}-2.0.so.0.800.6
gtk+-2.8.12: libgdk_pixbuf-2.0.so.0.800.12
libpng-1.2.8: libpng12.so.0.1.2.8
pango-1.10.3: libpango{,ft2}-1.0.so.0.1003.0
xorg-6.8.2: lib{X11.so.6.2,Xext.so.6.4,Xrender.so.1.2.2}

There's nothing to upgrade, resolving FIXED. Also please remember that EXISTING
RENDERINGS HAVEN'T ALL BEEN REBUILT YET. TEST NEW SIZES BEFORE MAKING FURTHER COMMENT.

Ok, action=purge can now be used on an image page to force images to rerender.
(Be careful about caching; make sure it reloads the new image!)

phrood wrote:

The current version seems to ignore line widths. For instance,
http://de.wikipedia.org/wiki/Bild:Raytracing.svg should look like
http://de.wikipedia.org/wiki/Bild:Raytracing.png .

broberg wrote:

Regarding #8, I've just submitted a bug about this in librsvg's bug tracker:
http://bugzilla.gnome.org/show_bug.cgi?id=333057.
Before that and other bugs are fixed, there is no upgrade to make, and therefore
this bug should remain "FIXED".

Furthermore there is still a problem with one of the referenced pictures in the
first post:
http://en.wikipedia.org/wiki/Image:DDC_field.svg vs.
http://en.wikipedia.org/wiki/Image:DDC_field.png
(the arrows are too small, I have mailed the librsvg list about this)

I still think this version is much better than the previous one, so thank you
Brion for the upgrade!

Do *NOT* reopen this bug ever again. This bug was for a specific
upgrade, which has been performed.

If there are other problems, open a new bug about them.

kaldari renamed this task from Upgrade librsvg for better SVG support to Upgrade librsvg to 2.14 for better SVG support.Jan 3 2017, 6:42 PM