<div dir="auto"><div dir="auto">Hi Harald, Thomas,</div><div dir="auto"><br></div>I think it also makes sense to remove sqlite3 subdirectory, since it's no longer used. Any reason it's kept in the repository?<br><br><div data-smartmail="gmail_signature">Please excuse typos. Written with a touchscreen keyboard.<br><br>--<br>Regards,<br>Alexander Chemeris<br>CEO Fairwaves, Inc.<br><a href="https://fairwaves.co">https://fairwaves.co</a></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Jan 26, 2017 18:46, "Neels Hofmeyr" <<a href="mailto:nhofmeyr@sysmocom.de">nhofmeyr@sysmocom.de</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Looks related to this osmo-trx nightly package build breakage?<br>
<a href="https://osmocom.org/issues/1928" rel="noreferrer" target="_blank">https://osmocom.org/issues/<wbr>1928</a><br>
<br>
~N<br>
<br>
On Wed, Jan 25, 2017 at 07:42:53PM +0000, Harald Welte wrote:<br>
> Harald Welte has submitted this change and it was merged.<br>
><br>
> Change subject: Do not embed sqlite3 when building<br>
> ..............................<wbr>..............................<wbr>..........<br>
><br>
><br>
> Do not embed sqlite3 when building<br>
><br>
> Change-Id: If5edadc04c3ff953b451676e55ad3<wbr>d00d4e43c82<br>
> ---<br>
> M Makefile.am<br>
> M Makefile.common<br>
> 2 files changed, 1 insertion(+), 2 deletions(-)<br>
><br>
> Approvals:<br>
>   Tom Tsou: Looks good to me, approved<br>
>   Harald Welte: Looks good to me, but someone else must approve<br>
>   Jenkins Builder: Verified<br>
><br>
><br>
><br>
> diff --git a/Makefile.am b/Makefile.am<br>
> index 06251f0..f9f5a5c 100644<br>
> --- a/Makefile.am<br>
> +++ b/Makefile.am<br>
> @@ -28,7 +28,6 @@<br>
><br>
>  # Order must be preserved<br>
>  SUBDIRS = \<br>
> -     sqlite3 \<br>
>       CommonLibs \<br>
>       GSM \<br>
>       Transceiver52M<br>
> diff --git a/Makefile.common b/Makefile.common<br>
> index 734ab3a..d7927a5 100644<br>
> --- a/Makefile.common<br>
> +++ b/Makefile.common<br>
> @@ -33,6 +33,6 @@<br>
><br>
>  COMMON_LA = $(top_builddir)/CommonLibs/<a href="http://libcommon.la" rel="noreferrer" target="_blank">lib<wbr>common.la</a><br>
>  GSM_LA = $(top_builddir)/GSM/libGSM.la<br>
> -SQLITE_LA = $(top_builddir)/sqlite3/<a href="http://libsqlite.la" rel="noreferrer" target="_blank">libsql<wbr>ite.la</a> -ldl<br>
> +SQLITE_LA = -lsqlite3<br>
><br>
>  MOSTLYCLEANFILES = *~<br>
><br>
> --<br>
> To view, visit <a href="https://gerrit.osmocom.org/1484" rel="noreferrer" target="_blank">https://gerrit.osmocom.org/<wbr>1484</a><br>
> To unsubscribe, visit <a href="https://gerrit.osmocom.org/settings" rel="noreferrer" target="_blank">https://gerrit.osmocom.org/<wbr>settings</a><br>
><br>
> Gerrit-MessageType: merged<br>
> Gerrit-Change-Id: If5edadc04c3ff953b451676e55ad3<wbr>d00d4e43c82<br>
> Gerrit-PatchSet: 2<br>
> Gerrit-Project: osmo-trx<br>
> Gerrit-Branch: master<br>
> Gerrit-Owner: Max <<a href="mailto:msuraev@sysmocom.de">msuraev@sysmocom.de</a>><br>
> Gerrit-Reviewer: Harald Welte <<a href="mailto:laforge@gnumonks.org">laforge@gnumonks.org</a>><br>
> Gerrit-Reviewer: Jenkins Builder<br>
> Gerrit-Reviewer: Ruben Undheim <<a href="mailto:ruben.undheim@gmail.com">ruben.undheim@gmail.com</a>><br>
> Gerrit-Reviewer: Tom Tsou <tom@tsou.cc><br>
<br>
--<br>
- Neels Hofmeyr <<a href="mailto:nhofmeyr@sysmocom.de">nhofmeyr@sysmocom.de</a>>          <a href="http://www.sysmocom.de/" rel="noreferrer" target="_blank">http://www.sysmocom.de/</a><br>
==============================<wbr>==============================<wbr>===========<br>
* sysmocom - systems for mobile communications GmbH<br>
* Alt-Moabit 93<br>
* 10559 Berlin, Germany<br>
* Sitz / Registered office: Berlin, HRB 134158 B<br>
* Geschäftsführer / Managing Directors: Harald Welte<br>
</blockquote></div></div>