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

Linker error in OSX #205

Closed
ys1382 opened this Issue Jun 18, 2017 · 5 comments

Comments

Projects
None yet
2 participants
@ys1382

ys1382 commented Jun 18, 2017

ld: warning: directory not found for option '-L//Pods/OpenSSL-Universal/lib-ios'

I see I have the directory lib-macos, but it seems to be looking for the iOS one.

To reproduce: add themis pod to Podfile, compile project.

Using OSX Sierra, Xcode 8.3.2

@vixentael

This comment has been minimized.

Member

vixentael commented Jun 19, 2017

Thank you @ys1382

I think i found a problem in podspec
https://github.com/cossacklabs/themis/blob/master/themis.podspec#L16

I'll fix it soon

@vixentael vixentael self-assigned this Jun 19, 2017

@vixentael vixentael added the bug label Jun 19, 2017

@vixentael

This comment has been minimized.

Member

vixentael commented Jun 28, 2017

hi @ys1382

i've updated podspec, linking to correct path for Open SSL for MacOS.

Commit 2e69c2f

Raw podspec
https://gist.githubusercontent.com/vixentael/4fb81d87c5627fbef2004e8a496454ec/raw/9e9d4a64dbf322934a5e9619b59f0f5995f2392e/themis_fix.podspec

I can confirm that newly created MacOS project in xcode is building success with this podspec. Can you please take a look if it works for you?

@ys1382

This comment has been minimized.

ys1382 commented Jun 29, 2017

I still get the error on OSX: ld: warning: directory not found for option '-L/

/gochat/apple/osx/Pods/OpenSSL-Universal/lib-ios'

You can try it for yourself here: https://github.com/ys1382/gochat/tree/themis/apple/osx

@vixentael

This comment has been minimized.

Member

vixentael commented Jun 29, 2017

That's true, because I haven't pushed new Themis pod without your confirmation :)

To check that everything is working, please link pod to the updated podspec directly. Link this
ys1382/gochat#5

After your confirmation, I'll merge this podspec into master. It will be updated in CocoaPods repo when Themis 0.9.5 will be released. Until then, please link to the podspec directly.

Thank you

@ys1382

This comment has been minimized.

ys1382 commented Jun 30, 2017

Fixed

@ys1382 ys1382 closed this Jun 30, 2017

vixentael added a commit that referenced this issue Jun 30, 2017

Merge pull request #211 from cossacklabs/fix/fix-205
fix #205 update podspec: use correct library search path for MacOS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment