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

Switches DYLIB_INSTALL_NAME_BASE to @rpath to fix "dyld: Library not … #186

Merged
merged 2 commits into from
Dec 1, 2017

Conversation

R4N
Copy link
Contributor

@R4N R4N commented Nov 13, 2017

…loaded: @loader_path/Frameworks" when GTLR is embedded in main application and linked in separate framework as referenced in issue #69 (adjusted on GTLRiOS and GTLROSX)

…loaded: @loader_path/Frameworks" when GTLR is embedded in main application and linked in separate framework as referenced in issue 69.
@thomasvl
Copy link
Member

fyi - We picked up an updated AppAuth/GTMAppAuth dependency in another CL and we're trying to get travis happy with that (another active PR) before doing yet more build releated changes.

@thomasvl
Copy link
Member

Told travis to respin, not sure if that will move up to more recent files on not, but hopefully it will and we'll get a better state of things.

@R4N
Copy link
Contributor Author

R4N commented Nov 20, 2017

Cool, thanks for the update. Let me know if I need to do anything else on my end to have it re-trigger whenever appropriate.

@thomasvl
Copy link
Member

Looks like it reruns things still based on the state of your branch. Can you merge/rebase to the the current state of master so we can confirm things are good?

@R4N
Copy link
Contributor Author

R4N commented Nov 30, 2017

Should be all set. Sorry, I should have commented when I merged to catch it up.

@thomasvl
Copy link
Member

thomasvl commented Dec 1, 2017

And it looks like travis was happy with it, so thanks!

@thomasvl thomasvl merged commit 37d3080 into google:master Dec 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants