[cmake-developers] Assembly/preprocessed targets for Fortran

[Tim Gallagher]

Here's to hoping 3rd time's the charm...

Also, what version of gfortran do you have that requires both -E and -cpp to do the preprocessing? I don't need that on my version, I'm using 4.7.1. There may need to be more sophisticated logic in the Compiler module to add -cpp to the command line for versions that require it.

Tim

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

On 11/05/2014 12:14 PM, Tim Gallagher wrote:
> I have attached an updated patch

Thanks!  Please split this into two patches.  The first one should
do the refactoring of the variable name and corresponding logic
with no functionality changes.  The second one can add the Fortran
feature.

Also please keep C++ source lines to 79 columns or below.

The FortranOnly test fails for me with:

 f95: error: gfortran does not support -E without -cpp

because it doesn't enable preprocessing for lower-case extensions.
You'll need to add another .F test source with an upper-case
extension to activate preprocessing without special flags.

Thanks,
-Brad

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