osmith has posted comments on this change. ( https://gerrit.osmocom.org/12302 )
Change subject: msc_vty.c: configurable retrieval of IMEI, IMEISV ...................................................................... Patch Set 1: (2 comments) https://gerrit.osmocom.org/#/c/12302/1/src/libmsc/msc_vty.c File src/libmsc/msc_vty.c: https://gerrit.osmocom.org/#/c/12302/1/src/libmsc/msc_vty.c@437 PS1, Line 437: "Send each IMEI to the EIR to ask if it is permitted or not. The EIR is implemented as part of osmo-hlr, " > slightly out-of-scope, we don't know whether the user is using OsmoHLR or > something else. True, but I think it would make it a lot easier to understand for the users, what this option is for. If they don't use OsmoHLR, then they can ignore that sentence. With that in mind, do you still prefer to have it removed? https://gerrit.osmocom.org/#/c/12302/1/src/libmsc/msc_vty.c@440 PS1, Line 440: "1 = send each IMEI to the EIR\n") > (we normally don't name the values again, just write the string for that > value. […] Ack, and I'll submit a patch later to fix this in the description above. -- To view, visit https://gerrit.osmocom.org/12302 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iee516b9cd7877b21207ce9a6d954109f19558163 Gerrit-Change-Number: 12302 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <[email protected]> Gerrit-Reviewer: Jenkins Builder (1000002) Gerrit-Reviewer: osmith <[email protected]> Gerrit-CC: Max <[email protected]> Gerrit-CC: Neels Hofmeyr <[email protected]> Gerrit-CC: Stefan Sperling <[email protected]> Gerrit-Comment-Date: Mon, 07 Jan 2019 16:09:04 +0000 Gerrit-HasComments: Yes Gerrit-HasLabels: No
