<div dir="ltr">See: <a href="https://trac.osgeo.org/gdal/ticket/6547">https://trac.osgeo.org/gdal/ticket/6547</a><div class="gmail_signature" data-smartmail="gmail_signature"></div>
<div><br></div><div>I propose to remove the $id line from the comments at the top of the file for files that already have CPL_CVSID. Having two lines means we have a redundant copy of all this info, which is changing for every commit.</div><div><br></div><div>In addition, I propose to set the svn:keywords for these files to just be Id. Some have more and some have none. e.g. </div><div><br></div><div>svn propset svn:keywords Id gdal_rpc.cpp<br></div><div>svn propset svn:keywords Id gdaltransformgeolocs.cpp</div><div>svn commit</div><div><br></div><div><a href="https://trac.osgeo.org/gdal/changeset/34506">https://trac.osgeo.org/gdal/changeset/34506</a><br></div><div><br></div><div><div>grep Id gdal_rpc.cpp gdaltransformgeolocs.cpp</div><div><br></div><div>gdal_rpc.cpp: * $Id: gdal_rpc.cpp 34506 2016-07-01 17:08:58Z goatbar $</div><div>gdal_rpc.cpp:CPL_CVSID("$Id: gdal_rpc.cpp 34506 2016-07-01 17:08:58Z goatbar $");</div><div>gdaltransformgeolocs.cpp: * $Id: gdaltransformgeolocs.cpp 34506 2016-07-01 17:08:58Z goatbar $</div><div>gdaltransformgeolocs.cpp:CPL_CVSID("$Id: gdaltransformgeolocs.cpp 34506 2016-07-01 17:08:58Z goatbar $");</div></div><div><br></div><div>Then removing the redundant line, it becomes:</div><div><br></div><div><div>grep Id gdal_rpc.cpp gdaltransformgeolocs.cpp</div></div><div><br></div><div><div>gdal_rpc.cpp:CPL_CVSID("$Id: gdal_rpc.cpp 34506 2016-07-01 17:08:58Z goatbar $");<br></div><div>gdaltransformgeolocs.cpp:CPL_CVSID("$Id: gdaltransformgeolocs.cpp 34506 2016-07-01 17:08:58Z goatbar $");<br></div></div><div><br></div><div>Why? It's extra churn in diffs/patches and adds yet another line to check for those of use using merge tools like meld a lot (which might just be me).</div><div><br></div><div>Anyone against this? I'm proposing this just for head on trunk.</div><div><br></div><div>-kurt</div><div><br></div></div>