Skip to content
This repository was archived by the owner on Aug 31, 2021. It is now read-only.

[[ Bug 14625 ]] Update the way iOS externals are linked - #1868

Merged
livecodesebastien merged 3 commits into
livecode:release-6.7.3from
livecodesebastien:bugfix-14625
Feb 19, 2015
Merged

[[ Bug 14625 ]] Update the way iOS externals are linked#1868
livecodesebastien merged 3 commits into
livecode:release-6.7.3from
livecodesebastien:bugfix-14625

Conversation

@livecodesebastien

Copy link
Copy Markdown
Contributor

build-extension-ios.sh (used by engine-mobile.xcodeproj) and the S/B are now following Xcode way:

  • build for armv7
  • build for arm64
  • lipo both of the generated files

The generation of dylib has been removed, since unused by the mobile standalones

build-extension-ios.sh (used by engine-mobile.xcodeproj) and the S/B are now following Xcode way:
 * build for armv7
 * build for arm64
 * lipo both of the generated files

The generation of dylib has been removed, since unused by the mobile standalones

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you mean to leave this change in?

It might make it a bit of a pain for testing certain things from Xcode...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes: otherwise, the building stops since revsecurity.dylib is not created anymore.
That can be better to remove the whole script though, as there is only this line.

@runrevmark

Copy link
Copy Markdown
Contributor

Reviewed-by: @runrevmark.

livecodesebastien added a commit that referenced this pull request Feb 19, 2015
[[ Bug 14625 ]] Update the way iOS externals are linked
@livecodesebastien
livecodesebastien merged commit 221924b into livecode:release-6.7.3 Feb 19, 2015
@livecodesebastien
livecodesebastien deleted the bugfix-14625 branch February 19, 2015 15:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants