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

don't include Frameworks into DTTableViewManager.framework #28

Merged
merged 1 commit into from
Dec 10, 2015

Conversation

etataurov
Copy link
Contributor

Otherwise iTunes Connect rejects app that embeds DTTableViewManager.framework

[18:46:06]: ERROR ITMS-90206: "Invalid Bundle. The bundle at 'MyApp.app/Frameworks/DTModelStorage.framework' contains disallowed file 'Frameworks'."

I'm using Carthage, but seems like CocoaPods might complain about EMBEDDED_CONTENT_CONTAINS_SWIFT = NO;
There is an ongoing discussion in CocoaPods CocoaPods/CocoaPods#4515

@codecov-io
Copy link

Current coverage is 87.56%

Merging #28 into master will not affect coverage as of 341c5af

@@            master     #28   diff @@
======================================
  Files            4       4       
  Stmts          603     603       
  Branches         0       0       
  Methods          0       0       
======================================
  Hit            528     528       
  Partial          0       0       
  Missed          75      75       

Review entire Coverage Diff as of 341c5af

Powered by Codecov. Updated on successful CI builds.

@DenTelezhkin DenTelezhkin self-assigned this Dec 10, 2015
DenTelezhkin added a commit that referenced this pull request Dec 10, 2015
don't include Frameworks into DTTableViewManager.framework
@DenTelezhkin DenTelezhkin merged commit 0087756 into DenTelezhkin:master Dec 10, 2015
@DenTelezhkin
Copy link
Owner

Yep, it seems like Apple knows about this stuff.
https://developer.apple.com/library/ios/qa/qa1881/_index.html

Thanks, @etataurov!

@DenTelezhkin
Copy link
Owner

I removed this build setting from all three repos, DTTableViewManager, DTModelStorage and DTCollectionViewManager. Can you try building from master branch and tell me if submission worked?

@etataurov
Copy link
Contributor Author

@DenHeadless Thanks a lot!
Also I've forgotten tvOS in my PR, but seems you've made it

I'll try it

@etataurov
Copy link
Contributor Author

I've only tried DTTableViewManager's master branch, but DTModelStorage was 2.3.0, because there is strict version in DTTableViewManager's Cartfile

unfortunately DTTableViewManager.framework in my Carthage/Build/iOS still contains Frameworks directory. Even after clean.
But when I build DTTableViewManager myself using Xcode, DTTableViewManager.framework in DerivedData does not contain it

Is DTModelStorage 2.3.0 a cause of it?

@DenTelezhkin
Copy link
Owner

It might. Please try building from master again, i added 2.3.1 tag to DTModelStorage, and refer to it in Cartfile.

@etataurov
Copy link
Contributor Author

Oh, actually it worked, after I removed ModuleCache. My fault.

DTModelStorage from master branch doesn't contain Frameworks dir in *.framework.
(iTunes Connect rejects build with Frameworks dir in *.framework. So it possible just to delete this dir and build will be accepted, that worked for me yesterday. So absence of Frameworks dir means that build is OK)

So everything is fine. Thank you!
But I've only tested DTTableViewManager and DTModelStorage for iOS

@etataurov
Copy link
Contributor Author

Build with DTTableViewManager from master & DTModelStorage = 2.3.1 was accepted by iTunes Connect
Thanks again!

@DenTelezhkin
Copy link
Owner

No problem! 🍻

@DenTelezhkin
Copy link
Owner

Aaaand now DTTableViewManager and DTCollectionViewManager tests are failing, because apparently DTModelStorage has to have this setting turned on. Oh Apple, you are making me sad.

I will probably need to reconfigure all CI setup because of this.

Possibly related links:

CocoaPods/CocoaPods#2894
https://twitter.com/jamayers/status/582997970477400066
http://stackoverflow.com/questions/25777958/validation-error-invalid-bundle-the-bundle-at-contains-disallowed-file-fr

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

3 participants