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

Swift 4 support? #13

Closed
AndrewSB opened this issue Oct 23, 2017 · 7 comments · May be fixed by #14
Closed

Swift 4 support? #13

AndrewSB opened this issue Oct 23, 2017 · 7 comments · May be fixed by #14

Comments

@AndrewSB
Copy link

I'm trying to use this library in Swift 4. From what I see, there exist forks that have upgraded to swift 4 (https://github.com/apocolipse/Thrift-Swift/network). See https://github.com/Petro-Lomaka/Thrift-Swift, for example.
The framework they vend seems to work for me.

The command line thrift, on the other hand, seems not to. I'm trying to use @Petro-Lomaka's fork (same guy), but the generated code doesn't conform to the Thrift swift module. I'm currently trying to uninstall my thrift, then reinstall from brew, and then follow the instructions in #11 with the thrift fork.

Can we start the process of moving this library to swift 4?

@AndrewSB
Copy link
Author

From what I see, no one has updated the thrift fork, so the generated code is not currently usable in swift 4

@ghost
Copy link

ghost commented Oct 23, 2017

I have forked repo for tests. Look into feature branch.

generated code is not currently usable in swift 4
Xcode 9 supports Swift3.2 and Swift4. So generated code can be easily used in the project. There are some issues you should fix manualy.

@AndrewSB
Copy link
Author

But that means I'll need to switch all my code to swift 3.2, right? I can't use swift 4 in other targets?

@AndrewSB
Copy link
Author

Also, swift 3.2 doesn't work either. I switched my target to be swift 3.2, the errors reduce, but there are still 144+

screen shot 2017-10-23 at 11 41 00 am

@ghost
Copy link

ghost commented Oct 23, 2017

Not related to Swift4

AndrewSB added a commit to AndrewSB/Thrift-Swift that referenced this issue Oct 24, 2017
Happy to take in edits! I'm assuming a number of the people who look at the thrift docs for swift are going to be exclusively iOS engineers and completely unfamiliar with the unixisms to install

closes apocolipse#11
closes apocolipse#13
@AndrewSB
Copy link
Author

apocolipse's repositories currently support swift 4, I apologize for my confusion

@wyland
Copy link
Collaborator

wyland commented Apr 9, 2018

Ok to close? Or are there any remaining open questions in this issue?

@AndrewSB AndrewSB closed this as completed Apr 9, 2018
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 a pull request may close this issue.

2 participants