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

Add iOS testing to Travis #182

Merged
merged 1 commit into from
Mar 5, 2020
Merged

Add iOS testing to Travis #182

merged 1 commit into from
Mar 5, 2020

Conversation

noloader
Copy link
Contributor

@noloader noloader commented Mar 5, 2020

This PR adds iOS testing to Travis.

The iOS testing is similar to Android testing. The environment is set, OpenSSL and Expat are built, and then Unbound is built. iOS uses a prefix of $HOME/iPhoneOS-$IOS_CPU, $HOME/AppleTVOS or $HOME/WatchOS.

Android testing was changed to use a prefix of $HOME/android$ANDROID_API-$ANDROID_CPU to make the procedure consistent with iOS. It had the added benefit we did not need to expalin problems with installing into sysroot.

iOS builds for AppleTVOS are currently failing. I don't have a reason why at the moment. I've got another project running a similar configuration. For the other project, AppleTVOS builds are OK, but WatchOS builds are failing. Since TV and Watch seem to have chronic problems they are listed in allow_failures. (For the other project and failures see here).

Both Android and iOS now perform a make install to test installation into prefix.

sudo: false was also removed. Current Travis machinery ignore the setting and issue a warning when the sudo key is set.

@gthess gthess self-assigned this Mar 5, 2020
Copy link
Member

@wcawijngaards wcawijngaards left a comment

Choose a reason for hiding this comment

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

Changes look good

@wcawijngaards wcawijngaards merged commit a29a615 into NLnetLabs:master Mar 5, 2020
wcawijngaards added a commit that referenced this pull request Mar 5, 2020
@wcawijngaards
Copy link
Member

Thanks for the fixes, I appreciate the further fixups to the battery of scripts.

@noloader noloader deleted the ios branch March 5, 2020 17:59
jedisct1 added a commit to jedisct1/unbound that referenced this pull request Mar 20, 2020
* nlnet/master: (149 commits)
  - Fix .travis.yml error, missing 'env' option.
  - Merge PR#194: Add libevent testing to Travis, by Jeffrey Walton.
  Add changelog entries for PR#134.
  - Log warning when using outgoing-port-permit and outgoing-port-avoid   while explicit port randomisation is disabled.
  - Fix NLnetLabs#158: open tls-session-ticket-keys as binary, for Windows. By Daisuke   HIGASHI.
  - Merge PR#191: Update iOS testing on Travis, by Jeffrey Walton.
  Add libevent testing to Travis
  Sync with upstream
  - Fix NLnetLabs#192: In the unbound-checkconf tool, the module config of   dns64 subnetcache respip validator iterator is whitelisted, it was   reported it seems to work.
  - Fix compile of test tools without protobuf.
  - Add check to make sure RPZ records are subdomain of configured zone origin.
  - Changelog entry for (Fix NLnetLabs#189, Merge PR NLnetLabs#190).
  Changelog for NLnetLabs#188 and configure script created.  Removed unneeded whitespace.
  Fix NLnetLabs#188: unbound-control.c:882:6: error: 'execlp' is unavailable: not available on tvOS
  Fix NetBSD compile (GH NLnetLabs#189)
  - Changelog note for PR NLnetLabs#186: Fix unrecognized 'echo -n' option on OS X,   by noloader.
  Fix unrecognized 'echo -n' option on OS X Also see NLnetLabs#183. This PR also updates a few typos in README-Travis.md, and expands the discussion of PKG_CONFIG_PATH for those who are not familiar with it.
  Fix changelog note, it is NLnetLabs#182, not NLnetLabs#184.
  Changelog note for NLnetLabs#184. - Fix PR NLnetLabs#184 from noloader: Add iOS testing to Travis.
  Add iOS testing to Travis
  ...
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

3 participants