[GitHub] thrift issue #723: dub.sdl for integration into Dlang package registry

2018-01-25 Thread jeking3
Github user jeking3 commented on the issue:

https://github.com/apache/thrift/pull/723
  
Well, I was wrong; it's a link ordering issue.  I could not fix/force the 
link order coming out of dmd, so I switched to the ld.gold linker for the 
docker images and that one is smart enough to get it right (also it's way 
faster than the bfd linker).


---


[GitHub] thrift issue #723: dub.sdl for integration into Dlang package registry

2018-01-24 Thread jeking3
Github user jeking3 commented on the issue:

https://github.com/apache/thrift/pull/723
  
See https://issues.apache.org/jira/browse/THRIFT-4308

Essentially the upstream deimos modules for openssl and libevent no longer 
work, and we need to move to something else for secure and for non-blocking 
modes.  I think there's probably a better D crypto lib we can use, but I'm not 
sure about libevent.  In any case, there hasn't been much of an uproar from 
folks about these things not working.


---


[GitHub] thrift issue #723: dub.sdl for integration into Dlang package registry

2018-01-24 Thread nrTQgc
Github user nrTQgc commented on the issue:

https://github.com/apache/thrift/pull/723
  
I replaced dub.sdl with dub.json and fixed deprecation warnings. Can you 
explain issues with current versions of openssl and libevent? What options we 
have? 


---


[GitHub] thrift issue #723: dub.sdl for integration into Dlang package registry

2018-01-23 Thread jeking3
Github user jeking3 commented on the issue:

https://github.com/apache/thrift/pull/723
  
All parties here, please move this forward or it will eventually be closed 
due to inactivity.  It looks like we're fairly close to completing this; 
however I have to point out that the project's use of deimos has shacked it in 
a bad way, and we don't support current versions of openssl or libevent any 
more because of it...


---


[GitHub] thrift issue #723: dub.sdl for integration into Dlang package registry

2017-04-07 Thread nrTQgc
Github user nrTQgc commented on the issue:

https://github.com/apache/thrift/pull/723
  
dub.json is fine too


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] thrift issue #723: dub.sdl for integration into Dlang package registry

2017-04-07 Thread jfarrell
Github user jfarrell commented on the issue:

https://github.com/apache/thrift/pull/723
  
Any objections to switching this to dub.json as follows ?

```
{
  "name": "apache-thrift",
  "description": "Apache Thrift D library",
  "authors": [
"Apache Thrift Developers "
  ],
  "homepage": "http://thrift.apache.org";,
  "license": "Apache-2.0",
  "dependencies": {
"libevent": {
  "version": "*",
  "optional": "true"
},
"openssl": {
  "version": "*",
  "optional": "true"
}
  },
  "targetType": "library",
  "sourcePaths": [
"src"
  ],
  "importPaths": [
"src"
  ],
  "excludedSourceFiles": [
"src/thrift/index.d"
  ]
}
```


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] thrift issue #723: dub.sdl for integration into Dlang package registry

2017-04-06 Thread jeking3
Github user jeking3 commented on the issue:

https://github.com/apache/thrift/pull/723
  
THRIFT-3458


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---