-
Notifications
You must be signed in to change notification settings - Fork 248
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
Non-public API usage: The app references non-public selectors: _terminateWithStatus #138
Comments
That file is meant to be used only for tests, it should not be compiled and linked into an app and sent to Apple. |
Makes sense. Then I have to figure out how to avoid having it compiled and linked... I'm sure other people will run into this trouble though. |
It has come up before (#130), but we haven't heard how folks added the dependency on their main app instead of just for tests. |
I met this issue when I switched from cordova iOS 4.3 to 4.4.0 earlier today, so I'm guessing that the dependency has to be automatically generated by cordova (or one of the cordova plugins I use). |
@pwbs Facing the same issue. |
@ishaq-bhojani since it's not meant to be linked anyway, what you can do is to simply delete it... I didn't investigate why cordova puts it there and makes it be embedded in the app, nor how to fix cordova for it to stop doing it... |
I add the GoogleToolboxForMac via pod and got rejected for this issue as well. I modified the method's name locally but this is not the solution I am going for. Is there any way to solve this problem without manual deletion of the file? |
Since it sounds like the problem is with something else using this pod (cordova?), has anyone reported the problem to them? As far as I know, there isn't really something we can do as they are the ones forcing the dependency in non testing code. |
I just opened an issue at Cordova's: https://issues.apache.org/jira/browse/CB-12843 |
Apparently it's coming from |
This has come up on issues google#130 and google#138.
Apparently Apple doesn't like the use of
_terminateWithStatus
in https://github.com/google/google-toolbox-for-mac/blob/master/UnitTesting/GTMIPhoneUnitTestDelegate.mThe email I just received:
Then I did
sed -i 's/_terminateWithStatus/_terminateWithStatusX/g' platforms/ios/Pods/GoogleToolboxForMac/UnitTesting/GTMIPhoneUnitTestDelegate.m
and now Apple's happy with it...I'm not sure what to do about it, but adding that
X
is probably not the right move.The text was updated successfully, but these errors were encountered: