Re: [E-devel] E SVN: rui IN trunk/eeze: . src/lib src/modules src/tests

2012-09-23 Thread Vincent Torri
On Sun, Sep 23, 2012 at 2:17 PM, Enlightenment SVN
 wrote:
> Log:
>  * Add support to eeze_sensor lib in the spec
>* Fix dist build by including a header file
>* svn:ignore on .deps, Makefile and Makefile.in on modules and tests.
>
>
> Author:   rui
> Date: 2012-09-23 05:17:22 -0700 (Sun, 23 Sep 2012)
> New Revision: 77021
> Trac: http://trac.enlightenment.org/e/changeset/77021
>
> Modified:
>   trunk/eeze/eeze.spec.in trunk/eeze/src/lib/Makefile.am 
> trunk/eeze/src/modules/ trunk/eeze/src/tests/
>
> Modified: trunk/eeze/eeze.spec.in
> ===
> --- trunk/eeze/eeze.spec.in 2012-09-22 18:29:03 UTC (rev 77020)
> +++ trunk/eeze/eeze.spec.in 2012-09-23 12:17:22 UTC (rev 77021)
> @@ -73,6 +73,8 @@
>  %{_libdir}/*.a
>  %{_libdir}/*.so
>  %{_libdir}/*.la
> +%{_libdir}/*/*.so
> +%{_libdir}/*/*.la
>  %{_libdir}/pkgconfig/*
>
>  %changelog
>
> Modified: trunk/eeze/src/lib/Makefile.am
> ===
> --- trunk/eeze/src/lib/Makefile.am  2012-09-22 18:29:03 UTC (rev 77020)
> +++ trunk/eeze/src/lib/Makefile.am  2012-09-23 12:17:22 UTC (rev 77021)
> @@ -3,7 +3,7 @@
>  AM_CPPFLAGS = @EEZE_CFLAGS@ \
>  -DPACKAGE_LIB_DIR=\"$(libdir)\"
>
> -includes_HEADERS = Eeze.h Eeze_Net.h Eeze_Sensor.h
> +includes_HEADERS = Eeze.h Eeze_Net.h Eeze_Sensor.h eeze_sensor_private.h

no :) By doing that, you *install* eeze_sensor_private.h, which is not
what we want. Add it into  libeeze_la_SOURCES

Vincent

>
>  libeeze_la_SOURCES = \
>  eeze_main.c \
>
>
> Property changes on: trunk/eeze/src/modules
> ___
> Added: svn:ignore
>+ .deps
> Makefile
> Makefile.in
>
>
>
> Property changes on: trunk/eeze/src/tests
> ___
> Added: svn:ignore
>+ .deps
> Makefile
> Makefile.in
>
>
>
> --
> Everyone hates slow websites. So do we.
> Make your web apps faster with AppDynamics
> Download AppDynamics Lite for free today:
> http://ad.doubleclick.net/clk;258768047;13503038;j?
> http://info.appdynamics.com/FreeJavaPerformanceDownload.html
> ___
> enlightenment-svn mailing list
> enlightenment-...@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-svn

--
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://ad.doubleclick.net/clk;258768047;13503038;j?
http://info.appdynamics.com/FreeJavaPerformanceDownload.html
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] E SVN: rui IN trunk/eeze: . src/lib src/modules src/tests

2012-09-23 Thread Rui Miguel Silva Seabra
On Sun, 23 Sep 2012 14:27:27 +0200
Vincent Torri  wrote:

> On Sun, Sep 23, 2012 at 2:17 PM, Enlightenment SVN
>  wrote:
> > Log:
> >  * Add support to eeze_sensor lib in the spec
> >* Fix dist build by including a header file
> >* svn:ignore on .deps, Makefile and Makefile.in on modules and
> > tests.
> >
> >
> > Author:   rui
> > Date: 2012-09-23 05:17:22 -0700 (Sun, 23 Sep 2012)
> > New Revision: 77021
> > Trac: http://trac.enlightenment.org/e/changeset/77021
> >
> > Modified:
> >   trunk/eeze/eeze.spec.in trunk/eeze/src/lib/Makefile.am
> > trunk/eeze/src/modules/ trunk/eeze/src/tests/
> >
> > Modified: trunk/eeze/eeze.spec.in
> > ===
> > --- trunk/eeze/eeze.spec.in 2012-09-22 18:29:03 UTC (rev 77020)
> > +++ trunk/eeze/eeze.spec.in 2012-09-23 12:17:22 UTC (rev 77021)
> > @@ -73,6 +73,8 @@
> >  %{_libdir}/*.a
> >  %{_libdir}/*.so
> >  %{_libdir}/*.la
> > +%{_libdir}/*/*.so
> > +%{_libdir}/*/*.la
> >  %{_libdir}/pkgconfig/*
> >
> >  %changelog
> >
> > Modified: trunk/eeze/src/lib/Makefile.am
> > ===
> > --- trunk/eeze/src/lib/Makefile.am  2012-09-22 18:29:03 UTC
> > (rev 77020) +++ trunk/eeze/src/lib/Makefile.am  2012-09-23
> > 12:17:22 UTC (rev 77021) @@ -3,7 +3,7 @@
> >  AM_CPPFLAGS = @EEZE_CFLAGS@ \
> >  -DPACKAGE_LIB_DIR=\"$(libdir)\"
> >
> > -includes_HEADERS = Eeze.h Eeze_Net.h Eeze_Sensor.h
> > +includes_HEADERS = Eeze.h Eeze_Net.h Eeze_Sensor.h
> > eeze_sensor_private.h
> 
> no :) By doing that, you *install* eeze_sensor_private.h, which is not
> what we want. Add it into  libeeze_la_SOURCES

oops! Correct fix committed.

Thanks. :)

--
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://ad.doubleclick.net/clk;258768047;13503038;j?
http://info.appdynamics.com/FreeJavaPerformanceDownload.html
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Merging Evas-Eo to upstream

2012-09-23 Thread Gustavo Sverzut Barbieri
On Sunday, September 23, 2012, Tom Hacohen wrote:

> On 23/09/12 00:13, Gustavo Sverzut Barbieri wrote:
>
>> it shouldn't, but it always do :-(
>>
>> we don't need major releases, but at least minor we should
>>
>>
>>
> It's not the wait I care about (well, not entirely). The main problem, as
> I said, is the annoying merge process because of the relayouting of files.
> I.e evas/src/lib becoming src/lib/evas and evas/src/modules become
> src/modules/evas... The merge will involve a lot of manual intervention
> which can be a source of bugs.


Do a single diff and a sed to change the path of files. :-)

Another option is to be sure it's stable and just copy files over.


> --
> Tom.
>


-- 
Gustavo Sverzut Barbieri
http://profusion.mobi embedded systems
--
MSN: barbi...@gmail.com
Skype: gsbarbieri
Mobile: +55 (19) 9225-2202
--
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://ad.doubleclick.net/clk;258768047;13503038;j?
http://info.appdynamics.com/FreeJavaPerformanceDownload.html
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Merging Evas-Eo to upstream

2012-09-23 Thread Tom Hacohen
On 23/09/12 16:03, Gustavo Sverzut Barbieri wrote:
> Do a single diff and a sed to change the path of files. :-)

Yeah, I thought about that, but that's not optimal.
>
> Another option is to be sure it's stable and just copy files over.
>

... That's what I'm trying to avoid.

--
Tom.


--
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://ad.doubleclick.net/clk;258768047;13503038;j?
http://info.appdynamics.com/FreeJavaPerformanceDownload.html
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Merging Evas-Eo to upstream

2012-09-23 Thread Lucas De Marchi
On Sun, Sep 23, 2012 at 11:10 AM, Tom Hacohen  wrote:
> On 23/09/12 16:03, Gustavo Sverzut Barbieri wrote:
>> Do a single diff and a sed to change the path of files. :-)
>
> Yeah, I thought about that, but that's not optimal.

Or use tools that can take prefix and --directory. See svn-git-am
using git-apply or git-apply(1)


>>
>> Another option is to be sure it's stable and just copy files over.
>>

meh. it's easier to just apply the patches.


Lucas De Marchi

--
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://ad.doubleclick.net/clk;258768047;13503038;j?
http://info.appdynamics.com/FreeJavaPerformanceDownload.html
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Merging Evas-Eo to upstream

2012-09-23 Thread Lucas De Marchi
On Sun, Sep 23, 2012 at 3:29 AM, Tom Hacohen  wrote:
> On 23/09/12 00:13, Gustavo Sverzut Barbieri wrote:
>> it shouldn't, but it always do :-(
>>
>> we don't need major releases, but at least minor we should
>>
>>
>
> It's not the wait I care about (well, not entirely). The main problem,
> as I said, is the annoying merge process because of the relayouting of
> files. I.e evas/src/lib becoming src/lib/evas and evas/src/modules
> become src/modules/evas... The merge will involve a lot of manual
> intervention which can be a source of bugs.

It sounds like an excuse to not do the right thing. The manual
intervention should be very minimal and it shouldn't be a source of
bugs... once it compiles, it should work as if the paths haven't
changed.


Lucas De Marchi

--
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://ad.doubleclick.net/clk;258768047;13503038;j?
http://info.appdynamics.com/FreeJavaPerformanceDownload.html
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Merging Evas-Eo to upstream

2012-09-23 Thread Tom Hacohen
On 23/09/12 23:14, Lucas De Marchi wrote:
> It sounds like an excuse to not do the right thing. The manual
> intervention should be very minimal and it shouldn't be a source of
> bugs... once it compiles, it should work as if the paths haven't
> changed.

It's easier to mis-merge, and that's a source of bugs. Everything that
is not completely automatic is a potential source of bugs. The word
"should" is responsible for 87% of all software bugs, so I don't really
trust that.

With that being said, I do trust you, and you said that svn-git-am can
handle it gracefully, lets just hope it does.

I'm fine with waiting with applying the patches, but lets have a
schedule for that.

What's the plan for this merge? When will I be able to merge stuff? And
btw, do we make sure history is correctly kept for source files? Cause
that can be crazily annoying.

--
Tom.

--
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://ad.doubleclick.net/clk;258768047;13503038;j?
http://info.appdynamics.com/FreeJavaPerformanceDownload.html
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Merging Evas-Eo to upstream

2012-09-23 Thread Cedric BAIL
On Sun, Sep 23, 2012 at 6:13 AM, Gustavo Sverzut Barbieri
 wrote:
> On Sat, Sep 22, 2012 at 6:11 PM, Tom Hacohen  wrote:
>> On 23/09/12 00:10, Gustavo Sverzut Barbieri wrote:
>>> On Sat, Sep 22, 2012 at 5:46 PM, Tom Hacohen  wrote:
 On 22/09/12 17:05, Gustavo Sverzut Barbieri wrote:
> Isn't better to wait merge, release, then add eo?

 Hm... what? I don't quite get it. We just released, we waited until
 after the release just for that, why not upstream our changes that add Eo?
>>>
>>> we wouldn't know if the breakage is due merge or eo.
>>>
>>> IMO: release merge, add eo and release again
>>
>> It'll break the layout of the libraries and thus will make merging very
>> annoying. Also, breakages should be different, i.e the change of layout
>> shouldn't cause any runtime issues.
>
> it shouldn't, but it always do :-(
>
> we don't need major releases, but at least minor we should

I have been wondering about this subject for the weekend. Maybe we
could do a packager preview or something like that. I don't like the
idea of doing a 1.7.1 based on current trunk (1.7.1 should in my
opinion only come from branch) and I don't really like the idea of
doing a 1.8.0 so soon after 1.7. A packager preview would fit the
biggest change we have done, force people to update their build script
and seems reasonable. So it would a 1.8.0-preview or something like
that.
-- 
Cedric BAIL

--
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Toolbar transverse expand set

2012-09-23 Thread Daniel Juyung Seo
This looks good to me.
Can you send a patch?

Daniel Juyung Seo (SeoZ)

On Fri, Sep 21, 2012 at 4:22 PM, Jaehwan Kim  wrote:
> Dear raster
>
> Yesterday, you discussed about toolbar with Seoz.
> Currently the vertical length of the item is decided by the min value of
> item.
> (When the toolbar is horizontal mode)
> If we want to expand the item according toolbar height, there's any API
> which set the mode.
> Since the toolbar has the horizontal/vertical mode, I can not use the word
> horizontal.
> So I decide the API name elm_toolbar_transverse_expanded_set/get.
> Please review this name is proper.
>
> Thanks
> --
> Jaehwan Kim.
> --
> Got visibility?
> Most devs has no idea what their production app looks like.
> Find out how fast your code is with AppDynamics Lite.
> http://ad.doubleclick.net/clk;262219671;13503038;y?
> http://info.appdynamics.com/FreeJavaPerformanceDownload.html
> ___
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

--
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] eio Example for eio_file_stat_ls

2012-09-23 Thread Guillaume Friloux

On 21/09/2012 17:37, Leandro Dorileo wrote:

Hi Guillaume,


On Fri, Sep 21, 2012 at 01:55:54PM +0200, Guillaume Friloux wrote:

Hello e people,
I have written a little example app for eio_file_stat_ls() that
could have its place under "trunk/eio/src/examples/" or
"trunk/EXAMPLES/eio/".
I think this could help people that needs to use it.
It also warns about the filter_cb being called in eio's thread. A
trap i stepped on by badly reading the doc (totally my fault).

Example file is attached to this mail.
/*
  * Compile :
  * gcc -g -o eio_file_stat_ls eio_file_stat_ls.c `pkg-config --cflags --libs 
eina ecore eio`
  *
  * Prepare :
  * mkdir test
  * touch test/456
  * touch test/excluded
  * touch test/backuped
  *
  * Run :
  * ./eio_file_stat_ls test
  */

