[GitHub] ccollins476ad commented on issue #632: BLE Host - Remove BLE_GAP_EVENT_CONN_CANCEL

[GitBox <gi...@apache.org>]

ccollins476ad commented on issue #632: BLE Host - Remove BLE_GAP_EVENT_CONN_CANCEL
URL: https://github.com/apache/mynewt-core/pull/632#issuecomment-337989429
 
 
   Thanks, Andrzej.
   
   I did want to implement the full change, but I chickened out because of the "Backwards Compatibility" section of the release processs: https://cwiki.apache.org/confluence/display/MYNEWT/Release+and+Support+Policy#ReleaseandSupportPolicy-BackwardsCompatibility.  That page indicates that code using the deprecated feature should successfully build for at least six months after the feature was marked as deprecated.
   
   I am thinking you are right - there is not much benefit in keeping the old symbols around.  I don't think it is possible to slowly drop support for this feature.  Any application which actually depends on the `BLE_GAP_EVENT_CONN_CANCEL` event would benefit much more from a build error rather than silently breaking.
   
   I would like to send an email about this to the dev list to see what others think.  I am sure this is a scenario that will come up again, so we should make the appropriate change to the release policy.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services