<div dir="ltr">Hi All<br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jan 6, 2018 at 11:21 AM, Even Rouault <span dir="ltr"><<a href="mailto:even.rouault@spatialys.com" target="_blank">even.rouault@spatialys.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><u></u>
<div style="font-family:"Sans Serif";font-size:9pt;font-weight:400;font-style:normal"><span class="gmail-">
<p style="margin:0px;text-indent:0px">> Also do you think</p>
<p style="margin:0px;text-indent:0px">> <a href="https://trac.osgeo.org/gdal/browser/branches/2.2/autotest/gdrivers/ecw.py" target="_blank">https://trac.osgeo.org/gdal/<wbr>browser/branches/2.2/autotest/<wbr>gdrivers/ecw.py</a></p>
<p style="margin:0px;text-indent:0px">> is setup to test the 3.3 SDK nicely? If so I can verify the patch on LINUX</p>
<p style="margin:0px;text-indent:0px">> and MacOSX with that.</p>
<p style="margin:0px;text-indent:0px"> </p>
</span><p style="margin:0px;text-indent:0px">The tests pass fine (*) for me on Linux with ECW SDK 3.3 + libecwj2-3.3.patch. No idea on Mac</p>
<p style="margin:0px;text-indent:0px"> </p></div></blockquote><div><br></div><div>I finally got around to doing testing on MacOSX. Using the MacOSX patches from John and the libecwj2-3.3.patch I was able to compile and successfully test the ECW driver with GDAL 2.2.3 on MacOSX 10.13.2. Test results below:</div><div><br></div><div>python ecw.py</div><div>  TEST: ecw_init ... success</div><div>  TEST: ecw_1 ... success</div><div>  TEST: ecw_2 ... success</div><div>  TEST: ecw_3 ... success</div><div>  TEST: ecw_4 ... success</div><div>  TEST: ecw_5 ... success</div><div>  TEST: ecw_6 ... success</div><div>  TEST: ecw_7 ... Warning 6: NITF only supports WGS84 geographic and UTM projections.</div><div><br></div><div>success</div><div>  TEST: ecw_8 ... success</div><div>  TEST: ecw_9 ... success</div><div>  TEST: ecw_10 ... success</div><div>  TEST: ecw_11 ... success</div><div>  TEST: ecw_12 ... success</div><div>  TEST: ecw_13 ... success</div><div>  TEST: ecw_14 ... success</div><div>  TEST: ecw_15 ... success</div><div>  TEST: ecw_16 ... success</div><div>  TEST: ecw_17 ... Diff at pixel (0, 0) : -5.000000</div><div>Diff at pixel (1, 0) : 1.000000</div><div>Diff at pixel (2, 0) : -2.000000</div><div>Diff at pixel (3, 0) : -1.000000</div><div>Diff at pixel (4, 0) : -3.000000</div><div>Diff at pixel (5, 0) : -3.000000</div><div>Diff at pixel (6, 0) : -1.000000</div><div>Diff at pixel (9, 0) : 2.000000</div><div>Diff at pixel (10, 0) : -3.000000</div><div>Diff at pixel (13, 7) : -6.000000</div><div>Max diff : 6</div><div>Number of diffs : 302</div><div>success</div><div>  TEST: ecw_18 ... success</div><div>  TEST: ecw_19 ... success</div><div>  TEST: ecw_20 ... success</div><div>  TEST: ecw_21 ... success</div><div>  TEST: ecw_22 ... success</div><div>  TEST: ecw_23 ... success</div><div>  TEST: ecw_24 ... success</div><div>  TEST: ecw_25 ... success</div><div>  TEST: ecw_26 ... success</div><div>  TEST: ecw_27 ... success</div><div>  TEST: ecw_28 ... success</div><div>  TEST: ecw_29 ... success</div><div>  TEST: ecw_30 ... success</div><div>  TEST: ecw_31 ... skip</div><div>  TEST: ecw_32 ... success</div><div>  TEST: ecw_33 ... success</div><div>  TEST: ecw_33_bis ... success</div><div>  TEST: ecw_34 ... skip</div><div>  TEST: ecw_35 ... success</div><div>  TEST: ecw_36 ... skip</div><div>  TEST: ecw_37 ... skip</div><div>  TEST: ecw_38 ... skip</div><div>  TEST: ecw_39 ... skip</div><div>  TEST: ecw_40 ... ERROR 1: An error has occurred: Error 86 "File is invalid or corrupt"  file "" line 0</div><div>ERROR 1: Cannot open data/stefan_full_rgba_ecwv3_meta.ecw which looks like a ECW format v3 file, that requires ECW SDK 5.0 or later</div><div>skip</div><div>  TEST: ecw_41 ... skip</div><div>  TEST: ecw_42 ... skip</div><div>  TEST: ecw_43 ... success</div><div>  TEST: ecw_44 ... skip</div><div>  TEST: ecw_45 ... success</div><div>  TEST: ecw_46 ... success</div><div>  TEST: ecw_47 ... skip</div><div>  TEST: ecw_48 ... success</div><div>  TEST: ecw_49 ... success</div><div>  TEST: ecw_online_1 ... As GDAL_DOWNLOAD_TEST_DATA environment variable is not defined, some tests relying on data to downloaded from the Web will be skipped</div><div>skip</div><div>  TEST: ecw_online_2 ... skip</div><div>  TEST: ecw_online_4 ... skip</div><div>  TEST: ecw_online_5 ... skip</div><div>  TEST: ecw_online_6 ... success</div><div>  TEST: ecw_online_7 ... skip</div><div>  TEST: ecw_cleanup ... success</div><div><br></div><div>Test Script: ecw</div><div>Succeeded: 42</div><div>Failed:    0 (0 blew exceptions)</div><div>Skipped:   16</div><div>Expected fail:0</div><div>Duration:  12.09s</div><div>As GDAL_DOWNLOAD_TEST_DATA environment variable is not defined, 5 tests relying on data to downloaded from the Web have been skipped</div><div><br></div><div>The combined patch is here: <a href="https://gist.github.com/palmerj/6035fa41b3c30ab573c12741012ab011">https://gist.github.com/palmerj/6035fa41b3c30ab573c12741012ab011</a> </div><div><br></div><div>Note I've also successfully built and tested the combined patch on Ubuntu 14.04 and 16.06 with GDAL 1.11 and GDAL 2.2.3 :)</div><div><br></div><div>Cheers</div><div>Jeremy </div></div></div></div>