#include 
#include 

struct _Filter
{
const char *name;
regex_t preg;
};

Eina_List *l_filters;

unsigned int count_filtered,
  count_passed,
  jobs;

Eina_Bool _list_cb_filter(void *data, Eio_File *handler, const 
Eina_File_Direct_Info *file);
void _list_cb(void *data, Eio_File *handler, const Eina_File_Direct_Info *file);
void _list_cb_done(void *data, Eio_File *handler);
void _list_cb_error(void *data, Eio_File *handler, int error);




You don't need these prototypes, reorder your functions.




void _free_cb(void *data)
{
return;
}

void _filter_add(const char *name)
{
struct _Filter *obj;

obj = calloc(1, sizeof(struct _Filter));
if (!obj)
  return;

if (regcomp(&(obj->preg), name, REG_EXTENDED))
  {
 fprintf(stderr, "Failed to compile regex for \"%s\"\n", name);
 free(obj);
 return;
  }
obj->name = eina_stringshare_add(name);
l_filters = eina_list_append(l_filters, obj);
}

/*
  * This function is called inside eio's thread, so dont use anything not
  * thread-safe inside!
  */
Eina_Bool
_list_cb_filter(void *data, Eio_File *handler, const Eina_File_Direct_Info 
*file)
{
char *last_slash;
struct _Filter *obj_filter;
Eina_List *l;

last_slash = strrchr(file->path, '/');

if (file->type==EINA_FILE_DIR)
  return EINA_TRUE;

EINA_LIST_FOREACH(l_filters, l, obj_filter)
  {
 if (!regexec(&(obj_filter->preg), last_slash + 1, 0, 0, 0))
   {
  eio_file_associate_add(handler, file->path, obj_filter, _free_cb);
  return EINA_TRUE;
   }
  }
count_filtered++;
return EINA_FALSE;
}

