Scratch that, I found it. Wasn't looking in the right places! The export command works for Fortran, so that's not an issue.
I'll take a look at the testing and see if I can get that going. I'll send along a patch when I get it worked out. Thanks, Tim ----- Original Message ----- From: "Tim Gallagher" <tim.gallag...@gatech.edu> To: "Brad King" <brad.k...@kitware.com> Cc: cmake-developers@cmake.org Sent: Tuesday, November 4, 2014 4:08:49 PM Subject: Re: [cmake-developers] Assembly/preprocessed targets for Fortran Hi Brad, Thanks for the info. What does the CMAKE_EXPORT_COMPILE_COMMANDS do? Or maybe more precisely, what output should I expect when it is set to ON instead of OFF? I tried it with my changes and nothing breaks, but I also don't see any difference in outputs with it set to ON so maybe I am missing something. I'll also dig into the testing to see what it would take to add the coverage. Thanks again, Tim ----- Original Message ----- From: "Brad King" <brad.k...@kitware.com> To: "tim gallagher" <tim.gallag...@gatech.edu>, cmake-developers@cmake.org Sent: Tuesday, November 4, 2014 3:45:39 PM Subject: Re: [cmake-developers] Assembly/preprocessed targets for Fortran On 11/04/2014 03:19 PM, Tim Gallagher wrote: > I looked through the CMake source code and it didn't seem that > hard to make it work correctly for Fortran also. I made the > changes on my local branch and it works great for the Intel > and GNU compiler suites on Linux (those are the only ones I > have access to). I modified the other compiler modules on the > assumption that the options were the same for C, C++ and Fortran > source files (which is true for Intel and GNU). Great, thanks for looking into this. > adding "Fortran" to the check for "C" and "C++" when assigning > the `lang_is_c_or_cxx` variable. That variable name should change Perhaps "lang_has_preprocessor"? You could also add bool const lang_has_assembly = lang_has_preprocessor; and update each use of the variable to use the proper name. In the future the answer may not be the same. Watch out for other uses of lang_is_c_or_cxx, like for CMAKE_EXPORT_COMPILE_COMMANDS. Check if each use case makes sense for Fortran. > 2) How would/could I go about testing the changes to the other > compilers/platforms that I do not have personal access to? You could look at adding coverage of these make targets to the test suite under the proper conditions. That will likely be a harder change than your main fix though. Thanks, -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 -- 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