Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 588516 - sci-visualization/paraview-5.1.0 needs CMake 3.5 or higher
Summary: sci-visualization/paraview-5.1.0 needs CMake 3.5 or higher
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Matthias Maier
URL: https://github.com/Kitware/ParaView/b...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-07-10 12:09 UTC by Markus Kaindl
Modified: 2016-07-11 10:37 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Kaindl 2016-07-10 12:09:51 UTC
When emerging paraview-5.1.0 it fails, showing that CMake 3.5 or higher is required.

The ebuild seems to depend on cmake only with inheriting cmake-utils.eclass, which depends on >=dev-util/cmake-2.8.12
Comment 1 Markus Kaindl 2016-07-10 12:18:10 UTC
It seems, CMake 3.5 is not really needed, we could apply the following "patch" and only need to depend on CMake 3.3:

https://gitlab.kitware.com/paraview/paraview/commit/26831364525fe03903ee3effaf1c9e11968d3001
Comment 2 Adam Feldman gentoo-dev 2016-07-11 00:53:08 UTC
(In reply to Markus Kaindl from comment #1)
> It seems, CMake 3.5 is not really needed, we could apply the following
> "patch" and only need to depend on CMake 3.3:
> 
> https://gitlab.kitware.com/paraview/paraview/commit/
> 26831364525fe03903ee3effaf1c9e11968d3001

Obviously up to the maintainer, but generally speaking, most maintainers opt to keep closest to upstream possible.  Keeps maintenance simple.

Adding reference to upstream requirement.
Comment 3 Matthias Maier gentoo-dev 2016-07-11 10:37:30 UTC
commit 54d87e88e7adda88600515f73c46f4a5e15862a9
Author: Matthias Maier <tamiko@gentoo.org>
Date:   Mon Jul 11 05:02:11 2016 -0500

    sci-visualization/paraview: add DEPEND on >=cmake-3.3, bug #588516
    
     * Require >=dev-util/cmake-3.3
     * Apply upstream patch to lower the version requirement for cmake to 3.3
    
    Thanks to Markus Kaindl.
    
    Package-Manager: portage-2.2.28