On 2015-10-22 13:30-0400 Brad King wrote:

On 10/22/2015 11:28 AM, Orion Poplawski wrote:
This appears to have broken plplot's ada build on Fedora.

FYI - builds still fail with cmake 3.4.0-rc2.  Have had time to look at it
closer.  plplot issue seems to be triggered by a change in Ada_FLAGS:

-Ada_FLAGS =
-I/home/orion/fedora/plplot/plplot-5.11.1/build-3.3.2/examples/ada
-I/home/orion/fedora/plplot/plplot-5.11.1/bindings/ada
+Ada_FLAGS =

but plplot I believe has custom Ada cmake platform support.  I am still
concerned about possible regressions here.

Plplot's Ada support uses CMake internal APIs so it is plplot's
responsibility to adapt to our changes:

https://cmake.org/gitweb?p=cmake.git;a=blob;f=Modules/CMakeAddNewLanguage.txt;hb=v3.4.0-rc2
Maintainers of external language support are responsible for porting
it to each version of CMake as upstream changes are made.

Our 3.4.0-rc2 release notes point out a change likely causing this problem:

https://cmake.org/gitweb?p=cmake.git;a=blob;f=Help/release/3.4.rst;hb=v3.4.0-rc2#l271
* The internal "CMAKE_<LANG>_COMPILE_OBJECT" rule variable now
  substitutes compiler include flags in a separate "<INCLUDES>"
  placeholder instead of the main "<FLAGS>" placeholder.

Where Plplot currently writes:

 SET(CMAKE_Ada_COMPILE_OBJECT
   "<CMAKE_Ada_COMPILER> <FLAGS> -c <SOURCE> -o <OBJECT>
   ")

try:

 if(NOT CMAKE_VERSION VERSION_LESS 3.4)
   set(CMAKE_Ada_COMPILE_OBJECT
     "<CMAKE_Ada_COMPILER> <INCLUDES> <FLAGS> -c <SOURCE> -o <OBJECT>")
 else()
   set(CMAKE_Ada_COMPILE_OBJECT
     "<CMAKE_Ada_COMPILER> <FLAGS> -c <SOURCE> -o <OBJECT>")
 endif()

Hi Brad:

I will give your suggestion a try.

Alan
__________________________
Alan W. Irwin

Astronomical research affiliation with Department of Physics and Astronomy,
University of Victoria (astrowww.phys.uvic.ca).

Programming affiliations with the FreeEOS equation-of-state
implementation for stellar interiors (freeeos.sf.net); the Time
Ephemerides project (timeephem.sf.net); PLplot scientific plotting
software package (plplot.sf.net); the libLASi project
(unifont.org/lasi); the Loads of Linux Links project (loll.sf.net);
and the Linux Brochure Project (lbproject.sf.net).
__________________________

Linux-powered Science
__________________________
--

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

Reply via email to