[GitHub] csantanapr commented on a change in pull request #59: implement temporary workaround to issue_3646, swift 4.1 bug with structs

[GitBox <gi...@apache.org>]

csantanapr commented on a change in pull request #59: implement temporary workaround to issue_3646, swift 4.1 bug with structs
URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/59#discussion_r187786369
 
 

 ##########
 File path: tools/build/compile.sh
 ##########
 @@ -43,7 +43,9 @@ else
 fi
 DEST_PACKAGE_SWIFT="$BASE_PATH/spm-build/Package.swift"
 
-BUILD_FLAGS=""
+# Due to a current bug in the Swift Docker image compile optimization is disabled by default.
+# If you need compiler optimization you can override the BUILD_FLAGS to enable it
+BUILD_FLAGS="-Xswiftc -Onone"
 
 Review comment:
   yes add a check to only limit to swift 4 to leave swift3 unchanged.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services