Rolf Eike Beer wrote: > David Cole wrote: >> On Wed, Aug 15, 2012 at 3:31 PM, Rolf Eike Beer wrote: >>> David Cole wrote: >>>> On Wed, Aug 15, 2012 at 1:28 PM, Rolf Eike Beer wrote:
>>>>> I'll work around this for now, but I wonder if the restriction in
>>>>> list(REMOVE_DUPLICATES) actually makes sense. We don't give an error
>>>>> if
>>>>> there
>>>>> are no duplicates, why should we give an error if there are no
>>>>> items?
>>>>
>>>> Is it that there are no items, or that there is not even a list with
>>>> that name?
> > >
> > > In line 151 of FindX11.cmake there is a
> > >
> > > set(X11_LIBRARY_DIR "")
> > >
> > > so the variable should exists.
> >
> > Well, I can't think of why anybody would care if REMOVE_DUPLICATES on an
> > empty list produces a still empty list with no error.
>
> [...]
>
> > Silliness aside, I would be ok with changing the behavior to avoid the
> > error in this particular case. Unless somebody has a real-world
> > counter-argument where it is considered important to preserve our
> > presently
> > silly behavior.
>
> That thing is even more silly. I just took the test for this
> (Tests/CMakeTests/ListTest.cmake.in) which looked like this:
>
> set(result bill andy bill brad ken ken ken)
> list(REMOVE_DUPLICATES result)
> TEST("REMOVE_DUPLICATES result" "bill;andy;brad;ken")
>
> Now I went and changed it to be
>
> list(REMOVE_DUPLICATES result foo bar baz)
>
> And guess what? It worked.
The same applies to SORT and REVERSE. I've pushed a fix for this as topic
list-command-argument-fixes and merged it to next.
For the empty variable: it looks like an empty variable and a not existing one
are the same. Is there a sane way to query this or should I just ignore it
anyway?
Eike
--
signature.asc
Description: This is a digitally signed message part.
-- Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Please keep messages on-topic and check the CMake FAQ at: http://www.cmake.org/Wiki/CMake_FAQ Follow this link to subscribe/unsubscribe: http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers
