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

Xcode 8.3 beta compilation fails #119

Closed
taher-mosbah opened this issue Jan 25, 2017 · 18 comments
Closed

Xcode 8.3 beta compilation fails #119

taher-mosbah opened this issue Jan 25, 2017 · 18 comments

Comments

@taher-mosbah
Copy link

On Xcode 8.3 beta (8W109m) compilation fails with the following error :
redefinition of module 'DeepLinkKit'

not sure if it is a DeepLinkKit issue or a Cocoapods issue ..

@aataraxiaa
Copy link

aataraxiaa commented Feb 3, 2017

Likewise, seeing this compilation failure with Xcode 8.3 beta.

screen shot 2017-02-03 at 10 30 43

@taher-mosbah
Copy link
Author

@seanjsummers deleting the content of the file "fixed" the problem for me.

@ranganathagv
Copy link

If you just want to run you can comment that file content and run, but not sure how functionally it breaks

@salabaha
Copy link

https://developer.apple.com/library/content/releasenotes/DeveloperTools/RN-Xcode/Chapters/Introduction.html

Known Issues
When building a framework that has module maps in both the source and the install directories that define the same module, the compiler will show a redefinition message. (28638816)
Workaround: Rename the module map file in the source directory to a non-default name (the default name is module.modulemap or module.map), and set the Module Map File build setting to the renamed module map.

@igor-makarov
Copy link
Contributor

I suggest specifying the public and private headers in the podspec and getting rid of the manually defined module map altogether.

Link to relevant chapter in the Podspec docs

@Ekhoo
Copy link

Ekhoo commented Mar 28, 2017

Any chance to release a fix ?

@igor-makarov
Copy link
Contributor

My PR #121 fixes that.

@Unihilator
Copy link

can't wait to merge it guys

@Ekhoo
Copy link

Ekhoo commented Mar 28, 2017

👍

@igor-makarov
Copy link
Contributor

My PR got merged. @chrismaddern can you please tag out a new release for @taher-mosbah @superpeteblaze @Ekhoo who are waiting for it in this thread.

@platedbay
Copy link
Contributor

Definitely excited for this to roll out too, just a friendly reminder that you can point your source to a specific commit/branch if you don't want to wait around for a new release and subsequent pod update (if using cocoapods). We've been doing this to keep development rolling.

@igor-makarov
Copy link
Contributor

Yeah, I've got my Podfile pointed at my fork for a couple of days @platedbay

@chrismaddern
Copy link
Contributor

@wessmith will be shipping a new version today @igor-makarov 😄

@wessmith
Copy link
Member

Closed in #121 and part of 1.2.2

@igor-makarov
Copy link
Contributor

BTW all tests are passing but Slather can't find the coverage output, so Coveralls isn't updated. I don't know enough about those tools to fix it 😂 @chrismaddern @wessmith

@Unihilator
Copy link

New arrives after a fix. But probably it's only mine.

../DebugTest-iphonesimulator/DeepLinkKit/DeepLinkKit.framework/Modules/module.modulemap:2:19: Umbrella header 'DeepLinkKit-umbrella.h' not found

@igor-makarov
Copy link
Contributor

@Unihilator Well, since now the module map is auto generated, it's a CP issue...

@Unihilator
Copy link

Probably.. removed workspace and pod dir. updated pods - and now it's ok

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

No branches or pull requests

10 participants