D17529: [proxysetting] Fix build with NM 1.4

Wolfgang Bauer noreply at phabricator.kde.org
Wed Dec 12 12:15:56 GMT 2018


wbauer added a comment.


  In D17529#375882 <https://phabricator.kde.org/D17529#375882>, @wbauer wrote:
  
  > I don't think that really is a problem here though, as it is defined in libnm/nm-setting-proxy.h (in NM 1.6+) which is included by this file (via NetworkManager.h that's included by setting.h).
  >  I.e. the enum will still be there when building against NM 1.6.0+ (otherwise the build would fail in the first place anyway, as it is used just a few lines below).
  
  
  Sorry, I somehow read "API incompatible"...
  
  TBH, I'm not sure about ABI either. But as this is outside of any class, it shouldn't be a problem? (as mentioned, it would be defined by another include file anyway when building against  NM 1.6.0+)

REPOSITORY
  R282 NetworkManagerQt

REVISION DETAIL
  https://phabricator.kde.org/D17529

To: wbauer, jgrulich
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181212/f1d37578/attachment.html>


More information about the Kde-frameworks-devel mailing list