Opened 14 years ago
Last modified 14 years ago
#3344 assigned defect
Fix build problems with AGG/SVG
Reported by: | theduckylittle | Owned by: | theduckylittle |
---|---|---|---|
Priority: | normal | Milestone: | 6.0 release |
Component: | MapServer C Library | Version: | svn-trunk (development) |
Severity: | normal | Keywords: | |
Cc: | dmorissette |
Description
Code needs updated to allow AGG/SVG to be chosen by the user:
- IFDEF the SVG Code
- Add appropriate autoconf flags
- Rebuild configure
- Add nmake.opt flags for Windows
Change History (8)
comment:1 by , 14 years ago
Owner: | changed from | to
---|
comment:2 by , 14 years ago
Status: | new → assigned |
---|
comment:3 by , 14 years ago
I have a problem building mapserver r9907 related to missing expat dependency. It is not included through gdal automatically.
error: renderers/agg/src/agg_svg_parser.cpp:24:19: error: expat.h: No such file or dire
comment:4 by , 14 years ago
Cc: | added |
---|
comment:5 by , 14 years ago
comment:6 by , 14 years ago
In r9874 (comment:2), theduckylittle's added a --with-agg-svg-symbols[=yes/no]
configure switch, but there was a little bug: if the option was not provided at all, a yes was assumed. I just fixed it in r9908 to go the other way around: do not assume a yes unless explicitly requested (also did a bit a cleanup while I was at it).
Note that before we can call this ticket fixed we need to provide a --with-expat switch to allow specifying the location of the expat lib if it's not in the default system dirs.
comment:7 by , 14 years ago
I set it to default itself in the same way as AGG. C'est la vie.
danmo, did you want to add the --with-expat or did you want me to do it? I might have some time tonight...
comment:8 by , 14 years ago
Since the SVG symbol stuff comes with a dependency we couldn't enable it by default, unless that dependency is checked and met first.
Please go ahead with the --with-expath switch if you have time, I don't think I'd have time in the near future anyway
sdlime reported that without linking to GDAL, the -lexpat requirement was not met. I've added the checks to the configure scripts to support detecting whether -lexpat is necessary.