On Mon, Jan 14, 2013 at 1:54 AM, Arvind R <arvin...@gmail.com> wrote: > On Mon, Jan 14, 2013 at 6:16 AM, Gustavo Sverzut Barbieri > <barbi...@profusion.mobi> wrote: >> On Sat, Jan 12, 2013 at 8:08 PM, Arvind R <arvin...@gmail.com> wrote: >>> fix compile failure due to missing definitions >> >> applied, but I wonder how you got these if generic module just builds >> statically and thus uses the definitions given to libemotion.la :-/ >> > I dislike static builds and generally try to build shared if I can -:) > Patched, of course.
While I understand that, what about actually discuss this with the ML first to understand the reasons? We have been putting lots of effort to get efl single tree right, and this includes the review of module linkage. In this case it's unexpected to have the generic loader as a module as it's not a real one. We just kept the code split to avoid the merge effort, but in reality it's just a different kind of loader, thus makes 0 sense to have it out -- IOW: remove your patch :-) It's the same with evas, the generic loader will always be builtin. As is the buffer engine. For ecore-evas I even merged buffer engine back in libecore_evas.so The only stuff open for review is eet, png and jpeg engines of Evas. They are static now, but they don't need to. The reason they are like that now is that if you don't have those, nothing will work properly... then you always need them -> move the linkage to mandatory and avoid dlopen(). Also eet links to jpeg... making the evas+jpeg "no extra overhead". And the png is pretty simple as well. It may be that we decide to link in all of the Evas modules that adds not much code and zero dependencies, thinks like BMP are more costly to have outside than inside. tl;dr; don't do these things without a real reason, ok? -- Gustavo Sverzut Barbieri http://profusion.mobi embedded systems -------------------------------------- MSN: barbi...@gmail.com Skype: gsbarbieri Mobile: +55 (19) 9225-2202 ------------------------------------------------------------------------------ Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS, MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current with LearnDevNow - 3,200 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_122412 _______________________________________________ enlightenment-users mailing list enlightenment-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-users
