[cmake-developers] Assembly/preprocessed targets for Fortran

[Tim Gallagher]

Sorry about the TABs, I guess emacs defaults to it and I never noticed.

I have attached an updated patch where the tabs are removed and the test for CMAKE_EXPORT_COMPILE_COMMANDS is also removed. 

Tim

----- Original Message -----
From: "Brad King" <brad.king at kitware.com>
To: "tim gallagher" <tim.gallagher at gatech.edu>, cmake-developers at cmake.org
Sent: Wednesday, November 5, 2014 11:26:26 AM
Subject: Re: [cmake-developers] Assembly/preprocessed targets for Fortran

On 11/04/2014 06:37 PM, Tim Gallagher wrote:
> I have attached the patch to enable the targets for Fortran.

Thanks.  Please update it to avoid using hard TABs for indentation.
Also in the CompileCommandOutput test hunk:

> -project (CompileCommandOutput CXX)
> +project (CompileCommandOutput)
> +enable_language(CXX)
> +enable_language(Fortran)

there are a couple problems:

- By removing any explicit languages from the project() call
  it will enable C and CXX by default.  Use NONE to suppress
  that.

- We cannot assume that Fortran will be available.  The other
  Fortran tests are all guarded by availability of a Fortran
  compiler.

The test for CMAKE_EXPORT_COMPILE_COMMANDS was already missing
for C, so let's just skip Fortran for the test too.  They can
be fixed together as a separate change later.

Thanks,
-Brad

-------------- next part --------------
A non-text attachment was scrubbed...
Name: FortranAssemblyPreprocessor.patch
Type: text/x-patch
Size: 13938 bytes
Desc: not available
URL: <http://public.kitware.com/pipermail/cmake-developers/attachments/20141105/b234f19e/attachment-0001.bin>