Re-enable OS X and iOS on Travis (GH #570, PR #627)

.travis.yml

@@ -47,9 +47,6 @@ matrix:
     - os: osx
       compiler: clang
       env: BUILD_MODE="ubsan"
-    # Travis infrastructure problems, Issue 570
-    # Re-enable when OS X and iOS improves
-    - os: osx
 
   include:
     - os: linux
@@ -84,21 +81,22 @@ matrix:
       env:
         - BUILD_MODE=android
         - PLATFORM=x86_64
-
-    # Travis infrastructure problems, Issue 570
-    # Re-enable when OS X and iOS improves
-    #- os: osx
-    #  env:
-    #    - BUILD_MODE=ios
-    #    - PLATFORM=iPhoneOS
-    #- os: osx
-    #  env:
-    #    - BUILD_MODE=ios
-    #    - PLATFORM=WatchOS
-    #- os: osx
-    #  env:
-    #    - BUILD_MODE=ios
-    #    - PLATFORM=AppleTVOS
+    - os: osx
+      env:
+        - BUILD_MODE=ios
+        - PLATFORM=iPhoneOS
+    - os: osx
+      env:
+        - BUILD_MODE=ios
+        - PLATFORM=WatchOS
+    - os: osx
+      env:
+        - BUILD_MODE=ios
+        - PLATFORM=AppleTVOS
+    - os: osx
+      env:
+        - BUILD_MODE=ios
+        - PLATFORM=iPhoneSimulator
 
 before_install:
   - |
@@ -130,7 +128,7 @@ branches:
     - master
 
 addons:
-  sonarcloud: true
+  sonarcloud: false
 
   coverity_scan:
     project:

TestScripts/cryptest-android.sh

@@ -40,7 +40,7 @@ do
 				. ./setenv-android.sh "$platform" "$runtime" > /dev/null 2>&1
 				make -f GNUmakefile-cross static dynamic cryptest.exe
 				if [ "$?" -eq "0" ]; then
-					echo "$platform:$runtime ==> SUCCESSFUL" >> /tmp/build.log
+					echo "$platform:$runtime ==> SUCCESS" >> /tmp/build.log
 				else
 					echo "$platform:$runtime ==> FAILURE" >> /tmp/build.log
 					touch /tmp/build.failed
@@ -56,6 +56,7 @@ do
 done
 
 cat /tmp/build.log
+
 # let the script fail if any of the builds failed
 if [ -f /tmp/build.failed ]; then
 	exit 1

TestScripts/cryptest-ios.sh

@@ -20,26 +20,33 @@ for platform in ${PLATFORMS[@]}
 do
 	make -f GNUmakefile-cross distclean > /dev/null 2>&1
 
-	MESSAGE="Testing for Xcode support of $platform"
-	LEN=${#MESSAGE}
-	HEADER=$(seq  -f "*" -s '' $LEN)
-
 	echo
-	echo "$HEADER"
-	echo "$MESSAGE"
+	echo "====================================================="
+	echo "Testing for iOS support of $platform"
 
 	# Test if we can set the environment for the platform
-	./setenv-ios.sh "$platform" > /dev/null 2>&1
+	./setenv-ios.sh "$platform" "$runtime"
 
 	if [ "$?" -eq "0" ]; then
 		echo
-		echo "Building for $platform..."
+		echo "Building for $platform using $runtime..."
 		echo
 
-		. ./setenv-ios.sh "$platform"
+		# run in subshell to not keep any env vars
+		(
+			. ./setenv-ios.sh "$platform" > /dev/null 2>&1
 			make -f GNUmakefile-cross static dynamic cryptest.exe
+			if [ "$?" -eq "0" ]; then
+				echo "$platform ==> SUCCESS" >> /tmp/build.log
+			else
+				echo "$platform ==> FAILURE" >> /tmp/build.log
+				touch /tmp/build.failed
+			fi
+		)
 	else
 		echo
 		echo "$platform not supported by Xcode"
+		echo "$platform ==> FAILURE" >> /tmp/build.log
+		touch /tmp/build.failed
 	fi
 done