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

Use GCDWebServer from a pod #135

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

sergiou87
Copy link

This pull request depends on this one: ConnectSDK/Connect-SDK-iOS-Core#13

The problem is our project depends on GCDWebServer pod, with the WebDav sub-pod, and having these files also in ConnectSDK results in build errors because of having duplicated definitions of GCDWebServer classes.

So I have deleted these files here and added GCDWebServer as a pod dependency in the ConnectSDK/Core podspec.

IMHO this should be done with the rest of the dependencies, what do you think?

@eunikolsky
Copy link

@sergiou87, thanks for the pull request. We are not going to merge it at the moment, because there is a few things to think about. Please feel free to discuss the improvement here: #136

@iadnan
Copy link
Contributor

iadnan commented Feb 27, 2015

@sergiou87 Thank you for the pull request and using the sdk. If you have any issues with connect sdk in the future or if you would like to provide feedback or feature requests you can contact us at [email protected] . Looking forward to hearing from you.

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