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

CocoaLumberjack 2.x #141

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

Conversation

tommeier
Copy link

Hi,

I was recently unable to use this dependency with the latest cocapods upgrade (0.37.2 -> 0.38.1).

screen shot 2015-07-24 at 4 38 23 pm

I'm using RoutingHTTPServer which has a dependency of CocoaHTTPServer.

TBH I'm wandering around in the dark on this project, I'm not sure why DDLogX is duplicated by the HTTPLogger? Is the real fix to change it to have a custom cocoalumberjack logger for the HTTP logging?

Anyway, I have no idea if this is the real fix or not, but this got it working again for me, would love to start the conversation of what the real fix would/should be as I wander in the darkness :)

@tommeier
Copy link
Author

My particular issue was resolved by CocoaPods/CocoaPods#3898. However, would still like to start the conversation on a conversion to 2.x cocoalumberjack, is this feasible? and would it remove a lot of code by having a custom logger setup instead of DDLog.h duplicated?

@HardipAtWpost
Copy link

👍 Need this for a project we are working on that uses this library for unit tests.

@wmtylerdavis
Copy link

👍 Yeah this seems relevant again with cocoapods 1.0.0.beta.3

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