Skip to content
This repository has been archived by the owner on Nov 29, 2020. It is now read-only.

Fix BDBOAuth1Manager pod #2

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

Conversation

minhchau273
Copy link
Contributor

@chug2k @harley

We got some problems with BDBOAuth1Manager pod. The first time running pod install , the AFNetworking and BDBOAuth1Manager are installed the new one and the project cannot be built successfully due to the issue of BDBOAuth1Manager which Charles mentioned here. I tried many ways to frozen these pods but I cannot. So I had to edit the original files.

What do you think?

@chug2k
Copy link

chug2k commented Oct 17, 2016

How did we ever fix this? I was so irresponsible; I didn't reply to the issue.

@chug2k
Copy link

chug2k commented Oct 17, 2016

@minhchau273 if this is the best fix, I'd recommend you fork the BDBOAuthManager Pod, open a Pull Request, and then in our Podfile just reference your git branch.

@minhchau273
Copy link
Contributor Author

@chug2k take a look at the last comment on your issue on BDBOAuthManager repo.
Yeah, I totally agree with you. We should folk the BDBOAuthManager and refer to our branch in the Podfile.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants