Cinterion bug? (Re: Handling received SMS)

[Colin Helliwell]

> On 14 February 2017 at 09:19 Aleksander Morgado <aleksander at aleksander.es> wrote:
> 
> On Tue, Feb 14, 2017 at 9:11 AM, Colin Helliwell
> 
> <colin.helliwell at ln-systems.com> wrote:
> 
> > > > All I'm seeing in the logs is "+CIEV: message,1".
> > > > Checking back through the logs for unsolicited event setups, I also see
> > > > [src/mm-broadband-modem.c:5888]
> > > > set_messaging_unsolicited_events_handlers(): (ttyMux0) Setting messaging
> > > > unsolicited events handlers
> > > > [src/mm-port-serial.c:1347] _close_internal(): (ttyMux0) device open count
> > > > is 1 (close)
> > > > [src/mm-iface-modem-messaging.c:867] enable_unsolicited_events_ready():
> > > > Couldn't enable unsolicited events: 'SMS settings don't accept [2,1]
> > > > 
> > > > '
> > > > 
> > > > That inner error message seems to originate in the Cinterion plugin's
> > > > messaging_enable_unsolicited_events (). From what I can make out, that
> > > > function is looking for a supported mode in the response to "AT+CNMI=?".
> > > > Doing this from the command line gives the response " +CNMI:
> > > > (0-3),(0,1),(0,2,3),(0,2),(1) ". Furthermore a "AT+CNMI=2" also from the
> > > > command line - which I think the plugin would have otherwise tried to do
> > > > if it had found the mode - does give an OK response.
> > > 
> > > The +CNMI=? parser didn't expect the "range" format given for the
> > > 
> > > field. I've fixed that in git master now.
> > 
> > 'Fraid I'm not geared up to log bug or supply patch, but I think there's a bug in
> > messaging_enable_unsolicited_events() of plugins/cinterion/mm-broadband-modem-cinterion.c :
> > Line 215 (on Master) should be an 'else if' ?
> 
> Yes, good catch!
> 
> Fixed in git master, mm-1-6 and mm-1-4; thanks.
> 
> --
> Aleksander
> https://aleksander.es

I'm currently pulling a source tarball (for Yocto environment) from http://www.freedesktop.org/software/ModemManager/
I wonder if I should, whilst in development at least, keep in step with your git mm-1-6 instead - I guess this branch is fairly stable i.e. you're just putting severe bug fixes into it?