<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=utf-8"><meta name=Generator content="Microsoft Word 15 (filtered medium)"><style><!--
/* Font Definitions */
@font-face
{font-family:Wingdings;
panose-1:5 0 0 0 0 0 0 0 0 0;}
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:Consolas;
panose-1:2 11 6 9 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
pre
{mso-style-priority:99;
mso-style-link:"HTML Preformatted Char";
margin:0in;
font-size:10.0pt;
font-family:"Courier New";}
span.HTMLPreformattedChar
{mso-style-name:"HTML Preformatted Char";
mso-style-priority:99;
mso-style-link:"HTML Preformatted";
font-family:"Consolas",serif;}
span.EmailStyle22
{mso-style-type:personal-reply;
font-family:"Calibri",sans-serif;
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
/* List Definitions */
@list l0
{mso-list-id:671687326;
mso-list-template-ids:-1953466214;}
@list l0:level1
{mso-level-number-format:bullet;
mso-level-text:;
mso-level-tab-stop:.5in;
mso-level-number-position:left;
text-indent:-.25in;
mso-ansi-font-size:10.0pt;
font-family:Symbol;}
@list l0:level2
{mso-level-number-format:bullet;
mso-level-text:o;
mso-level-tab-stop:1.0in;
mso-level-number-position:left;
text-indent:-.25in;
mso-ansi-font-size:10.0pt;
font-family:"Courier New";
mso-bidi-font-family:"Times New Roman";}
@list l0:level3
{mso-level-number-format:bullet;
mso-level-text:;
mso-level-tab-stop:1.5in;
mso-level-number-position:left;
text-indent:-.25in;
mso-ansi-font-size:10.0pt;
font-family:Wingdings;}
@list l0:level4
{mso-level-number-format:bullet;
mso-level-text:;
mso-level-tab-stop:2.0in;
mso-level-number-position:left;
text-indent:-.25in;
mso-ansi-font-size:10.0pt;
font-family:Wingdings;}
@list l0:level5
{mso-level-number-format:bullet;
mso-level-text:;
mso-level-tab-stop:2.5in;
mso-level-number-position:left;
text-indent:-.25in;
mso-ansi-font-size:10.0pt;
font-family:Wingdings;}
@list l0:level6
{mso-level-number-format:bullet;
mso-level-text:;
mso-level-tab-stop:3.0in;
mso-level-number-position:left;
text-indent:-.25in;
mso-ansi-font-size:10.0pt;
font-family:Wingdings;}
@list l0:level7
{mso-level-number-format:bullet;
mso-level-text:;
mso-level-tab-stop:3.5in;
mso-level-number-position:left;
text-indent:-.25in;
mso-ansi-font-size:10.0pt;
font-family:Wingdings;}
@list l0:level8
{mso-level-number-format:bullet;
mso-level-text:;
mso-level-tab-stop:4.0in;
mso-level-number-position:left;
text-indent:-.25in;
mso-ansi-font-size:10.0pt;
font-family:Wingdings;}
@list l0:level9
{mso-level-number-format:bullet;
mso-level-text:;
mso-level-tab-stop:4.5in;
mso-level-number-position:left;
text-indent:-.25in;
mso-ansi-font-size:10.0pt;
font-family:Wingdings;}
ol
{margin-bottom:0in;}
ul
{margin-bottom:0in;}
--></style></head><body lang=EN-US link=blue vlink=purple style='word-wrap:break-word'><div class=WordSection1><p class=MsoNormal>Also +1<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Mike<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><div style='border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in'><p class=MsoNormal><b><span style='font-size:12.0pt;color:black'>From: </span></b><span style='font-size:12.0pt;color:black'>mapserver-dev <mapserver-dev-bounces@lists.osgeo.org> on behalf of Even Rouault <even.rouault@spatialys.com><br><b>Date: </b>Tuesday, March 30, 2021 at 5:02 PM<br><b>To: </b>Seth G <sethg@geographika.co.uk>, Steve Lime <sdlime@gmail.com><br><b>Cc: </b>MapServer Dev Mailing List <mapserver-dev@lists.osgeo.org><br><b>Subject: </b>Re: [mapserver-dev] MS RFC 133: Mapfile Syntax Cleanup<o:p></o:p></span></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><p>I'm +1 on the RFC<o:p></o:p></p><p>Even<o:p></o:p></p><div><p class=MsoNormal>Le 30/03/2021 à 22:57, Seth G a écrit :<o:p></o:p></p></div><blockquote style='margin-top:5.0pt;margin-bottom:5.0pt'><div><p class=MsoNormal>Hi Steve,<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>I'm not sure where I got CLASS MAXSCALE from - it is not a property of the classObj or referenced anywhere in the sourcecode/tests. I'll remove from the RFC. <o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>Anyone else on the PSC more votes on this?<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>I'm tempted to merge <a href="https://github.com/MapServer/MapServer/pull/6264/">https://github.com/MapServer/MapServer/pull/6264/</a> (this is mainly Steve's work, and then I updated the msautotests), it touches a lot of files so may need to be rebased quite often if not merged soon.<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>Seth<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div id=sig62266145><div><p class=MsoNormal>--<o:p></o:p></p></div><div><p class=MsoNormal>web:<a href="http://geographika.co.uk">http://geographika.co.uk</a><o:p></o:p></p></div><div><p class=MsoNormal>twitter: @geographika<o:p></o:p></p></div></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>On Mon, Mar 29, 2021, at 4:54 PM, Steve Lime wrote:<o:p></o:p></p></div><blockquote style='margin-top:5.0pt;margin-bottom:5.0pt' id=qt><div><div><p class=MsoNormal>Hi Seth: In general I'm a strong +1 on this, anything that simplifies the code base is a good thing.. Only one I wonder about is <i>CLASS MAXSCALE</i> since MINSCALE is currently supported as well? Personally I'm selfish and I really like the MINSCALE/MAXSCALE aliases so retaining those across the board would save me a ton of work when 8.0 arrives.<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>--Steve<o:p></o:p></p></div></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><div><p class=MsoNormal>On Tue, Mar 16, 2021 at 6:08 PM Seth G <<a href="mailto:sethg@geographika.co.uk">sethg@geographika.co.uk</a>> wrote:<o:p></o:p></p></div><blockquote style='border:none;border-left:solid windowtext 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in;border-left-color:rgb(204,
204,
204)'><div><p class=MsoNormal><o:p> </o:p></p></div><div><div><p class=MsoNormal>Hi all,<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>I've made a start of implementing RFC 133 [1] by removing the now redundant LAYER DUMP keyword [2].<o:p></o:p></p></div><div><p class=MsoNormal>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. <o:p></o:p></p></div><div><p class=MsoNormal>Any objections to specific keywords being removed can be raised on individual pull requests. <o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>Seth<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>[1] <a href="https://mapserver.org/development/rfc/ms-rfc-133.html" target="_blank">https://mapserver.org/development/rfc/ms-rfc-133.html</a><o:p></o:p></p></div><div><p class=MsoNormal>[2] <a href="https://github.com/MapServer/MapServer/pull/6264" target="_blank">https://github.com/MapServer/MapServer/pull/6264</a><o:p></o:p></p></div><div id="qt-gmail-m_-994561705765618940sig62266145"><div><p class=MsoNormal>--<o:p></o:p></p></div><div><p class=MsoNormal>web:<a href="http://geographika.co.uk" target="_blank">http://geographika.co.uk</a><o:p></o:p></p></div><div><p class=MsoNormal>twitter: @geographika<o:p></o:p></p></div></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>On Tue, Feb 2, 2021, at 11:18 PM, Seth G wrote:<o:p></o:p></p></div><blockquote style='margin-top:5.0pt;margin-bottom:5.0pt' id="qt-gmail-m_-994561705765618940qt"><div><p class=MsoNormal>Hi Steve,<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>If you could do a reference pull request I can use this as a template for the other keywords.<o:p></o:p></p></div><div><p class=MsoNormal>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"...<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>Seth<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div id="qt-gmail-m_-994561705765618940qt-sig62266145"><div><p class=MsoNormal>--<o:p></o:p></p></div><div><p class=MsoNormal>web:<a href="http://geographika.co.uk" target="_blank">http://geographika.co.uk</a><o:p></o:p></p></div><div><p class=MsoNormal>twitter: @geographika<o:p></o:p></p></div></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>On Tue, Feb 2, 2021, at 12:31 AM, Steve Lime wrote:<o:p></o:p></p></div><blockquote style='margin-top:5.0pt;margin-bottom:5.0pt' id="qt-gmail-m_-994561705765618940qt-qt"><div><div><p class=MsoNormal>I don't recall any keyword removals specifically. Should have to touch:<o:p></o:p></p></div><div><ul type=disc><li class=MsoNormal style='mso-margin-top-alt:auto;mso-margin-bottom-alt:auto;mso-list:l0 level1 lfo1'>maplexer.l to remove keywords from the grammar <o:p></o:p></li><li class=MsoNormal style='mso-margin-top-alt:auto;mso-margin-bottom-alt:auto;mso-list:l0 level1 lfo1'>mapfile.c/mapfile.h to remove read/write cases and symbols<o:p></o:p></li><li class=MsoNormal style='mso-margin-top-alt:auto;mso-margin-bottom-alt:auto;mso-list:l0 level1 lfo1'>mapserver.h to remove structure members<o:p></o:p></li></ul><div><p class=MsoNormal>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...<o:p></o:p></p></div></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>--Steve<o:p></o:p></p></div></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><div><p class=MsoNormal>On Fri, Jan 29, 2021 at 3:06 PM Seth G <<a href="mailto:sethg@geographika.co.uk" target="_blank">sethg@geographika.co.uk</a>> wrote:<o:p></o:p></p></div><blockquote style='border:none;border-left:solid windowtext 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in;border-left-color:rgb(204,
204,
204)'><div><p class=MsoNormal>Hi all,<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>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. <o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal><a href="https://mapserver.org/development/rfc/ms-rfc-133.html" target="_blank">https://mapserver.org/development/rfc/ms-rfc-133.html</a><o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>It might be too early to vote on the RFC without some discussion about which keywords should be removed. <o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>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?<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>Thoughts/comments welcome,<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>Seth<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>--<o:p></o:p></p></div><div><p class=MsoNormal>web:<a href="http://geographika.co.uk" target="_blank">http://geographika.co.uk</a><o:p></o:p></p></div><div><p class=MsoNormal>twitter: @geographika<o:p></o:p></p></div><div><p class=MsoNormal>_______________________________________________<o:p></o:p></p></div><div><p class=MsoNormal>mapserver-dev mailing list<o:p></o:p></p></div><div><p class=MsoNormal><a href="mailto:mapserver-dev@lists.osgeo.org" target="_blank">mapserver-dev@lists.osgeo.org</a><o:p></o:p></p></div><div><p class=MsoNormal><a href="https://lists.osgeo.org/mailman/listinfo/mapserver-dev" target="_blank">https://lists.osgeo.org/mailman/listinfo/mapserver-dev</a><o:p></o:p></p></div></blockquote></div></blockquote><div><p class=MsoNormal><o:p> </o:p></p></div></blockquote><div><p class=MsoNormal><o:p> </o:p></p></div></div><div><p class=MsoNormal>_______________________________________________<o:p></o:p></p></div><div><p class=MsoNormal>mapserver-dev mailing list<o:p></o:p></p></div><div><p class=MsoNormal><a href="mailto:mapserver-dev@lists.osgeo.org" target="_blank">mapserver-dev@lists.osgeo.org</a><o:p></o:p></p></div><div><p class=MsoNormal><a href="https://lists.osgeo.org/mailman/listinfo/mapserver-dev" target="_blank">https://lists.osgeo.org/mailman/listinfo/mapserver-dev</a><o:p></o:p></p></div></blockquote></div></blockquote><div><p class=MsoNormal><o:p> </o:p></p></div><p class=MsoNormal><br><br><o:p></o:p></p><pre>_______________________________________________<o:p></o:p></pre><pre>mapserver-dev mailing list<o:p></o:p></pre><pre><a href="mailto:mapserver-dev@lists.osgeo.org">mapserver-dev@lists.osgeo.org</a><o:p></o:p></pre><pre><a href="https://lists.osgeo.org/mailman/listinfo/mapserver-dev">https://lists.osgeo.org/mailman/listinfo/mapserver-dev</a><o:p></o:p></pre></blockquote><pre>-- <o:p></o:p></pre><pre><a href="http://www.spatialys.com">http://www.spatialys.com</a><o:p></o:p></pre><pre>My software is free, but my time generally not.<o:p></o:p></pre><p class=MsoNormal>_______________________________________________ mapserver-dev mailing list mapserver-dev@lists.osgeo.org https://lists.osgeo.org/mailman/listinfo/mapserver-dev <o:p></o:p></p></div></body></html>