Problem 1485

Summary:	"aida-config --implementation" doesn't work
Product:	Geant4	Reporter:	Mojca Miklavec <mojca.miklavec.lists>
Component:	cmake	Assignee:	Ivana.Hrivnacova
Status:	RESOLVED FIXED
Severity:	trivial
Priority:	P5
Version:	other
Hardware:	Apple
OS:	Mac OS X

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.