<p> Attention is currently required from: Joshua Colp. </p>
<p>George Joseph <strong>uploaded patch set #4</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/18127">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res_geolocation: eprofile, parsing, tests and more<br><br>There's a lot of stuff in this commit so hold on to<br>your knickers.<br><br>* Added ability to embed xml and xslt documents into a module.<br>  It's actually quite simnple and works for gcc and clang.<br>  See Makefile.rules, res/Makefile and geoloc_eprofile for<br>  more info.<br><br>* Fixed an issue with ast_variable_list_from_string where<br>  quotes weren't being handled correctly.  This required<br>  the addition of ast_strsep_strict() to utils.c<br><br>* Renamed some things...<br>   pidf_lo_section was renamed to just pidf_element.<br>   The "effective_profile" API names were shortened to<br>   "eprofile".<br><br>* Added a ast_xml_find_child_element() convenience macro.<br><br>* Added pidf_to_eprofile.xslt that condenses pidf-lo xml<br>  documents to a simpler format that's common to both GML<br>  and civicAddress formats.  This cut quite a bit of custom<br>  parsing code.<br><br>* Effective profiles can now be created directly from a pidf<br>  xml document or a URI.<br><br>* Created unit tests for effective profile creation.<br><br>Change-Id: I50b66bd041b2a62ab329406f20dbaeef1fa68fc1<br>---<br>M Makefile.rules<br>M configs/samples/geolocation.conf.sample<br>M include/asterisk/config.h<br>M include/asterisk/res_geolocation.h<br>M include/asterisk/strings.h<br>M include/asterisk/xml.h<br>M main/config.c<br>M main/utils.c<br>M res/Makefile<br>M res/res_geolocation.c<br>M res/res_geolocation/geoloc_channel.c<br>A res/res_geolocation/geoloc_common.c<br>M res/res_geolocation/geoloc_config.c<br>M res/res_geolocation/geoloc_doc.xml<br>A res/res_geolocation/geoloc_eprofile.c<br>M res/res_geolocation/geoloc_private.h<br>A res/res_geolocation/pidf_lo_test.xml<br>A res/res_geolocation/pidf_to_eprofile.xslt<br>M tests/test_config.c<br>19 files changed, 1,241 insertions(+), 132 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/27/18127/4</pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/18127">change 18127</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/18127"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: development/16/geolocation </div>
<div style="display:none"> Gerrit-Change-Id: I50b66bd041b2a62ab329406f20dbaeef1fa68fc1 </div>
<div style="display:none"> Gerrit-Change-Number: 18127 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Attention: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>