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

Update iOS build scripts to work with iOS 7, Xcode 5 #269

Merged
merged 114 commits into from
Oct 16, 2013
Merged

Conversation

alanjrogers
Copy link
Contributor

While maintaining Xcode 4.6/10.7 compatibility.

Continuation of #260

  • Update build scripts to work with iOS 7 SDK.
  • Adjust build scripts to still build with Xcode 4.6 on 10.7.
  • Make iOS 7 deployment (+arm64 and X86_64) an option (iOS 5.0 the default).
  • Move all of the common build logic into one place.

Depends on xcconfigs/xcconfigs#16

@joshaber
Copy link
Member

Something's rustled but it's not clear what from the build logs :(

@alanjrogers
Copy link
Contributor Author

@joshaber WIP bro. update_libgit2_ios is not very worky at the moment.

@joshaber
Copy link
Member

joshaber commented Oct 1, 2013

I just get so excited when I see a PR, I can't stop myself.

@alanjrogers
Copy link
Contributor Author

@jspahrsummers
Copy link
Contributor

@alanjrogers Still having the same issue. Anything I can grab?

@alanjrogers
Copy link
Contributor Author

@jspahrsummers maybe check the versions of libssh2 libtool autoconf automake you have installed?

@jspahrsummers
Copy link
Contributor

$ brew info libssh2 libtool autoconf automake

libssh2: stable 1.4.3
http://www.libssh2.org/
/usr/local/Cellar/libssh2/1.4.3 (177 files, 1,3M) *
  Built from source
From: https://github.com/mxcl/homebrew/commits/master/Library/Formula/libssh2.rb

libtool: stable 2.4.2
http://www.gnu.org/software/libtool/
/usr/local/Cellar/libtool/2.4.2 (66 files, 2,2M) *
  Built from source
From: https://github.com/mxcl/homebrew/commits/master/Library/Formula/libtool.rb
==> Options
--universal
    Build a universal binary
==> Caveats
In order to prevent conflicts with Apple's own libtool we have prepended a "g"
so, you have instead: glibtool and glibtoolize.

autoconf: stable 2.69
http://www.gnu.org/software/autoconf
/usr/local/Cellar/autoconf/2.69 (69 files, 2,0M) *
  Built from source
From: https://github.com/mxcl/homebrew/commits/master/Library/Formula/autoconf.rb

automake: stable 1.14
http://www.gnu.org/software/automake/
/usr/local/Cellar/automake/1.14 (127 files, 2,5M) *
  Built from source
From: https://github.com/mxcl/homebrew/commits/master/Library/Formula/automake.rb
==> Dependencies
Required: autoconf

$ brew update
$ brew install libssh2 libtool autoconf automake
Warning: libssh2-1.4.3 already installed
Warning: libtool-2.4.2 already installed
Warning: autoconf-2.69 already installed
Warning: automake-1.14 already installed

@jspahrsummers
Copy link
Contributor

Here are the updated build logs.

@alanjrogers
Copy link
Contributor Author

@jspahrsummers Can you give these latest changes a go?

@jspahrsummers
Copy link
Contributor

@alanjrogers /nope

@alanjrogers
Copy link
Contributor Author

@jspahrsummers That doesn't look like it used the latest changes (It's still calling /usr/bin/xcrun clang).

@jspahrsummers
Copy link
Contributor

@alanjrogers /nope

@alanjrogers
Copy link
Contributor Author

@jspahrsummers I say we just merge this, it works everywhere but your machine. 🚎

@jspahrsummers
Copy link
Contributor

jspahrsummers added a commit that referenced this pull request Oct 16, 2013
Update iOS build scripts to work with iOS 7, Xcode 5
@jspahrsummers jspahrsummers merged commit 8fa7824 into master Oct 16, 2013
@jspahrsummers jspahrsummers deleted the xcode5-ios7 branch October 16, 2013 17:47
phatblat pushed a commit to phatblat/objective-git that referenced this pull request Sep 13, 2014
Update iOS build scripts to work with iOS 7, Xcode 5
@jspahrsummers jspahrsummers removed their assignment May 22, 2015
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.

6 participants