<div dir="ltr"><div></div>Hi Martin,<br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Dec 20, 2015 at 10:04 AM,  <span dir="ltr"><<a href="mailto:svn_grass@osgeo.org" target="_blank">svn_grass@osgeo.org</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">Author: martinl<br>
Date: 2015-12-20 07:04:40 -0800 (Sun, 20 Dec 2015)<br>
New Revision: 67282<br>
<br>
Modified:<br>
   grass/trunk/lib/python/script/setup.py<br>
Log:<br>
pythonlib: add wxpython modules to pythonpath when calling gsetup.init()<br>
<br>
<br>
Modified: grass/trunk/lib/python/script/setup.py<br>
===================================================================<br>
--- grass/trunk/lib/python/script/setup.py      2015-12-20 13:51:21 UTC (rev 67281)<br>
+++ grass/trunk/lib/python/script/setup.py      2015-12-20 15:04:40 UTC (rev 67282)<br>
@@ -174,7 +174,7 @@<br>
     # Set PYTHONPATH to find GRASS Python modules<br>
     # TODO: isn't this useless? user already imported this somehow<br>
     path = os.getenv('PYTHONPATH')<br>
-    etcpy = os.path.join(gisbase, 'etc', 'python')<br>
+    etcpy = os.path.join(gisbase, 'etc', 'python') + os.pathsep + os.path.join(gisbase, 'gui', 'wxpython')<br>
     if path:<br>
         path = etcpy + os.pathsep + path<br>
     else:<br></blockquote><br><div>I think this is too risky. If I understand it correctly, it puts all the GUI modules/packages on path. So, one will get:<br><br>animation<br>core<br>datacatalog<br>dbmgr<br>docs<br>gcp<br>...<br><br></div><div>Considering the amount and naming, it is likely to get name collision. This will happen even if the caller doesn't have any interest in GUI.<br><br></div><div>Simple way to solve this would be adding GUI to path optionally by a parameter or by providing a separate function.<br><br></div><div>Better way to solve this is probably putting GUI modules/packages to a bigger package. Let's say grassgui, or perhaps grasswxgui. Another option is to have grass.wxgui package. In this case, putting things to etc/python/ would be mandatory, in the other cases it is optional.<br><br></div><div>Vaclav<br></div><div><br><a href="https://trac.osgeo.org/grass/changeset/67282">https://trac.osgeo.org/grass/changeset/67282</a><br><br></div></div></div></div>