<p>Neels Hofmeyr <strong>uploaded patch set #5</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/11120">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">build: move include/{mtp,sccp} to include/osmocom/<br><br>Anywhere else in the Osmocom code base, we arrange headers in<br>include/osmocom/foo/ and pass -I ${root_srcdir}/include/.<br>This way including an osmocom header always has the format<br>  #include <osmocom/foo/bar.h><br>whether we are including from the local source tree or from $prefix.<br><br>For some reason not clear to me, the mtp and sccp folders, even though they are<br>being installed to $prefix/include/osmocom/, were kept *next* to the osmocom/<br>dir, instead of inside it. Fix that weird situation.<br><br>The motivation is that I wanted to use a definition from sccp_types.h in a<br>public-API header. That is impossible if it requires<br>  #include <sccp/sccp_types.h><br>in a local build, but<br>  #include <osmocom/sccp/sccp_types.h><br>for any other source tree using libosmo-sccp. After this patch, both are<br>identical and including works without quirks. (The other patch that needed this<br>has changed in the meantime on and no longer needs this, but this still makes<br>sense for future hacking.)<br><br>The installed result does not change, since both mtp/*.h and sccp/*.h have<br>always been installed to $prefix/include/osmocom/{mtp,sccp}/. This merely<br>changes their position in the source tree.<br><br>The most curious situation before this is that any patch #including<br><osmocom/sccp/sccp_types.h> might not get a notice that the header didn't<br>exist, but might instead include an older system-installed file.<br><br>Change-Id: I1209a4ecf9f692a8030b5c93cd281fc9dd58d105<br>---<br>M configure.ac<br>M include/Makefile.am<br>M include/osmocom/Makefile.am<br>R include/osmocom/mtp/Makefile.am<br>R include/osmocom/mtp/mtp_level3.h<br>R include/osmocom/mtp/mtp_pcap.h<br>R include/osmocom/sccp/Makefile.am<br>R include/osmocom/sccp/sccp.h<br>R include/osmocom/sccp/sccp_types.h<br>M src/mtp_pcap.c<br>M src/sccp.c<br>M src/sccp2sua.c<br>M src/sccp_sclc.c<br>M src/sccp_scoc.c<br>M src/sccp_scrc.c<br>M tests/mtp/mtp_parse_test.c<br>M tests/sccp/sccp_test.c<br>17 files changed, 12 insertions(+), 12 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/libosmo-sccp refs/changes/20/11120/5</pre><p>To view, visit <a href="https://gerrit.osmocom.org/11120">change 11120</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.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.osmocom.org/11120"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-sccp </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I1209a4ecf9f692a8030b5c93cd281fc9dd58d105 </div>
<div style="display:none"> Gerrit-Change-Number: 11120 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>