[CMake] [New Module] FindHDF5.cmake
Will Dicharry
wdicharry at stellarscience.com
Thu Aug 20 11:25:01 EDT 2009
Brad King wrote:
> Will Dicharry wrote:
>> Sorry for the month of delay, but I've addressed Mike Jackson's concerns
>> below and I think I'm close to having the HDF5 find module ready for
>> submission.
>
> Excellent. I have a few comments from quickly glancing at them, but
> I don't have time for thorough testing. Overall it looks good.
>
>> There are two modules attached to this message: The FindHDF5.cmake
>> module and an AdjustLibraryVariables.cmake module, which is essentially
>> a copy of what the FindQt4 module does. It seems useful enough to
>> incorporate into the CMake Modules, and I can maintain it if you need a
>> maintainer.
>
> I'd like to choose a better name for AdjustLibraryVariables. Perhaps
> "SelectLibraryConfigurations"? Does it have all the functionality needed
> to update FindQt4 to use it too (you don't need to do this but it should
> be easy for the FindQt4 maintainer to do it)?
I agree, SelectLibraryConfigurations is better. I'll rename it. It
looks like I need to set ${basename}_LIBRARIES (plural) too in order for
the Qt4 module to use it, I'll go ahead and do that.
>
> The find_path and find_library calls need some tweaking. Please read
> the documentation of these commands to distinguish the cases of PATHS
> and HINTS keywords. The PATHS should only be last-resort guesses.
> The HINTS should be locations computed from the system, such as those
> reported by the hdf5 compiler wrapper tools. Also, paths like
>
> /usr/local/include
> /usr/include
>
> are searched automatically and need not be listed.
>
I'll clean that up, I think the only path I'm specifying that should be
in the PATH section is the $HOME/.local/ guess. It seems everything
else should be a HINT. Thanks for the tip.
>> How I addressed Mike Jackson's concerns is addressed in the module
>> documentation at the top of the file, please let me know if anyone has
>> any other concerns.
>
> Try placing these modules in the CMake/Modules source tree and running
>
> cmake --help-module FindHDF5
> cmake --help-module AdjustLibraryVariables
>
> to make sure the documentation formats correctly. Also, any macro in
> the public interface of the module should be documented using a format
> similar to the CMake command documentation.
What is the convention for keeping a macro out of the public interface?
Thanks for your help,
Will
>
> Thanks,
> -Brad
--
Will Dicharry
Software Developer
Stellar Science Ltd Co
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3344 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://www.cmake.org/pipermail/cmake/attachments/20090820/0812e206/attachment.bin>
More information about the CMake
mailing list