<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 19, 2014 at 3:57 AM, Aleksander Morgado <span dir="ltr"><<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Tue, Feb 18, 2014 at 9:42 PM, Ben Chan <<a href="mailto:benchan@chromium.org">benchan@chromium.org</a>> wrote:<br>


> This patch renames 'MBIM_CID_DEVICE_SERVICE_SUBSCRIBER_LIST' to<br>
> 'MBIM_CID_DEVICE_SERVICE_SUBSCRIBE_LIST' to be consistent with the MBIM<br>
> specification. This CID is for subscribing notifications for CIDs that<br>
> the host is interested in, and is not related to mobile 'subscriber'.<br>
> ---<br>
> Aleksander,<br>
><br>
> We may want to bump the version of libmbim to make it easier to align with the<br>
> ModemManager code. How do you think?<br>
><br>
<br>
</div>Ok, this one is more problematic :)<br>
<br>
Do you have more of these? I think I'll create a branch to handle all<br>
the API break, and once we have some more we can tag a development<br>
version with all these changes, and directly bump the requirement in<br>
ModemManager git master. We'll need to sync releases afterwards, or<br>
even add some additional MBIM_CHECK_VERSION() checks in the current<br>
stable version.<br>
<div class=""><br></div></blockquote><div><br></div><div>Yeah, let's put this on hold. I'm going through the API and see if any further change is needed.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="">
> BTW, I didn't update MBIM_CID_BASIC_CONNECT_DEVICE_SERVICE_SUBSCRIBER_LIST in NEWS :)<br>
<br>
</div>No worries, I'll get to update NEWS myself before the next release.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Aleksander<br>
<a href="https://aleksander.es" target="_blank">https://aleksander.es</a><br>
</font></span></blockquote></div><br></div></div>