On Sun, 30 Jan 2011, Carsten Haitzler (The Rasterman) wrote:

> On Sun, 30 Jan 2011 09:47:12 +0100 (CET) Vincent Torri <vto...@univ-evry.fr>
> said:
>
>>
>>
>> On Sat, 29 Jan 2011, Enlightenment SVN wrote:
>>
>>> Log:
>>> lock externals to major version of edje.
>>
>>
>>
>>> -EDJE_VERSION="1.0.0"
>>> +EDJE_VERSION=`pkg-config edje --modversion | awk -F . '{printf("%s.0.0",
>>> $1);}'`
>>
>> in shell, i have 1.0.0, but when i display the value of EDJE_VERSION, i
>> get
>>
>> yes.0.0
>
> wtf? how do you get yes? that's a bit of shell.. and $1 is the first field...
> totally bizarre. i get 1.0.0 here. how does the simple 1 liner + awk get
> yes.0.0 for you and 1.0.0 for me.. and.. your m4 gets 1.0.0 always? how do 
> they
> differ so they both always get 1.0.0??? first... i'd like to know why it 
> doesnt
> get the major version right. it's the same issue lurking in both

i added:

AC_MSG_WARN([$EDJE_VERSION])

to check the value (I also checked in edje_external/Makefile te value of 
pkgdir). I'll investigate.

>
>
> one thing you are right on - i should use ${PKG_CONFIG} .

after PKG_PROG_PKG_CONFIG, and after edje is checked too.

Vincent

> now what we need is EDJE_VERSION to be 1.0.0 for edje 1.x.x and 2.0.0 for 
> 2.x.x
> etc. etc. - your changes are a bit more extensive than that. lets first figure
> out the "yes" thing. why "yes" for you and "1" for me?
>
>> i don't know how to fix that.
>>
>> below is first a better approach of getting EDJE_VERSION (that is, does
>> that only if edje external is chosen)
>>
>> Vincent
>>
>> Index: m4/efl_edje_external.m4
>> ===================================================================
>> --- m4/efl_edje_external.m4  (revision 56398)
>> +++ m4/efl_edje_external.m4  (working copy)
>> @@ -29,7 +29,7 @@
>>   _efl_enable_edje_external="no"
>>   if ! test "x${_efl_want_edje_external}" = "xno" ; then
>>      PKG_CHECK_MODULES([EDJE_EXTERNAL],
>> -      [edje],
>> +      [edje >= 1.0.0],
>>         [_efl_enable_edje_external="yes"],
>>         [_efl_enable_edje_external="no"])
>>   fi
>> @@ -41,6 +41,16 @@
>>      AC_MSG_ERROR([Edje EXTERNAL support requested, but Edje not found by
>> pkg-config.]) fi
>>
>> +if test "x${_efl_enable_edje_external}" = "xyes" ; then
>> +   EDJE_VERSION=`${PKG_CONFIG} edje --modversion | awk -F . '{printf("%
>> s.0.0", $1);}'`
>> +   AC_MSG_WARN([$host_os-$host_cpu])
>> +   AC_MSG_WARN([$EDJE_VERSION])
>> +   MODULE_EDJE="$host_os-$host_cpu-${EDJE_VERSION}"
>> +   AC_DEFINE_UNQUOTED(MODULE_EDJE, "$MODULE_EDJE", "Edje module
>> architecture") +fi
>> +
>> +AC_SUBST(MODULE_EDJE)
>> +
>>   AM_CONDITIONAL([ENABLE_EDJE_EXTERNAL], [test "x
>> ${_efl_enable_edje_external}" = "xyes"])
>>
>>   AS_IF([test "x${_efl_enable_edje_external}" = "xyes"], [$1], [$2])
>> Index: configure.ac
>> ===================================================================
>> --- configure.ac     (revision 56398)
>> +++ configure.ac     (working copy)
>> @@ -54,25 +54,19 @@
>>   AC_SUBST(VMAJ)
>>
>>   ### Needed information
>> -EDJE_VERSION=`pkg-config edje --modversion | awk -F . '{printf("%s.0.0",
>> $1);}'`
>>
>>   want_vlc="no"
>>   case "$host_os" in
>>      mingw* | cegcc*)
>> -      MODULE_EDJE="$host_os-$host_cpu"
>>         want_xine="no"
>>         want_gstreamer="yes"
>>         ;;
>>      *)
>> -      MODULE_EDJE="$host_os-$host_cpu-${EDJE_VERSION}"
>>         want_xine="yes"
>>         want_gstreamer="yes"
>>         ;;
>>   esac
>>
>> -AC_SUBST(MODULE_EDJE)
>> -AC_DEFINE_UNQUOTED(MODULE_EDJE, "$MODULE_EDJE", "Edje module architecture")
>> -
>>   requirement_emotion=""
>>
>>   ### Additional options to configure
>> @@ -117,7 +111,7 @@
>>   requirement_emotion="ecore >= 1.0.0 evas >= 1.0.0 eina >= 1.0.0"
>>
>>   if test "x${have_emotion_test}" = "xyes" ; then
>> -   PKG_CHECK_MODULES(EMOTION_BIN, [evas >= 1.0.0 ecore >= 1.0.0 ecore-evas
>>> = 1.0.0 edje >= ${EDJE_VERSION}])
>> +   PKG_CHECK_MODULES(EMOTION_BIN, [evas >= 1.0.0 ecore >= 1.0.0 ecore-evas
>>> = 1.0.0 edje >= 1.0.0]) fi
>>
>>   PKG_CHECK_MODULES(ECORE_X, [ecore-x >= 1.0.0], [have_ecore_x="yes"],
>> [have_ecore_x="no"])
>>
>>
>> ------------------------------------------------------------------------------
>> Special Offer-- Download ArcSight Logger for FREE (a $49 USD value)!
>> Finally, a world-class log management solution at an even better price-free!
>> Download using promo code Free_Logger_4_Dev2Dev. Offer expires
>> February 28th, so secure your free ArcSight Logger TODAY!
>> http://p.sf.net/sfu/arcsight-sfd2d
>> _______________________________________________
>> enlightenment-devel mailing list
>> enlightenment-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>>
>
>
> -- 
> ------------- Codito, ergo sum - "I code, therefore I am" --------------
> The Rasterman (Carsten Haitzler)    ras...@rasterman.com
>
>

------------------------------------------------------------------------------
Special Offer-- Download ArcSight Logger for FREE (a $49 USD value)!
Finally, a world-class log management solution at an even better price-free!
Download using promo code Free_Logger_4_Dev2Dev. Offer expires 
February 28th, so secure your free ArcSight Logger TODAY! 
http://p.sf.net/sfu/arcsight-sfd2d
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to