Re: [cmake-developers] On-going UseSWIG troubles with the official versions --SOLVED

2017-04-26 Thread Alan W. Irwin
On 2017-04-25 08:54-0400 Brad King wrote: On 04/25/2017 04:08 AM, Alan W. Irwin wrote: set PREFIX to "" for all languages not specifically covered by the if and elseif blocks. Do you mean ``` diff --git a/Modules/UseSWIG.cmake b/Modules/UseSWIG.cmake index 277f4ca28a..bfe1a6f754 100644 ---

Re: [cmake-developers] Patch: Don't emit warning when config file not found

2017-04-26 Thread Brad King
On 04/25/2017 03:55 PM, Christoph Grüninger wrote: > please find attached a first attempt to implement OPTIONAL for > find_package. It is supposed to suppress all warnings and indicate the > negative result by a single line of output. Unfortunately, I still get a > warning and I could figure out

Re: [cmake-developers] On-going UseSWIG troubles with the official versions --SOLVED

2017-04-26 Thread Brad King
On 04/26/2017 04:14 AM, Alan W. Irwin wrote: > On 2017-04-25 08:54-0400 Brad King wrote: >> + else() >> +# assume empty prefix because we expect the module to be dynamically >> loaded >> +set_target_properties (${SWIG_MODULE_${name}_REAL_NAME} PROPERTIES >> PREFIX "") > > Yes, exactly.

Re: [cmake-developers] Patch: Don't emit warning when config file not found

2017-04-26 Thread Christoph Grüninger
Hi Brad, thanks for helping me out with the patch. Your wording of the message is better than my proposal. Please find attached an improved patch. Bye Christoph -- [..] Mathematicians are like theologians: we regard existence as the prime attribute of what we study. But unlike theologians, we

Re: [cmake-developers] Patch: Don't emit warning when config file not found

2017-04-26 Thread Brad King
On 04/26/2017 04:11 PM, Christoph Grüninger wrote: > thanks for helping me out with the patch. Your wording of the message is > better than my proposal. > Please find attached an improved patch. Thanks. This is a good start. Please see CONTRIBUTING.rst and open a merge request on

Re: [cmake-developers] Adding Microsoft MPI Fortran support in FindMPI

2017-04-26 Thread Christian Pfeiffer
I reckon it might be better to ignore the Fortran 90 module API for now. Adding a function for that would mean that the linked target has to be visible everywhere where the MPI package is being used with Fortran. Not to mention that all projects using the F90 module API then would need to have

Re: [cmake-developers] Requesting Advice: Installing object files

2017-04-26 Thread Brad King
On 01/18/2017 03:14 PM, Ben Boeckel wrote: > Installing the target can install the objects. [snip] On 01/18/2017 02:47 PM, Chris Bieneman wrote: > If this allowed TARGET_OBJECTS to be used in the install(FILES ...) > command, then we could create an object library and a custom install > rule for