Skip to content

meson: Don't build OSX plugin on iOS#284

Merged
janbrummer merged 2 commits intolibproxy:mainfrom
nirbheek:config-osx-not-ios
Apr 2, 2024
Merged

meson: Don't build OSX plugin on iOS#284
janbrummer merged 2 commits intolibproxy:mainfrom
nirbheek:config-osx-not-ios

Conversation

@nirbheek
Copy link
Copy Markdown
Contributor

@nirbheek nirbheek commented Mar 8, 2024

Fails to build with:

../src/backend/plugins/config-osx/config-osx.c:284:29: error: 'SCDynamicStoreCopyProxies' is unavailable: not available on iOS

Which is true, it's only available on macOS 10.1+

https://developer.apple.com/documentation/systemconfiguration/1517088-scdynamicstorecopyproxies

nirbheek added 2 commits April 2, 2024 14:52
Fails to build with:

../src/backend/plugins/config-osx/config-osx.c:284:29: error: 'SCDynamicStoreCopyProxies' is unavailable: not available on iOS

Which is true, it's only available on macOS 10.1+

https://developer.apple.com/documentation/systemconfiguration/1517088-scdynamicstorecopyproxies
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.73%. Comparing base (dd5f9c8) to head (0ba2b8a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #284   +/-   ##
=======================================
  Coverage   72.73%   72.73%           
=======================================
  Files          16       16           
  Lines         851      851           
  Branches      242      242           
=======================================
  Hits          619      619           
  Misses        137      137           
  Partials       95       95           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@janbrummer janbrummer merged commit e776242 into libproxy:main Apr 2, 2024
@nirbheek nirbheek deleted the config-osx-not-ios branch April 2, 2024 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants