<div dir="ltr">Hi Harald,<br><br>> The difference is:<br>><br>> * openbsc doesn't install any headers, and thus all 'include/openbsc'<br>>  headers are local to the program<br>> * libosmocore isntalls all headers, and thus all 'include/osmocom'<br>>  headers are installable<br><br>It is reasonable, and I support this idea. But anyway we have some<br>noinst_HEADERS in include/Makefile.am, they are:<br><br>noinst_HEADERS = \<br>    osmocom/core/timer_compat.h \<br>    osmocom/gsm/kasumi.h osmocom/gsm/gea.h<br><br>So why they are among with other installable headers?<br>Are there any reasons to keep them here?<br><br>> So if you want to have private header files in libosmcoore<br>> (or any other library) you have to put them somewhere else,<br>> e.g. in the 'src' directory, as it is already done at several<br>> other places.<br><br>I think, you already pointed the strongest reason, why this<br>header should be outside the include/ directory: because it<br>only needed during unit tests, and nowhere else. And I absolutely<br>agree with you.<br><br>Now the problem is solved, and this discussion should be closed.<br><br><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>With best regards,<br></div><div>Vadim Yanitskiy.<br></div></div></div></div></div></div>
<br><div class="gmail_quote">2017-05-03 3:36 GMT+07:00 Harald Welte <span dir="ltr"><<a href="mailto:laforge@gnumonks.org" target="_blank">laforge@gnumonks.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Vadim,<br>
<span class=""><br>
On Wed, May 03, 2017 at 03:19:03AM +0700, Vadim Yanitskiy wrote:<br>
<br>
> > We tend to put all headers used from several locations in<br>
> > openbsc/openbsc/include/<wbr>openbsc/ (as noinst_HEADERS in the Makefile.am).<br>
> > To #include, you would use something like #include <openbsc/conv.h>.<br>
><br>
> Yeah, I tried to put one into include/osmocom/tests/, but Harald and Sylvain<br>
> voted against this approach. So I decided to go this way.<br>
<br>
</span>The difference is:<br>
<br>
* openbsc doesn't install any headers, and thus all 'include/openbsc'<br>
  headers are local to the program<br>
* libosmocore isntalls all headers, and thus all 'include/osmocom'<br>
  headers are installable<br>
<br>
So if you want to have private header files in libosmcoore (or any other<br>
library) you have to put them somewhere else, e.g. in the 'src'<br>
directory, as it is already done at several other places.<br>
<span class="HOEnZb"><font color="#888888">--<br>
- Harald Welte <<a href="mailto:laforge@gnumonks.org">laforge@gnumonks.org</a>>           <a href="http://laforge.gnumonks.org/" rel="noreferrer" target="_blank">http://laforge.gnumonks.org/</a><br>
==============================<wbr>==============================<wbr>================<br>
"Privacy in residential applications is a desirable marketing option."<br>
                                                  (ETSI EN 300 175-7 Ch. A6)<br>
</font></span></blockquote></div><br></div></div>