Problem 1485 - "aida-config --implementation" doesn't work
Summary: "aida-config --implementation" doesn't work
Status: RESOLVED FIXED
Alias: None
Product: Geant4
Classification: Unclassified
Component: cmake (show other problems)
Version: other
Hardware: Apple Mac OS X
: P5 trivial
Assignee: Ivana.Hrivnacova
URL:
Depends on:
Blocks:
 
Reported: 2013-07-02 16:07 CEST by Mojca Miklavec
Modified: 2013-08-19 22:28 CEST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this problem.
Description Mojca Miklavec 2013-07-02 16:07:11 CEST
I'm compiling version 10.00.b01 on Mac OS X 10.7 using MacPorts, but the file cmake/Modules/FindAIDA.cmake tries to call aida-config --implementation which doesn't work: aida doesn't support that particular flag.

MacPorts comes with aida 3.2.1 from ftp://ftp.slac.stanford.edu/software/freehep/AIDA/v3.2.1/

My temporary workaround was the following:

--- cmake/Modules/FindAIDA.cmake.orig
+++ cmake/Modules/FindAIDA.cmake
@@ -30,10 +30,7 @@ else()
     OUTPUT_VARIABLE AIDA_LIBRARIES
     OUTPUT_STRIP_TRAILING_WHITESPACE)
 
-  execute_process(
-    COMMAND ${AIDA_CONFIG_EXECUTABLE} --implementation
-    OUTPUT_VARIABLE AIDA_IMPLEMENTATION
-    OUTPUT_STRIP_TRAILING_WHITESPACE)
+  set(AIDA_IMPLEMENTATION "MacPorts")
 
   # Make variables changeble to the advanced user
   mark_as_advanced(AIDA_CONFIG_EXECUTABLE)

but this needs a proper patch or some suggestion, where to get aida with support for "--implementation" flag.
Comment 1 Ben Morgan 2013-07-04 13:01:41 CEST
Reassigning to author of FindAIDA.cmake
Comment 2 Ivana.Hrivnacova 2013-07-05 22:30:42 CEST
The FindAIDA.cmake was tested with osc_batch AIDA implementation;
which supports the --implementation flag. We will put a protection in the file to behave correctly when this flag is not supported.
Could you, please, let me know if the option --version, which is also used in cmake is available in your package?

Thank you,
Comment 3 Mojca Miklavec 2013-07-05 22:52:44 CEST
Yes, it does.

> aida-config --help

usage: aida-config [<options>] 

where the optional <options> can be one of the following:

       -?, -h, --help : print this text
       -v, --version  : print version info

       -i, --includes : print out the list for the include directories: '-I...'
       -c, --cflags   : presently as -i or --includes
       -l, --libs     : print out the list for the libraries : '-L... -l.. -l.. ...'
       -r, --runtime  : print out the list to be added to the SharedLib Path : '<path-to-shared-libs>'

       --platform=<platform> : use <platform> for Platform string, overrides automatic determination

Now I see that aida-config actually comes from iAIDA (1.0.22), not from aida. (The author of iAIDA might be willing to add that flag if the flag is needed for some reason.)
Comment 4 Mojca Miklavec 2013-07-07 16:52:19 CEST
Author of iAIDA also offered to implement that option.
Comment 5 Mojca Miklavec 2013-07-27 22:48:00 CEST
Just a note: iAIDA now supports --implementation, so this issue is not as pressing any more. (I'm not sure how many different "aidas" exist though.)
Comment 6 Ivana.Hrivnacova 2013-08-19 12:33:42 CEST
I have modified FindAIDA.cmake so that it does not produce error messages in case aida-config does not support --version or --implementation options as these options are used only for printing info. If this info is not available its printing is just suppressed.
This change is scheduled for the next Geant4 release.
Comment 7 Mojca Miklavec 2013-08-19 22:28:24 CEST
Thank you.