GitHub user dhasenan opened a pull request:
https://github.com/apache/thrift/pull/1395
THRIFT-4306 dlang: public imports for dependencies
If you have an IDL file that imports another IDL file, you can define
a service that accepts, yields, or throws types defined in the
imported file. In this case, the dlang output should ensure that the
types defined in the imported IDL module are visible in the service
interface's module.
The generator failed to do that, resulting in a failure to compile
the generated source in these situations. This was previously
accepted by the compiler due to some long-standing bugs with symbol
visibility, but the compiler bugs have been resolved.
Fixes #4306.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dhasenan/thrift THRIFT-4306
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/thrift/pull/1395.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1395
----
commit 1eec5b0e4c379d927db014a3b89f01ea0b00713e
Author: dhasenan <[email protected]>
Date: 2017-10-21T15:29:44Z
THRIFT-4306 dlang: public imports for dependencies
If you have an IDL file that imports another IDL file, you can define
a service that accepts, yields, or throws types defined in the
imported file. In this case, the dlang output should ensure that the
types defined in the imported IDL module are visible in the service
interface's module.
The generator failed to do that, resulting in a failure to compile
the generated source in these situations. This was previously
accepted by the compiler due to some long-standing bugs with symbol
visibility, but the compiler bugs have been resolved.
Fixes #4306.
----
---
