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

ASF GitHub Bot commented on DISPATCH-1268:
------------------------------------------

jdanekrh commented on pull request #449: DISPATCH-1268 - Set 
cmake_minimum_required to VERSION 2.8.12
URL: https://github.com/apache/qpid-dispatch/pull/449
 
 
   A lot of changes from PR #288 is compatible with CMake 2.8.12 and can 
therefore still be used by the project.
   
   This PR contains only the first change, to `cmake_minimum_required` version, 
to see if there is a chance to get it in.
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Use more modern idiomatic cmake
> -------------------------------
>
>                 Key: DISPATCH-1268
>                 URL: https://issues.apache.org/jira/browse/DISPATCH-1268
>             Project: Qpid Dispatch
>          Issue Type: Improvement
>            Reporter: Jiri Daněk
>            Priority: Minor
>
> Dismissing the whole CMake modernization PR [1] is IMO too harsh. Much 
> modernization is possible, even if we maintain CMake 2.8 compatibility, and 
> that modernization is beneficial for overall sanity of the build code, as the 
> example of Qpid Proton project shows. [2], [3]
> RHEL 6.10 bundles cmake 2.8.12.2-4.el6 and RHEL 7.6 also has cmake 
> {{2.8.12.2}}. Therefore, the first step would be to change minimum required 
> version from {{2.6.2}} to this.
> {code}
>     cmake_minimum_required(VERSION 2.8.12)
> {code}
> The indentation changes are of course harmless in any case and easy to make. 
> I am not eager to bundle the whitespace change, as it is disruptive to 
> merging work in progress (if git not configured to ignore whitespace).
> [1] https://github.com/apache/qpid-dispatch/pull/288
> [2] https://issues.apache.org/jira/browse/PROTON-1878 Improve cmake 
> modularity - split out library detection code
> [3] https://issues.apache.org/jira/browse/PROTON-1879 Update cmake so that it 
> uses the easier to use imported target feature



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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org

Reply via email to