kdev4 assert with findQt4.cmake provided by default cmake install

Matt Rogers mattr at kde.org
Sun Apr 27 19:49:15 UTC 2008


On Sunday 27 April 2008 13:08:55 Aleix wrote:
> Ummm, this was fixed, yesterday :P
>
> On 4/26/08, cedric <cedric.borgese at gmail.com> wrote:
> >  when you try to import a cmake project that use qt4, this code in
> >  FindQt4.cmake (the one from the cmake project, in the kde svn provided
> > one, the code isn't present)
> >
> >     LIST(FIND ${qt4_files} OPTIONS _index)
> >     IF(NOT _index EQUAL -1)
> >       LIST(REMOVE_ITEM ${qt4_files} OPTIONS)
> >       LIST(LENGTH ${qt4_files} _length)
> >       WHILE(_length GREATER ${_index})
> >         LIST(GET ${qt4_files} ${_index} _opt_value)
> >         LIST(REMOVE_AT ${qt4_files} ${_index})
> >         LIST(APPEND ${qt4_options} ${_opt_value})
> >         LIST(LENGTH ${qt4_files} _length)
> >       ENDWHILE(_length GREATER ${_index})
> >     ENDIF(NOT _index EQUAL -1)
> >
> >  cause an assert in cmakeast.cpp line 2225 when parsing the line "
> >  LIST(REMOVE_AT ${qt4_files} ${_index})"
> >
> >  the problem comes from LIST(FIND ${qt4_files} OPTIONS _index) that is
> > not defined (see cmakeprojectvisitor.cpp line 1079) and _index (and its
> > value) is not added to the list of variables (m_vars)
> >  this finish as an assertion when trying to convert the value of _index
> > to an interger.
> >
> >  I write the attached patch (not sure at all it is the right way to do
> > it, but at least it works) and now I can import my project.
> >

Well, the list moderator is a bit slow. :)
--
Matt





More information about the KDevelop-devel mailing list