Attention is currently required from: Hoernchen, fixeria, laforge, osmith, pespin.
Timur Davydov has posted comments on this change by Timur Davydov. ( https://gerrit.osmocom.org/c/osmo-trx/+/42243?usp=email )
Change subject: build: install common/GSM/transceiver libraries for emscripten and add pkg-config files ......................................................................
Patch Set 3:
(1 comment)
File CommonLibs/Makefile.am:
https://gerrit.osmocom.org/c/osmo-trx/+/42243/comment/096fab28_b81b36a5?usp=... : PS3, Line 56: lib_LTLIBRARIES = libcommon.la
By doing this, you are installing the libraries, which is something we don't wanna do.
This is not intended for the regular native build.
The idea is to enable this only for the Emscripten/WebAssembly build, where we intentionally combine parts of `osmo-bts-trx` and `osmo-trx` into a single target. In that scenario, we are not installing system-wide libraries in the traditional sense — we just need proper linkage metadata to consume those components during the unified build.
For normal native builds, nothing would change and no additional libraries would be installed.
If installing the libraries is not acceptable even conditionally, perhaps you could suggest a preferred way to make the internal components linkable when building `osmo-bts-trx` and `osmo-trx` together as a single target? I'm open to structuring it in a way that fits the project’s expectations.
As an alternative, I could create a separate library specifically for the Emscripten build that would include only the source files required from osmo-trx.