You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Mario Emmenlauer (JIRA)" <ji...@apache.org> on 2019/01/26 14:27:00 UTC

[jira] [Commented] (THRIFT-4752) Add cmake package configuration support?

    [ https://issues.apache.org/jira/browse/THRIFT-4752?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16753092#comment-16753092 ] 

Mario Emmenlauer commented on THRIFT-4752:
------------------------------------------

What version should be set if the issue was experienced in current trunk (in the master branch HEAD revision)? I was seting the version that trunk currently reports, but I can also leave version empty if thats better?

> Add cmake package configuration support?
> ----------------------------------------
>
>                 Key: THRIFT-4752
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4752
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Deployment
>            Reporter: Mario Emmenlauer
>            Priority: Trivial
>              Labels: cmake
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> The thrift build with cmake is really great! In my daily use, there is a minor issue left: downstream packages all have to have a FindThrift.cmake script. This is slightly annoying, because it replicates the same detection script over and over in all downstream packages.
> Modern cmake suggests a nice solution: the use of package configuration files (see [https://cmake.org/cmake/help/v3.0/manual/cmake-packages.7.html#package-configuration-file).] With package configuration files, after building thrift, downstream packages can just use
> {noformat}
> find_package(thrift){noformat}
> and the library and compiler are "auto-magically" resolved. Transitive package dependencies are also resolved, and even build settings like compiler flags can be passed on.
> I have created a package configuration for thrift cmake build. Could this be considered for inclusion?



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)