void
_list_cb(void *data, Eio_File *handler, const Eina_File_Direct_Info *file)
{
struct _Filter *obj;

if (file->type==EINA_FILE_DIR)
  {
 jobs++;
 eio_file_stat_ls(file->path,
  _list_cb_filter,
  _list_cb,
  _list_cb_done,
  _list_cb_error,
  NULL);
 return;
  }
count_passed++;

obj = eio_file_associate_find(handler, file->path);
printf("File %s matched %s\n", file->path, obj->name);
}

void
_list_cb_done(void *data, Eio_File *handler)
{
if (!--jobs)
  ecore_main_loop_quit();
}

void
_list_cb_error(void *data, Eio_File *handler, int error)
{
fprintf(stderr, "An error occured : %s\n", strerror(error));
}

int main(int argc, char **argv)
{
if (argc != 2)
  {
 fprintf(stderr, "Usage ./eio_file_stat_ls testdir\n");
 return 1;
  }

eina_init();
ecore_init();
eio_init();




Check for initializations.





l_filters = NULL;
count_filtered = 0;
count_passed = 0;
jobs = 1;



You could use a struct do describe your status data and pass it to
eio_file_stat_ls and appropriately handle it.



_filter_add("^[0-9]+$");
_filter_add("^back.*");


eio_file_stat_ls(argv[1],
 _list_cb_filter,
 _list_cb,
 _list_cb_done,
 _list_cb_error,
 NULL);

ecore_main_loop_begin();

printf("Stats :\tFiltered = %d\tPassed = %d\n", count_filtered, 
count_passed);

eio_shutdown();
ecore_shutdown();
eina_shutdown();
return 0;
}



Regards


I made the changes : http://slexy.org/view/s204s5bbvk
<>--
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel