Swift support

[Jano Svitok <js...@whitestein.com>]

Hi all,

in the light of https://github.com/apache/thrift/pull/2149#issuecomment-630720402:
We tried to base our internal iOS app on Thrift, but we've found there is lot
of work to even make it compile (I'm talking about the Swift lib).

In the mean time, we switched to grpc and it worked out of the box. If their android
support is not broken, we will move also that to grpc (it's still on Thrift now).

I tried to upstream all of the fixes I did to make the Thrift server in iOS work.
Some of them were merged, some of them are in the queue and some of them were already
closed by stale bot. I also tried to report problem that I haven't had time to implement
proper fix (in our code I just avoided e.g. extends keyword).

I do not want to do a lot of development here, but I'd like to get as much
changes merged as possible, so that the work already done is not wasted, and
as a thank you for the project.

So what can I do to get these merged?

- https://github.com/apache/thrift/pull/2038
- https://github.com/apache/thrift/pull/2047
- https://github.com/apache/thrift/pull/2059

I will see if I can do something with this:
- https://github.com/apache/thrift/pull/2051

I will not implement the rest of this, but I'd say with a couple changes
it might at least test the compilation/connection:
https://github.com/apache/thrift/pull/2048


Side note: I understand the purpose of the stalebot, but from contributor's point of view it's
frustrating to keep the PRs open when the problem is
  a) lack of committers and/or their time
  b) broken CI.
I also understand both :) and I tried to help with b)...

Jano

Re: Swift support

[Jens Geyer <je...@apache.org>]

Good point. Absolutely.


-----Ursprüngliche Nachricht----- 
From: Jano Svitok
Sent: Tuesday, May 19, 2020 1:24 PM
To: dev@thrift.apache.org
Subject: Swift support

Hi all,

in the light of 
https://github.com/apache/thrift/pull/2149#issuecomment-630720402:
We tried to base our internal iOS app on Thrift, but we've found there is 
lot
of work to even make it compile (I'm talking about the Swift lib).

In the mean time, we switched to grpc and it worked out of the box. If their 
android
support is not broken, we will move also that to grpc (it's still on Thrift 
now).

I tried to upstream all of the fixes I did to make the Thrift server in iOS 
work.
Some of them were merged, some of them are in the queue and some of them 
were already
closed by stale bot. I also tried to report problem that I haven't had time 
to implement
proper fix (in our code I just avoided e.g. extends keyword).

I do not want to do a lot of development here, but I'd like to get as much
changes merged as possible, so that the work already done is not wasted, and
as a thank you for the project.

So what can I do to get these merged?

- https://github.com/apache/thrift/pull/2038
- https://github.com/apache/thrift/pull/2047
- https://github.com/apache/thrift/pull/2059

I will see if I can do something with this:
- https://github.com/apache/thrift/pull/2051

I will not implement the rest of this, but I'd say with a couple changes
it might at least test the compilation/connection:
https://github.com/apache/thrift/pull/2048


Side note: I understand the purpose of the stalebot, but from contributor's 
point of view it's
frustrating to keep the PRs open when the problem is
  a) lack of committers and/or their time
  b) broken CI.
I also understand both :) and I tried to help with b)...

Jano