<html><body><div style="color:#000; background-color:#fff; font-family:lucida console, sans-serif;font-size:10pt"><div><span>This should only really affect folks who have checked out the source via mercurial, so chances are high that they are developer types, who would hopefully have a clue what this means.</span></div><div style="color: rgb(0, 0, 0); font-size: 13px; font-family: 'lucida console', sans-serif; background-color: transparent; font-style: normal;"><span>Besides all of that, we should probably add a Mac Developers page with this stuff on it. There is one for win32 which goes through similar prerequisites.</span></div><div style="color: rgb(0, 0, 0); font-size: 13px; font-family: 'lucida console', sans-serif; background-color: transparent; font-style: normal;"><span><br></span></div><div style="color: rgb(0, 0, 0); font-size: 13px; font-family: 'lucida console', sans-serif; background-color: transparent; font-style:
normal;"><span>-J</span></div><div><br></div> <div style="font-family: 'lucida console', sans-serif; font-size: 10pt;"> <div style="font-family: HelveticaNeue, 'Helvetica Neue', Helvetica, Arial, 'Lucida Grande', sans-serif; font-size: 12pt;"> <div dir="ltr"> <hr size="1"> <font size="2" face="Arial"> <b><span style="font-weight:bold;">From:</span></b> "chirp.cordless@xoxy.net" <chirp.cordless@xoxy.net><br> <b><span style="font-weight: bold;">To:</span></b> chirp_devel@intrepid.danplanet.com <br> <b><span style="font-weight: bold;">Sent:</span></b> Friday, April 11, 2014 11:11 PM<br> <b><span style="font-weight: bold;">Subject:</span></b> Re: [chirp_devel] run_tests: No module named serial<br> </font> </div> <div class="y_msg_container"><br>Tom,<br><br>Well, that doesn't really fix any problem I have, or had.<br>Creating and using an alias for kk7ds python is all that's necessary<br>to run the run_tests script. But you have to recognize that
that's the<br>fix for "No module named serial".<br><br>Making the script run automagically by requiring it be the first<br>python found in your search path doesn't help with identifying<br>the problem when you see the error message, it just adds another<br>level of obscurity. And it's possible that some user may not want<br>kk7ds python first in their path.<br><br>I'm personally ok with the status quo; I'm unlikely to see<br>"No module named serial" and not know the solution - now.<br><br>I'm willing to do the try/exept patch I suggested a couple of days<br>ago, because it makes that connection, and would help the next<br>newbie developer who trips over it. Let me know if that's wanted.<br><br>-dan<br><br>On Apr 10, 2014, at 11:01 AM, Tom Hayward - <a ymailto="mailto:esarfl@gmail.com" href="mailto:esarfl@gmail.com">esarfl@gmail.com</a> wrote:<br>> On Wed, Apr 9, 2014 at 4:48 PM, Tom Hayward <<a ymailto="mailto:esarfl@gmail.com"
href="mailto:esarfl@gmail.com">esarfl@gmail.com</a>> wrote:<br>>> Sorry I don't have the time to focus on this right now, but here's another hint:<br>>> <a href="http://stackoverflow.com/questions/2429511/why-do-people-write-usr-bin-env-python-on-the-first-line-of-a-python-script" target="_blank">http://stackoverflow.com/questions/2429511/why-do-people-write-usr-bin-env-python-on-the-first-line-of-a-python-script</a><br>>> <br>>> Notice that this is *not* how Dan calls Python from run_tests. You<br>>> might fix it and submit it as your first bug fix :-)<br>>> <br>>> Tom KD7LXL<br>> <br>> Okay, here's what I was hinting but didn't have a chance to type up yesterday:<br>> <br>> Add kk7ds python to your path:<br>> <br>> PATH=/opt/kk7ds/bin/python:$PATH<br>> <br>> <br>> Then fix the script so that it runs the first "python" found in your path:<br>> <br>> diff -r fa9439f41b37
tests/run_tests<br>> --- a/tests/run_tests Wed Mar 19 10:38:45 2014 -0700<br>> +++ b/tests/run_tests Thu Apr 10 10:15:04 2014 -0700<br>> @@ -1,4 +1,4 @@<br>> -#!/usr/bin/python<br>> +#!/usr/bin/env python<br>> #<br>> # Copyright 2011 Dan Smith <<a ymailto="mailto:dsmith@danplanet.com" href="mailto:dsmith@danplanet.com">dsmith@danplanet.com</a>><br>> #<br>> <br>> Now your system will choose the first "python" found in your PATH,<br>> which now is kk7ds python, which includes all the libraries Chirp<br>> requires.<br>> <br>> Tom KD7LXL<br>> _______________________________________________<br>> chirp_devel mailing list<br>> <a ymailto="mailto:chirp_devel@intrepid.danplanet.com" href="mailto:chirp_devel@intrepid.danplanet.com">chirp_devel@intrepid.danplanet.com</a><br>> <a href="http://intrepid.danplanet.com/mailman/listinfo/chirp_devel"
target="_blank">http://intrepid.danplanet.com/mailman/listinfo/chirp_devel</a><br>> Developer docs: <a href="http://chirp.danplanet.com/projects/chirp/wiki/Developers" target="_blank">http://chirp.danplanet.com/projects/chirp/wiki/Developers</a><br>> <br><br><br>_______________________________________________<br>chirp_devel mailing list<br><a ymailto="mailto:chirp_devel@intrepid.danplanet.com" href="mailto:chirp_devel@intrepid.danplanet.com">chirp_devel@intrepid.danplanet.com</a><br><a href="http://intrepid.danplanet.com/mailman/listinfo/chirp_devel" target="_blank">http://intrepid.danplanet.com/mailman/listinfo/chirp_devel</a><br>Developer docs: <a href="http://chirp.danplanet.com/projects/chirp/wiki/Developers" target="_blank">http://chirp.danplanet.com/projects/chirp/wiki/Developers</a><br><br><br></div> </div> </div> </div></body></html>