-
Notifications
You must be signed in to change notification settings - Fork 380
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
Surport Swift 5 #499
Surport Swift 5 #499
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes look relatively straightforward and minor. However the build fails because of https://travis-ci.org/goktugyil/EZSwiftExtensions. Xcode 10.0 cannot be installed on macOS 10.12.
What version of XCode and macOS X are you using to locally build them? We need to match that configuration with the travis.yml file on.
} | ||
} | ||
|
||
// public func timePassed() -> TimePassed { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are we commenting this out?
@@ -7,6 +7,7 @@ | |||
// | |||
|
|||
import Foundation | |||
import AVFoundation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need this ?
@@ -6,6 +6,8 @@ | |||
// Copyright (c) 2015 Goktug Yilmaz. All rights reserved. | |||
// | |||
|
|||
import Foundation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need this ?
I have created a remote branch ceeyang-Swift5.0, I will keep pushing in changes to fix builds and stuff, but I would need your help since I am out of date on all the XCode versions and MacOSX versions for latest swift stuff. Regardless, I am attempting a few expts here : #501. This is by no means the official PR for 5.0, this is to facilitate build fixes only. |
Checklist