Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove all networkextension #119

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

dmastag
Copy link

@dmastag dmastag commented Jan 21, 2021

As per #116
We need to remove all empty com.apple.developer.networking.networkextension to be able to build for iOS

Requirements

  • Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
  • All new code requires tests to ensure against regressions

Description of the Change

I removed all empty com.apple.developer.networking.networkextension to be able to build for iOS
As per #116 this prevented me from builing on iOS

Alternate Designs

We could have added the String inside the PL file, but I am not sure which ones are used

Why Should This Be In Core?

So iOS is able to build

Benefits

People can build iOS Apps

Possible Drawbacks

Maybe we need the empty networkextension?

Applicable Issues

As per tripflex#116
We need to remove all empty com.apple.developer.networking.networkextension to be able to build for iOS
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.

None yet

1 participant