[GitHub] thrift issue #1410: THRIFT-82: Add Common Lisp support

2017-11-08 Thread dkochmanski
Github user dkochmanski commented on the issue: https://github.com/apache/thrift/pull/1410 Hello, I've send you an email a few hours ago with a question, if we can remove your "copyright" headers from the code found in de.setf.thrift, because that is what is required by Thrift team (I

[GitHub] thrift issue #1410: THRIFT-82: Add Common Lisp support

2017-11-08 Thread lisp
Github user lisp commented on the issue: https://github.com/apache/thrift/pull/1410 good evening; this popped up in my mail due to mr kochmanski's reference. what are you waiting for from me? ---

[GitHub] thrift issue #1410: THRIFT-82: Add Common Lisp support

2017-11-08 Thread dkochmanski
Github user dkochmanski commented on the issue: https://github.com/apache/thrift/pull/1410 see https://github.com/apache/thrift/pull/1412 ---

[GitHub] thrift issue #1410: THRIFT-82: Add Common Lisp support

2017-11-08 Thread dkochmanski
Github user dkochmanski commented on the issue: https://github.com/apache/thrift/pull/1410 closing, will reopen with squashed commits. ---

[GitHub] thrift issue #1410: THRIFT-82: Add Common Lisp support

2017-11-08 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1410 Let's follow @jfarrell 's lead. Whatever he says, he's managing the travis interaction. ---

[GitHub] thrift issue #1410: THRIFT-82: Add Common Lisp support

2017-11-08 Thread dkochmanski
Github user dkochmanski commented on the issue: https://github.com/apache/thrift/pull/1410 @jfarrell logged into travis a moment ago for the first time. @jeking3 I'm waiting for a response from @lisp if we can remove his copyright strings (we've removed ours). Should I close t

[GitHub] thrift issue #1410: THRIFT-82: Add Common Lisp support

2017-11-08 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1410 Why don't we close this PR and open a new one free of any copyright issues or inclusion of third party code. That should resolve the issue? ---

[GitHub] thrift issue #1410: THRIFT-82: Add Common Lisp support

2017-11-08 Thread jfarrell
Github user jfarrell commented on the issue: https://github.com/apache/thrift/pull/1410 @dkochmanski Travis support responded saying that no one from TurtleWarePL has logged into https://travis-ci.org/ ever and this is the reason this PR is not getting run within travis. Fro

[GitHub] thrift issue #1410: THRIFT-82: Add Common Lisp support

2017-11-07 Thread Jens-G
Github user Jens-G commented on the issue: https://github.com/apache/thrift/pull/1410 @jfarrell: What's your opinion re the (c) things above? ---

[GitHub] thrift issue #1410: THRIFT-82: Add Common Lisp support

2017-11-07 Thread jfarrell
Github user jfarrell commented on the issue: https://github.com/apache/thrift/pull/1410 Travis requests show an error with "Abuse detected". I've emailed Travis support asking them to look into the issue ---

[GitHub] thrift issue #1410: THRIFT-82: Add Common Lisp support

2017-11-07 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1410 Without a linux build, won't be able to do any merges... ---

[GitHub] thrift issue #1410: THRIFT-82: Add Common Lisp support

2017-11-06 Thread dkochmanski
Github user dkochmanski commented on the issue: https://github.com/apache/thrift/pull/1410 Hey, removed all externals and added `ensure-externals.sh` script. Also squashed all commits. ---

[GitHub] thrift issue #1410: THRIFT-82: Add Common Lisp support

2017-11-04 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1410 I think it would be better to make a shell script that downloads quicklisp and runs it to install the dependencies, and make the local build in lib/cl depend on successful execution of the shell scr