<div class="gmail_quote"><div class="gmail_quote"><br><div class="gmail_quote"><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Andreas said he is OK with applying them, so I will do that.<br>
<br></blockquote><div>OK :) <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
However, if I read your code correctly, it still seems to me that there is<br>
a fixed compile time decision if gpsd or built-in gps support is to be used.<br>
<br>
I think it would be better to keep it a runtime decision, i.e.<br>
<br>
1) if gpsd headers/library available during compilation,<br>
   build support for _both_ gpsd and built-in gps into the program,<br>
2) if they are not available, only include the built-in gps support.<br>
<br></blockquote><div>That's easy to do. I can maintain the original gps interface, so old code doesn't need to be fixed, and choose the support to use from an internal state.<br></div><div> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">


The decision which method to use should be a config file option.  Please make<br>
sure that a config file configured for built-in gps support will work with both<br>
versions of the program.<br>
<br>
A config file requesting the use of gpsd support should make the program<br>
abort if it was compiled without gpsd support included.<br>
<br></blockquote><div>Not sure to agree. Execs from layer23/misc don't rely on config files. Probably that's due to the low number of switches used. I think that, even if cell_log options are increased, we can still use cmd switches. Do you agree? <br>


</div><br>Ciao<br>Dario.<br></div>
</div><br>
</div><br>