review FindBlueZ.cmake
Stephen Kelly
steveire at gmail.com
Fri Sep 16 15:16:06 UTC 2011
Alexander Neundorf wrote:
> Hi,
>
> this file looks quite good.
>
> Only comments:
>
> * this part is unnecessary:
> if ( BLUEZ_INCLUDE_DIR AND BLUEZ_LIBRARIES )
> # in cache already
> set(BlueZ_FIND_QUIETLY TRUE)
> endif ( BLUEZ_INCLUDE_DIR AND BLUEZ_LIBRARIES )
>
> since find_package_handle_standard_args() anyway only prints something if
> this output differs from the previous run
>
> * the if(NOT WIN32) around the pkgconfig-stuff can be removed, if
> pkgconfig is not found, pkg_check_modules() simply does nothing
>
> * the additional search paths for find_path() and find_library() should
> probably be given with HINTS, not with PATHS, so they are preferred over
> the standard directories (as we do it in most other find-modules)
>
>
> Is git mv the right command to move a file from attic/ to modules/ with
> keeping history ?
> (I think I heard that git doesn't have mv, so I'm confused)
>
>
> Alex
Yes, git mv is what you want:
http://stackoverflow.com/questions/2314652/is-it-possible-to-move-rename-
files-in-git-and-maintain-their-history
More information about the Kde-buildsystem
mailing list