| 1 | = !MapGuide RFC 174 - Remove GD Renderer = |
| 2 | |
| 3 | This page contains a change request (RFC) for the !MapGuide Open Source project. |
| 4 | More !MapGuide RFCs can be found on the [wiki:MapGuideRfcs RFCs] page. |
| 5 | |
| 6 | == Status == |
| 7 | |
| 8 | ||RFC Template Version||(1.0)|| |
| 9 | ||Submission Date|||| |
| 10 | ||Last Modified|||| |
| 11 | ||Author||Jackie Ng|| |
| 12 | ||RFC Status|||| |
| 13 | ||Implementation Status|||| |
| 14 | ||Proposed Milestone||4.0|| |
| 15 | ||Assigned PSC guide(s)||(when determined)|| |
| 16 | ||'''Voting History'''||(vote date)|| |
| 17 | ||+1|||| |
| 18 | ||+0|||| |
| 19 | ||-0|||| |
| 20 | ||-1|||| |
| 21 | ||no vote|| || |
| 22 | |
| 23 | == Overview == |
| 24 | |
| 25 | This RFC proposes to remove the GD image renderer support from MapGuide. |
| 26 | |
| 27 | == Motivation == |
| 28 | |
| 29 | MapGuide current has 2 image-based map renderers: |
| 30 | |
| 31 | * One using the GD library |
| 32 | * One using Anti-Grain Geometry (AGG) library |
| 33 | |
| 34 | Since MapGuide Open Source 2.0, the AGG renderer has been the designated default image-based map renderer. |
| 35 | |
| 36 | Although a {{{serverconfig.ini}}} setting exists to change this over to GD, it is unlikely that many users are actually aware of this setting (and would be using the GD renderer as a result). |
| 37 | |
| 38 | To simplify the burden of maintenance and future image rendering enhancement work, we should focus on only a single image renderer going forward: The AGG renderer. |
| 39 | |
| 40 | == Proposed Solution == |
| 41 | |
| 42 | Remove the GD renderer from the {{{Renderers}}} project. |
| 43 | |
| 44 | Where renderer names were being checked in the rendering service to determine GD/AGG renderer, remove such code paths and always assume the AGG renderer. |
| 45 | |
| 46 | Remove the {{{Renderer}}} property from {{{serverconfig.ini}}} |
| 47 | |
| 48 | This RFC does not remove our use of the GD library altogether. We will still retain the use of the GD library for image format conversions where needed. |
| 49 | |
| 50 | == Implications == |
| 51 | |
| 52 | N/A |
| 53 | |
| 54 | == Test Plan == |
| 55 | |
| 56 | Ensure all existing rendering tests still pass after GD renderer removal. |
| 57 | |
| 58 | == Funding / Resources == |
| 59 | |
| 60 | Community |