New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add libRSVG as an alternative to libsvg-cairo #4715

Closed
wants to merge 13 commits into
base: master
from

Conversation

Projects
None yet
1 participant
@tbonfort
Member

tbonfort commented Jul 25, 2013

While implementing svg symbology support, we made the choice to go
with libsvg+libsvg-cairo for parsing and rendering of svg files, as
those were the libraries that had the least dependencies. It turns out
both those libraries are more or less abandonware, resulting in their
absence from a number of distros, and an incompatibility with newer
cairo apis.

Along with release 6.4, I propose to add an SVG symbol implementation
based on librsvg [ https://wiki.gnome.org/LibRsvg ], with the
following consequences:

  • the user can decide at compile time wether to use libsvgcairo or librsvg
  • librsvg is maintained and present in distros, being a core component of gnome
  • librsvg has a number of dependencies, making its compilation/usage
    on non-standard platforms possibly problematic (c.f. previous point:
    but it is distributed by distros).
  • librsvg is GPL, with all that that might imply for people releasing
    mapserver binaries.

This seems like the only way out if we want to continue supporting SVG
symbols in the longer run. I understand the limitations, however the
user still has the option to fall back to the current situation with
libsvg-cairo.

@ghost ghost assigned tbonfort Jul 25, 2013

tbonfort added a commit that referenced this pull request Jul 26, 2013

@tbonfort

This comment has been minimized.

Show comment
Hide comment
@tbonfort

tbonfort Jul 27, 2013

Member

rebased and applied to master in 731178b, closing

Member

tbonfort commented Jul 27, 2013

rebased and applied to master in 731178b, closing

@tbonfort tbonfort closed this Jul 27, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment