[CMake] Status of FindHDF5.cmake in CMake?

[J.S. van Bethlehem]

Hej Will,
I'm not sure if this is the proper way to reply and I haven't checked 
your second mail with the attachment. But I think in #3 you're 
definitely doing something wrong. It took me quite a while to figure 
out, because the documentation is rather sparse, but whenever there is a 
hdf5-config.cmake file, you should not read that from a Find*.cmake 
file. Whenever there is such a file present, what should happen is that 
CMake finds it and reads it and that's that.
Even more precise: whenever a user writes find_package(HDF5) and the 
hdf5-config.cmake exists, CMake will read that file and will NOT go look 
for FindHDF5.cmake. The hdf5-config.cmake should then explicitly set all 
the (cached) variables you normally would have to 'calculate' in the 
FindHDF5.cmake file.
So to be short: the FindHDF5.cmake file should never be required to look 
for hdf5-config.cmake - CMake should be left to be the one to do that. 
The HDF5 installer should be the one that installs the hdf5-config.cmake 
into for example <prefix>/share/HDF5/cmake or some similar path (look 
for documentation of find_package for possible locations where CMake 
will look)

But again: I haven't read the actual file you created :P Maybe I'm 
overlooking some details of this particular package.

Greetsz,
Jakob

Will Dicharry wrote:
> Hi All,
>
> I neglected the FindHDF5 module for a while because I was shifted to 
> another project that was kind of far from CMake work, but there were a 
> few requests for fixes features to the module. I think I have a file 
> that has the important ones. It's attached to this message. Brad, I 
> know I had CVS access, but I don't think I have Git access. If you'd 
> like me to push my commits, I can but I think I'd need you to 
> reauthorize me to do so.
>
> The changes to this module are:
> 1. Tim Gallagher's (cc'd) patch to handle the HDF5 Fortran library.
> 2. hdf5_hl is now an optional component that can be requested like the 
> language bindings.
> 3. If I can find hdf5-config.cmake from the native HDF5 cmake build, I 
> use it.
>
> I'd like it if the people I cc'd (who appear to be interested in at 
> least one of these features) could try it and see if it works for 
> them. I'd also like to know if I did #3 correctly, since there don't 
> really appear to be any modules in the CMake repository that wrap 
> around config files that don't set the variables defined in the Find 
> module.
>
> Thanks,
> Will
> ------------------------------------------------------------------------
>
> _______________________________________________
> 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://www.cmake.org/mailman/listinfo/cmake