On Wed, 9 Jan 2013 19:05:29 +0530 Arvind R <arvin...@gmail.com> said:

> Hi all,
> 
> eina_prefix_lib_get() in _get_player.c assumes emotion modules are
> installed in ${prefix}/lib and hence cannot find the default generic
> player. This patch fixes that.

this patch is wrong. this means emotion is not relocatable. eina_prefix exists
for a reason. to go FIND the installed location at runtime dynamically.

eg.
./configure --prefix=/opt/xxx
<do all the build and install>

mv /opt/xxx /usr/local/xxx

and it still WORKS!

> 
> Arvind
> ---
>  diff -Nru a/src/modules/generic/emotion_generic.c
> b/src/modules/generic/emotion_generic.c
> --- a/src/modules/generic/emotion_generic.c   2013-01-04
> 01:57:17.000000000 +0530 +++ b/src/modules/generic/emotion_generic.c
> 2013-01-09 18:31:55.000000000 +0530 @@ -57,7 +57,7 @@
>  _get_player(const char *name)
>  {
>     const char *selected_name = NULL;
> -   const char *libdir = eina_prefix_lib_get(pfx);
> +   const char *libdir = PACKAGE_LIB_DIR;
>     static char buf[PATH_MAX];
>     int i;
> 
> ------------------------------------------------------------------------------
> Master Java SE, Java EE, Eclipse, Spring, Hibernate, JavaScript, jQuery
> and much more. Keep your Java skills current with LearnJavaNow -
> 200+ hours of step-by-step video tutorials by Java experts.
> SALE $49.99 this month only -- learn more at:
> http://p.sf.net/sfu/learnmore_122612 
> _______________________________________________
> enlightenment-users mailing list
> enlightenment-users@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-users
> 


-- 
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler)    ras...@rasterman.com


------------------------------------------------------------------------------
Master Java SE, Java EE, Eclipse, Spring, Hibernate, JavaScript, jQuery
and much more. Keep your Java skills current with LearnJavaNow -
200+ hours of step-by-step video tutorials by Java experts.
SALE $49.99 this month only -- learn more at:
http://p.sf.net/sfu/learnmore_122612 
_______________________________________________
enlightenment-users mailing list
enlightenment-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-users

Reply via email to