[Piglit] [PATCH 5/8] cmake: Increase minimum version from 2.6 to 2.8.
Dylan Baker
baker.dylan.c at gmail.com
Tue Oct 1 18:42:10 PDT 2013
On Tuesday, October 01, 2013 05:13:52 PM Ken Phillis Jr wrote:
> The use of cmake 2.8 just happens to coincide with requiring slightly newer
> tools... see these release announcements...
> kitware - news: cmake 2.8.11 released ( May 22, 2013 )
> http://www.kitware.com/news/home/browse/464?siteid=9
> kitware - news: cmake 2.8.0 released ( November 4,2009)
> http://www.kitware.com/news/home/browse/252?siteid=9
> kitware - news: cmake 2.6.4 released ( May 5, 2009 )
> http://www.kitware.com/news/home/browse/229?siteid=9
>
> On Oct 1, 2013 4:37 PM, "Dylan Baker" <baker.dylan.c at gmail.com> wrote:
> > On Monday, September 30, 2013 10:11:05 PM Kenney Phillis wrote:
> > > Increase the Minimum version of CMake that is required from
> > > version 2.6 to version 2.8. This is because the minimum requirement
> > > to compile waffle is version 2.8
> > > ---
> > >
> > > CMakeLists.txt | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/CMakeLists.txt b/CMakeLists.txt
> > > index f064b35..0da074a 100644
> > > --- a/CMakeLists.txt
> > > +++ b/CMakeLists.txt
> > > @@ -1,4 +1,4 @@
> > > -cmake_minimum_required(VERSION 2.6)
> > > +cmake_minimum_required(VERSION 2.8)
> > >
> > > list(APPEND CMAKE_MODULE_PATH "${CMAKE_SOURCE_DIR}/cmake/Modules")
> >
> > I don't think we need this. Waffle is a seperate project and a not a
> > dependency
> > (if running on non-linux platforms), besides that dependency ordering is
> > waffle
> > then piglit, so either you have 2.8, or you don't need it.
> >
> > Just my 2cents.
OK, didn't see anything in there that explained why we need 2.8.x when 2.6.x
is working.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20131001/d2be9a5b/attachment.pgp>
More information about the Piglit
mailing list