<p dir="ltr">-1<br>
I see no compelling reason to ask Even to spend more of his time supporting his script on win32 given that it can be ran offline and/or on a free/libre VM.<br>
Thank you Even for handling this ingrate task.</p>
<p dir="ltr">--<br>
Thomas</p>
<div class="gmail_quote">On Feb 2, 2016 1:49 PM, "Jeff McKenna" <<a href="mailto:jmckenna@gatewaygeomatics.com">jmckenna@gatewaygeomatics.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 2016-02-02 1:35 PM, Jeff McKenna wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 2016-02-02 12:32 PM, Even Rouault wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
I've added in a branch ( <a href="https://github.com/mapserver/docs/pull/143" rel="noreferrer" target="_blank">https://github.com/mapserver/docs/pull/143</a> ) an<br>
interactive script ./scripts/fix_typos.sh that helps identifying<br>
common typo<br>
fixes, and the fixes it helped finding. This uses the codespell Python<br>
script (<br>
upstream is <a href="https://github.com/lucasdemarchi/codespell" rel="noreferrer" target="_blank">https://github.com/lucasdemarchi/codespell</a> . Actual<br>
version is my<br>
forked version with a few improvements ) that comes with its own<br>
dictionary.<br>
I've also added the Debian Lintian dictionary and the QGIS dictionary.<br>
The<br>
dictionaries are more code oriented than general purpose I guess, but<br>
there is<br>
a good amount of computerese in our docs, so it is still valuable.<br>
<br>
It requires Python 3 (and the first time it is run, git to download<br>
codespell<br>
and curl to download the additional dictionaries)<br>
<br>
You can use with : ./scripts/fix_typos.sh<br>
and answering the question when a typo is identified.<br>
<br>
A white list of words can be edited in ./scripts/fix_typos.sh itself,<br>
and when<br>
a word is OK only in a specific context, you can add the full line to<br>
whitelist<br>
in scripts/typos_whitelist.txt (sensitive to starting and trailing<br>
whitespace/tabulations, and only work on LF terminated files, hence a<br>
conversion of 2 files from CRLF to LF).<br>
<br>
Note: it is only run on en/ files, the dictionaries being only for<br>
English.<br>
<br>
Opinions ?<br>
<br>
Even<br>
<br>
</blockquote>
<br>
<br>
Sounds great. When you get to documenting the steps (likely in<br>
/development/documentation.txt) be sure to include steps for Windows<br>
users as well, as many doc contributors are on Windows.  Also, it would<br>
be great to include steps for Python2 users, as I know the MS4W<br>
environment was recently upgraded to Python 2.7.11 support. (yes I ask a<br>
lot, but, you asked for my opinion :)<br>
<br>
Thanks!<br>
<br>
-jeff<br>
<br>
<br>
<br>
</blockquote>
<br>
Maybe the answers to both of my questions/requests is to set this up on a shared server, and then document the steps to login and run.<br>
<br>
-jeff<br>
<br>
<br>
<br>
-- <br>
Jeff McKenna<br>
MapServer Consulting and Training Services<br>
<a href="http://www.gatewaygeomatics.com/" rel="noreferrer" target="_blank">http://www.gatewaygeomatics.com/</a><br>
_______________________________________________<br>
mapserver-dev mailing list<br>
<a href="mailto:mapserver-dev@lists.osgeo.org" target="_blank">mapserver-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/mapserver-dev" rel="noreferrer" target="_blank">http://lists.osgeo.org/mailman/listinfo/mapserver-dev</a></blockquote></div>