<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 30, 2015 at 4:12 PM, Jachym Cepicky <span dir="ltr"><<a href="mailto:jachym.cepicky@gmail.com" target="_blank">jachym.cepicky@gmail.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"><div>the script called g.search.module is now in trunk, I've added tests</div></blockquote><div><br></div><div>Great. It is really fast (without -m). It will be useful.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>(Vašek: they seem to be far too verbose to me, anything I could do about that?)</div></blockquote><div><br></div><div>Yes, partially. The it is written with verbosity rather higher than lower but some things would need more programming. No quiet option available now but at least some basic form might be relatively easy to add.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><br></div><div>I've left the colorize and print_output functions in the file, reason: better code structure would have to be implemented (read: IMHO decorator at least for colorize should be used and I'm to doubt for now to get working)</div></blockquote></div><br></div><div class="gmail_extra">Yes, perhaps we should let it settle.<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Now I see that termcolor is not necessarily installed (I had no reason to install it so far), so some better error handing is needed.<br><br></div><div class="gmail_extra">With -m, I'm getting a lot of these:<br></div><div class="gmail_extra"><br>g.search.module search -m<br>warning: file `<standard input>', around line 172:<br>  table wider than line width<br><br></div><div class="gmail_extra">I get those with test as well and additionally also:<br></div><div class="gmail_extra"><br>======================================================================<br>FAIL: test_json_output (__main__.TestSearchModule)<br>----------------------------------------------------------------------<br>Traceback (most recent call last):<br>  File "./test_g_search_module.py", line 37, in test_json_output<br>    self.assertEqual(stdout[3]['name'], 'r.basins.fill', 'r.basins.fill')<br>AssertionError: r.basins.fill <br>u'r.sim.water' != 'r.basins.fill'<br><br>======================================================================<br>FAIL: test_shell_outout (__main__.TestSearchModule)<br>----------------------------------------------------------------------<br>Traceback (most recent call last):<br>  File "./test_g_search_module.py", line 45, in test_shell_outout<br>    self.assertEqual(stdout[3], 'r.basins.fill')<br>AssertionError: 'r.sim.water' != 'r.basins.fill'<br>- r.sim.water<br>+ r.basins.fill<br><br></div><div class="gmail_extra">Can't investigate more now but I'm definitively looking forward to using it.<br></div></div>