<div dir="ltr"><div><div><div><div><div><div><div><div>Dear all,<br><br></div>I would like to remove the following comment:<br><br><span style="font-family:monospace,monospace"># TODO: why dirname is checked first - the logic should be revised</span><br><br></div><div>in the set_pat function:<br><br><a href="https://trac.osgeo.org/grass/browser/grass/trunk/lib/python/script/utils.py#L372">https://trac.osgeo.org/grass/browser/grass/trunk/lib/python/script/utils.py#L372</a><br><br></div>the reason is that if we invert the logic as suggested by the comment the function could find the libraries in the GRASS library path before the local directories.<br></div><div>Start looking for the module from the local directory it seems more obvious to me, as first option, than looking around in the system path. <br>Furthermore this choice is particularly useful for development because as developer we don't have to build every time, but we can simply run and test the module from the local shell with:<br><span style="font-family:monospace,monospace"><br></span></div><div><span style="font-family:monospace,monospace">python modulename.py key=value --overwrite</span><br></div><br></div>What do you think?<br></div>Is it clearer now?<br><br></div>Could I remove the comment?<br><br></div>Best regards<br><br></div>Pietro<br></div>