
<!DOCTYPE html><html><head><title></title><style type="text/css">

p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body><div>Hi all,<br></div><div><br></div><div>I've made a start of implementing RFC 133 [1] by removing the now redundant LAYER DUMP keyword [2].<br></div><div>It would be good to get some votes in before proceeding further in case people have objections to the overall aim of removing deprecated keywords for the 8.0 release. <br></div><div>Any objections to specific keywords being removed can be raised on individual pull requests. <br></div><div><br></div><div>Seth<br></div><div><br></div><div>[1] <a href="https://mapserver.org/development/rfc/ms-rfc-133.html">https://mapserver.org/development/rfc/ms-rfc-133.html</a><br></div><div>[2] <a href="https://github.com/MapServer/MapServer/pull/6264">https://github.com/MapServer/MapServer/pull/6264</a></div><div id="sig62266145"><div class="signature">--<br></div><div class="signature">web:http://geographika.co.uk<br></div><div class="signature">twitter: @geographika<br></div></div><div><br></div><div><br></div><div>On Tue, Feb 2, 2021, at 11:18 PM, Seth G wrote:<br></div><blockquote type="cite" id="qt" style=""><div>Hi Steve,<br></div><div><br></div><div>If you could do a reference pull request I can use this as a template for the other keywords.<br></div><div>I guess it needs a vote before accepting, so I'll start with my +1 - removing ambiguity around keywords will help new MapServer users. I was adding the DUMP keyword to all LAYER configs for several years "just in case"...<br></div><div><br></div><div>Seth<br></div><div><br></div><div id="qt-sig62266145"><div class="qt-signature">--<br></div><div class="qt-signature">web:http://geographika.co.uk<br></div><div class="qt-signature">twitter: @geographika<br></div></div><div><br></div><div><br></div><div>On Tue, Feb 2, 2021, at 12:31 AM, Steve Lime wrote:<br></div><blockquote type="cite" id="qt-qt" style=""><div dir="ltr"><div>I don't recall any keyword removals specifically. Should have to touch:<br></div><div><ul><li>maplexer.l to remove keywords from the grammar <br></li><li>mapfile.c/mapfile.h to remove read/write cases and symbols<br></li><li>mapserver.h to remove structure members<br></li></ul><div>Updating the maplexer.l effectively stops support, the rest is just cleanup. I'll commit to taking a run through all of these in the next day or so. It's a long list...<br></div></div><div><br></div><div>--Steve<br></div></div><div><br></div><div class="qt-qt-gmail_quote"><div dir="ltr" class="qt-qt-gmail_attr">On Fri, Jan 29, 2021 at 3:06 PM Seth G <<a href="mailto:sethg@geographika.co.uk">sethg@geographika.co.uk</a>> wrote:<br></div><blockquote class="qt-qt-gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-color:rgb(204, 204, 204);border-left-style:solid;border-left-width:1px;padding-left:1ex;"><div>Hi all,<br></div><div><br></div><div>I've got round to a draft RFC for cleaning up deprecated and legacy Mapfile keywords, based on current docs and going through all the allowed syntax as part of mappyfile development and ongoing MapScript docs updates. <br></div><div><br></div><div><a href="https://mapserver.org/development/rfc/ms-rfc-133.html" rel="noreferrer" target="_blank">https://mapserver.org/development/rfc/ms-rfc-133.html</a><br></div><div><br></div><div>It might be too early to vote on the RFC without some discussion about which keywords should be removed. <br></div><div><br></div><div>In terms of updates any keyword removal would be a separate pull request. The changes to mapfile.c seem fine (and the msautotest Mapfiles), but I've never used the lexer/parser/Bison process. Are there any sample commits when a Mapfile keyword has been removed previously?<br></div><div><br></div><div>Thoughts/comments welcome,<br></div><div><br></div><div>Seth<br></div><div><br></div><div><br></div><div>--<br></div><div>web:<a href="http://geographika.co.uk" rel="noreferrer" target="_blank">http://geographika.co.uk</a><br></div><div>twitter: @geographika<br></div><div>_______________________________________________<br></div><div>mapserver-dev mailing list<br></div><div><a href="mailto:mapserver-dev@lists.osgeo.org" target="_blank">mapserver-dev@lists.osgeo.org</a><br></div><div><a href="https://lists.osgeo.org/mailman/listinfo/mapserver-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/mapserver-dev</a><br></div></blockquote></div></blockquote><div><br></div></blockquote><div><br></div></body></html>