Enlightenment SVN <no-re...@enlightenment.org> wrote:

>Log:
>EPhysics: add BulletPhysics instructions
>  
>  Add BulletPhysics build and install instructions and re-enable
>  ephysics.
>  
>
>Author:       dorileo
>Date:         2013-01-04 10:22:29 -0800 (Fri, 04 Jan 2013)
>New Revision: 82221
>Trac:         http://trac.enlightenment.org/e/changeset/82221
>
>Modified:
>  trunk/efl/README trunk/efl/configure.ac 
>
>Modified: trunk/efl/README
>===================================================================
>--- trunk/efl/README   2013-01-04 18:00:12 UTC (rev 82220)
>+++ trunk/efl/README   2013-01-04 18:22:29 UTC (rev 82221)
>@@ -11,7 +11,44 @@
> developer man have such as data structures, communication, rendering,
> widgets and more.
> 
>+BULLET PHYSICS DEPENDENCY:
>------------------------------------------------------------------------------
>+EFL comes with EPhysics(a physics wrapper library) enabled by default,
>to
>+build it the user must have BulletPhysics engine installed.
>+
>+More informations about BulletPhysics can be obtained in the upstream
>project
>+web site at: http://bulletphysics.org.
>+
>+We have received many reports about BulletPhysics installation and
>distros
>+packages in bad shape, some without even a package. If your distro
>doesn't
>+ship a BulletPhysics package or you want to build it from source code
>follow the
>+instructions below:
>+
>+* Required Packages:
>+You should have cmake installed. Bullet comes with autotools and cmake
>build
>+systems, do not use the autotools alternative, it's unstable, bogus
>and hasn't
>+been maintained for quite some time.
>+
>+* Download the tarball from:
>+http://code.google.com/p/bullet/downloads/list
>+
>+NOTE: the current supported version is 2.80 or greater.
>+
>+* Compiling and Installing:
>+Uncompress it to(say) ~/bullet and:
>+
>+$ cd ~/bullet/build
>+$ cmake .. -DBUILD_CPU_DEMOS=OFF -DBUILD_DEMOS=OFF
>-DBUILD_SHARED_LIBS=ON
>+$ make
>+$ sudo make install
>+
>+* Ubuntu Users:
>+Alternatively ubuntu users have the option to install the
>BulletPhysics from
>+our oficial EFL PPA:
>+
>+https://launchpad.net/~efl/+archive/trunk
>+
>+------------------------------------------------------------------------------
> COMPILING AND INSTALLING:
> 
>   ./configure
>
>Modified: trunk/efl/configure.ac
>===================================================================
>--- trunk/efl/configure.ac     2013-01-04 18:00:12 UTC (rev 82220)
>+++ trunk/efl/configure.ac     2013-01-04 18:22:29 UTC (rev 82221)
>@@ -3229,7 +3229,7 @@
>        want_physics="no"
>     fi
>    ],
>-   [want_physics="no"])
>+   [want_physics="yes"])
> 
> EFL_LIB_START_OPTIONAL([EPhysics], [test "${want_physics}" = "yes"])
> 
>
>
>------------------------------------------------------------------------------
>Master HTML5, CSS3, ASP.NET, MVC, AJAX, Knockout.js, Web API and
>much more. Get web development skills now with LearnDevNow -
>350+ hours of step-by-step video tutorials by Microsoft MVPs and
>experts.
>SALE $99.99 this month only -- learn more at:
>http://p.sf.net/sfu/learnmore_122812
>_______________________________________________
>enlightenment-svn mailing list
>enlightenment-...@lists.sourceforge.net
>https://lists.sourceforge.net/lists/listinfo/enlightenment-svn

These changes are ok...Except that you now re-enabled physics by default :-( 

IMO this should not be the case.

 Physics is not a requirement for e17 and to my knowledge will not be one for 
e18 either. As it is not a hard-dep, it really should be optional. By turning 
on physics by default, you are now (sort of) requiring that anyone who 
downloads and tries to build Efl will have to have Bullet installed...even tho 
the physics stuff is not actually required. The reason I say that is because 
the average user will not know how to disable it.

A better approach is to have it disabled by default and those more advanced 
users can pass an enable option is they want to play/experiment with it.

I am sure some people would have tossed a fit if I had set the wayland optkons 
to be enabled by default (thus requiring average users to install the wayland 
libraries) even tho they are not Actually required.

So I would love to hear the reasoning behind this option being enabled by 
default...


dh
------------------------------------------------------------------------------
Master HTML5, CSS3, ASP.NET, MVC, AJAX, Knockout.js, Web API and
much more. Get web development skills now with LearnDevNow -
350+ hours of step-by-step video tutorials by Microsoft MVPs and experts.
SALE $99.99 this month only -- learn more at:
http://p.sf.net/sfu/learnmore_122812
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to