On 08/31/2015 04:47 AM, Paul Romano wrote:
> I'd like to offer the attached patch for consideration

Thanks for working on these changes.

Please split each independent fix out into its own commit with
corresponding commit message explaining the change.

For the HDF5_PREFER_PARALLEL implementation, please re-use
the existing find_program calls but simply prepare a variable
ahead of time that lists the preferred order of names.  OTOH,
there is another approach to solve this.  The find_library
command has a NAMES_PER_DIR option that says to consider all
the given names in each directory before moving on to the next
directory.  If find_program had this too then one would be able
to choose a preference simply by having one come first in PATH.

-Brad

-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers

Reply via email to