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 testing on Travis #191

Merged
merged 1 commit into from Mar 19, 2020
Merged

Update iOS testing on Travis #191

merged 1 commit into from Mar 19, 2020

Conversation

noloader
Copy link
Contributor

@noloader noloader commented Mar 11, 2020

This PR updates Travis testing for OS X and iOS.

The two big changes were updating setenv-ios.sh to use a specific -mxxx-min-ver, which allowed iSO simulator testing. Without -mxxx-min-ver changes the simulators would fail to compile under current versions of Xcode.

  • Update Brew packages in .travis.yml
  • Add TEST_OSX=yes to differentiate between OS X and iOS in .travis.yml
  • Use more consistent name: in .travis.yml
  • Switch to IOS_PREFIX="$HOME/$IOS_SDK-$IOS_CPU" due to sdk/cpu combos
  • Add iPhoneSimulator, AppleTVSimulator and WatchSimulator testing
  • Install Android and iOS tools using before_script in .travis.yml
  • Remove "$CFLAGS" from install_openssl.sh. OpenSSL already includes them.
  • Add no-tests and no-unit-tests to install_openssl.sh. OpenSSL was calling fork in several places
  • Patch OpenSSL in install_openssl.sh. OpenSSL was still calling fork in several places
  • Use pkg-config name rather than pkgconfig. It was causing a failure in apt-get
  • Update 15-ios.conf for OpenSSL. Flags are now added in the base recipe
  • Update setenv_ios.sh. It avoids -mios-version-min used in earlier Xcodes. It now uses a specific flag, like -miphoneos-version-min, -miphonesimulator-version-min, -mappletvos-version-min, -mappletvsimulator-version-min, -mwatchos-version-min and -mwatchsimulator-version-min
  • Use XCODE_ prefix consistently in setenv_ios.sh
  • Guard use of fork and execlp in daemon/unbound.c, libunbound/libworker.c and smallapp/unbound-control.c

Travis is kind of slow today. You can see the result of the iOS changes at Job 152871267.

@ralphdolmans ralphdolmans self-assigned this Mar 12, 2020
@ralphdolmans ralphdolmans merged commit 20dbd63 into NLnetLabs:master Mar 19, 2020
1 check passed
@ralphdolmans
Copy link
Contributor

ralphdolmans commented Mar 19, 2020

Thanks!

@noloader noloader deleted the ios branch Mar 25, 2020
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.

None yet

2 participants