<p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/15361">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/15361/2/src/gbproxy/Makefile.am">File src/gbproxy/Makefile.am:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15361/2/src/gbproxy/Makefile.am@35">Patch Set #2, Line 35:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">$(top_builddir)/src/gprs/gprs_gb_parse.o \<br> $(top_builddir)/src/gprs/gprs_llc_parse.o \<br>   $(top_builddir)/src/gprs/crc24.o \<br>    $(top_builddir)/src/gprs/gprs_utils.o \<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Can we just link against 'libcommon.la' here? If no, please describe why in the comment message.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Indeed, I'm introducing a licommon.la which in the end is not used, in order to have all .o files built on one target (to avoid several targets building same files coming from different subdirs, and issues associated to that). I'm open to other alternatives that you think can be better, but this one is good enough.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I could indeed link against libcommon.la here, but I prefer keeping it this way in order to keep information on exactly which files each application requires. Specially since some files may be splitted later since only 1 application is using a subset of it for instance, so we can shrink this common section without duplicating code.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/15361">change 15361</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/c/osmo-sgsn/+/15361"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-sgsn </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I2cc98d3a276d953609bbbbaa9782a0112687791e </div>
<div style="display:none"> Gerrit-Change-Number: 15361 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 02 Sep 2019 12:12:13 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>