On Sat, 6 Mar 2010 23:08:12 +0100 Joel Klinghed <the...@spawned.biz> said:
> On Tue, 2 Mar 2010 10:51:27 +1100 > Carsten Haitzler (The Rasterman) <ras...@rasterman.com> wrote: > > > On Mon, 1 Mar 2010 14:17:03 -0300 Gustavo Sverzut Barbieri > > <barbi...@profusion.mobi> said: > > > > > On Mon, Mar 1, 2010 at 2:00 PM, Carsten Haitzler > > > <ras...@rasterman.com> wrote: > > > > On Sun, 28 Feb 2010 17:22:13 +0100 Joel Klinghed > > > > <the...@spawned.biz> said: > > > > > > > >> Hi. > > > >> > > > >> The attached patch adds back some options I missed in the new > > > >> enlightenment_remote, namely: > > > >> -desks-get, -desktop-bg-add, -desktop-bg-del and > > > >> -desktop-bg-list. > > > >> > > > >> For this I added: > > > >> org.enlightenment.wm.Desktop.GetVirtualCount > > > >> org.enlightenment.wm.Desktop.Background.Add > > > >> org.enlightenment.wm.Desktop.Background.Del > > > >> org.enlightenment.wm.Desktop.Background.List > > > > > > > > don't mean to discourage.. but this has been aired on this list a > > > > few times now... these - they should go into a module that > > > > extends e's dbus calls - not into core. we didn't get rid of > > > > e_ipc_handlers.h just to have it all come back as dbus instead. > > > > extra modules to extend the dbus calls - load them if u want the > > > > extra dbus calls. :) > > > > > > And as Rasterman already said in couple of places, to avoid > > > exponential grow in number of modules just to handle dbus, we'd like > > > to see a patch that converts e/src/modules/msgbus_lang to a more > > > generic dbus helpers. At least the core calls could be added there > > > without the need to create another module. (ie: just mv msgbus_lang > > > to msgbus and add your stuff there) > > > > yeah. collect them together. not 1 module per call. like > > "msgbus-extras" for a whole lot of extra calls that are non-core but > > generally very useful - maybe a "msgbus-hacker" for stuff to twiddle > > deep down things that very few would ever want etc. etc. > > > > Something along the lines of attached then? awesome! i know i've been a little bitchy about this... but you hit the nail on the head wonderfully here. just right! kudos for doing that > Just named the new module msgbus, it includes old msgbus_lang and the > four functions I wanted. The patch does remove all references to > msgbus_lang but does not actually remove the files - also, I could not > figure out how to add a binary file to the patch in a nice way, so copy > msgbus_lang/e-module-msgbus_lang.edj to msgbus/e-module-msgbus.edj for > the module icon. sure! i'll put it in now and do the needed things. thanks! now you are happy, and i am happy.. and hopefully everyone else too! :) and you made a good step in making that "throw in all those misc extra dbus calls here" module so there's no more bitching about it! :) -- ------------- Codito, ergo sum - "I code, therefore I am" -------------- The Rasterman (Carsten Haitzler) ras...@rasterman.com ------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
