#36 indication_helper acinclude.m4 add check for PEGASUS?

Usability
closed-fixed
5
2005-04-19
2005-04-08
Gareth S Bestor
No

Currently the indication_helper acinclude.m4 has checks
for PEGASUS_2_3_2 vs PEGASUS_2_4. eg

AC_DEFUN([CHECK_PEGASUS_2_3_2],
[
AC_MSG_CHECKING(for Pegasus 2.3.2)
test_CIMSERVER=`cimserver -v`
if test "$test_CIMSERVER" == "2.3.2"; then
AC_MSG_RESULT(yes)

AC_DEFINE_UNQUOTED(HAVE_PEGASUS_2_3_2,1,[Defined to 1
if Pegasus 2.3.2 is used])
else
AC_MSG_RESULT(no)

fi
]
)

These checks should probably do a 'which cimserver'
first to make sure Pegasus is even installed. Currently
this causes an error message to be displayed when
running configure when Pegasus is not installed

[...]
checking for Pegasus 2.3.2... ./configure: line 1:
cimserver: command not found
no
checking for Pegasus 2.4... ./configure: line 1:
cimserver: command not found
no

Having an new check for PEGASUS itself (ie cimserver)
might also now be useful for sfcb

low priority

Discussion

  • Logged In: YES
    user_id=1198711

    First part: full ack!
    But, why should sfcb check for Pegasus? To report a
    potential conflict?

     
  • Logged In: YES
    user_id=1215486

    sfcb shouldnt need to check for Pegasus, but my
    understanding is that the indication_helper packages can be
    used for both sfcb and Pegasus now. Currently it kinda
    _assumes_ Pegasus and is more concerend with determining
    what version of Pegasus you are running. But with this
    package now being used for sfcb too, its autoconf tests
    should check for Pegasus (vs sfcb) first, and then check for
    the Pegasus version as necessary.

     
  • Logged In: YES
    user_id=852838

    FYI:The code needs to determine the version of OpenPegasus
    b/c in 2.3.2 CMEValuteSelExp was never implemented and
    always returned false.

    Hence there is a #ifdef for the 2.3.2 pegasus (ONLY). Since
    2.3.2 is old this check might not be neccesary anymore?

     
    • assigned_to: nobody --> mihajlov
    • status: open --> open-accepted
     
  • Logged In: YES
    user_id=1198711

    cimserver is now being checked for, since CVS tag
    SBLIM_INDICATION_HELPER_0_3_1A .
    I don't see a need for an SFCB check, but suggest to leave
    the Pegasus 2.3.2 check in there for compatibility.

     
    • status: open-accepted --> closed-fixed