Attention is currently required from: osmith, pespin. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/27923 )
Change subject: osmobts: list all features of latest osmobts ......................................................................
Patch Set 2:
(1 comment)
File src/osmo-bsc/bts_trx_vty.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/27923/comment/ed9d444a_47afb36e PS2, Line 347: * connected yet (thus not sent the feature vector), so we cannot know for I think this entire [existing] logic is flawed. Until the BTS conntext we don't know for sure, so printing warnings or even failing to start based on what we expect a recent software would support is ... weird.
If we don't know something, we should be silent about it. And once we know that information, we can report warnings,errors or whatever.
I'm not saying this should happen within this patch.