Unsubscribe it. On Mon, Jan 5, 2015 at 10:49 PM, < enlightenment-devel-requ...@lists.sourceforge.net> wrote:
> Send enlightenment-devel mailing list submissions to > enlightenment-devel@lists.sourceforge.net > > To subscribe or unsubscribe via the World Wide Web, visit > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > or, via email, send a message with subject or body 'help' to > enlightenment-devel-requ...@lists.sourceforge.net > > You can reach the person managing the list at > enlightenment-devel-ow...@lists.sourceforge.net > > When replying, please edit your Subject line so it is more specific > than "Re: Contents of enlightenment-devel digest..." > > > Today's Topics: > > 1. Weekly news from the automated build and QA front > (Stefan Schmidt) > 2. Re: [EGIT] [bindings/python/python-efl] master 08/08: Provide > the uninstall command for python-efl itself (Sebastian Dransfeld) > 3. Re: [EGIT] [bindings/python/python-efl] master 08/08: Provide > the uninstall command for python-efl itself (Davide Andreoli) > 4. Re: [EGIT] [core/elementary] master 01/01: genlist: > exceptional handling. (Cedric BAIL) > 5. Re: [EGIT] [core/efl] master 01/01: curl usage better > accounts for timeouts (Tom Hacohen) > 6. Re: PATCH: duplicated XDG_DATA_DIRS fix (Thanatermesis) > > > ---------------------------------------------------------------------- > > Message: 1 > Date: Mon, 05 Jan 2015 08:52:02 +0100 > From: Stefan Schmidt <ste...@datenfreihafen.org> > Subject: [E-devel] Weekly news from the automated build and QA front > To: 'Enlightenment developer list' > <enlightenment-devel@lists.sourceforge.net> > Message-ID: <54aa42a2.10...@datenfreihafen.org> > Content-Type: text/plain; charset=windows-1252 > > Hello. > > Summary: > o Its been a few weeks since the last QA report > o Most of the new coverity issues are coming from new tests that came > with the updated coverity client on jenkins. > > > This should give everyone an overview over what has happened in the last > week on the QA front. The numbers in parentheses reflect the values from > last week to give you a trend. > > CI: > o Overall build statistic: 13.76% (10.15%) failed. > https://build.enlightenment.org/ > > clang scan-build: > o EFL scan-build reports NA (454) issues. > https://build.enlightenment.org/job/nightly_efl_clang_x86_64/lastSuccessfu > lBuild/artifact/scan-build/build/ > o Elementary scan-build reports 78 (79) issues. > > https://build.enlightenment.org/job/nightly_elm_clang_x86_64/lastSuccessfulBuild/artifact/scan-build/build > > Unit tests: > o 416 (416) unit tests for efl and none failing > > Coverage: > o EFL total coverage is at 34.2% (34.1%) lines and 37.3% (37.2%) functions > https://build.enlightenment.org/view/Test%20Coverage/ > > Coverity: > o EFL: Outstanding defects 71 (47) with a density of 0.10 (0.07) > o Elm: Outstanding defects 6 (0) with a density of 0.02 (0) > o Evas Generic Loaders: Outstanding defects 2 (0) with a density of 0.02 > (0) > o Emotion Generic Players: Outstanding defects 0 (0) with a density of > 0 (0) > o Enlightenment: Outstanding defects 40 (36) with a density of 0.15 (0.13) > o Terminology: Outstanding defects 1 (0) with a density of 0.01 (0) > > Phab: > o Total bug count: 514 (488) > https://phab.enlightenment.org/maniphest/report/burn/ > o Pending patch reviews: 52 (57) > > regards > Stefan Schmidt > > > > ------------------------------ > > Message: 2 > Date: Mon, 05 Jan 2015 09:38:03 +0100 > From: Sebastian Dransfeld <s...@tango.flipp.net> > Subject: Re: [E-devel] [EGIT] [bindings/python/python-efl] master > 08/08: Provide the uninstall command for python-efl itself > To: Enlightenment developer list > <enlightenment-devel@lists.sourceforge.net> > Message-ID: <54aa4d6b.7080...@tango.flipp.net> > Content-Type: text/plain; charset=utf-8; format=flowed > > On 01/04/2015 03:56 PM, Dave Andreoli wrote: > > - python setup.py install > > + (sido) python setup.py install > > > > > > sido? > > S. > > > > ------------------------------ > > Message: 3 > Date: Mon, 5 Jan 2015 10:25:39 +0100 > From: Davide Andreoli <d...@gurumeditation.it> > Subject: Re: [E-devel] [EGIT] [bindings/python/python-efl] master > 08/08: Provide the uninstall command for python-efl itself > To: Enlightenment developer list > <enlightenment-devel@lists.sourceforge.net> > Message-ID: > <CAA65Se_3bZ1yqw6bBWKco+gmTs+515XOGaLCHbZqx= > kjnbr...@mail.gmail.com> > Content-Type: text/plain; charset=UTF-8 > > 2015-01-05 9:38 GMT+01:00 Sebastian Dransfeld <s...@tango.flipp.net>: > > > On 01/04/2015 03:56 PM, Dave Andreoli wrote: > > > - python setup.py install > > > + (sido) python setup.py install > > > > > > > > > > sido? > > > > Don't you know sido? it's the sudo of 2015 :D > > fixed, thanks for reporting > > > > > > > S. > > > > > > > ------------------------------------------------------------------------------ > > Dive into the World of Parallel Programming! The Go Parallel Website, > > sponsored by Intel and developed in partnership with Slashdot Media, is > > your > > hub for all things parallel software development, from weekly thought > > leadership blogs to news, videos, case studies, tutorials and more. Take > a > > look and join the conversation now. http://goparallel.sourceforge.net > > _______________________________________________ > > enlightenment-devel mailing list > > enlightenment-devel@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > > > ------------------------------ > > Message: 4 > Date: Mon, 5 Jan 2015 11:50:58 +0100 > From: Cedric BAIL <cedric.b...@free.fr> > Subject: Re: [E-devel] [EGIT] [core/elementary] master 01/01: genlist: > exceptional handling. > To: Enlightenment developer list > <enlightenment-devel@lists.sourceforge.net> > Message-ID: > <CAGv1asX0t826AM-Kvi4h5uW= > rt2c5tewaxnrbfzqwxsh6kd...@mail.gmail.com> > Content-Type: text/plain; charset=UTF-8 > > On Mon, Jan 5, 2015 at 3:53 AM, ChunEon Park <her...@naver.com> wrote: > > i don't think it's no much benefit for us in maintainance. > > it's just exceptional case. hide is enough. > > we don't need to get bother too much for it. > > Genlist needs love and care. Pushing under quality patch is not gona > help. I have done a closer review of this patch and discovered that in > fact this piece of code already leak stringshare and is under > performing. It needs a serious care. By fixing things correctly, you > will have increased performance and lower peak memory usage. The list > should actually be stored relative to the class. Looking at the code, > I understand why you are reluctant to do it, as it require some major > work to do it, but along the way you are going to start improving > genlist in necessary way. > -- > Cedric BAIL > > > > ------------------------------ > > Message: 5 > Date: Mon, 05 Jan 2015 11:25:13 +0000 > From: Tom Hacohen <tom.haco...@samsung.com> > Subject: Re: [E-devel] [EGIT] [core/efl] master 01/01: curl usage > better accounts for timeouts > To: Enlightenment developer list > <enlightenment-devel@lists.sourceforge.net> > Message-ID: <54aa7499.3060...@samsung.com> > Content-Type: text/plain; charset=utf-8; format=flowed > > Mike, > > This commit doesn't follow our commit guidelines. > > -- > Tom. > > On 03/01/15 04:30, Mike Blumenkrantz wrote: > > discomfitor pushed a commit to branch master. > > > > > http://git.enlightenment.org/core/efl.git/commit/?id=f55c756c17b9567a643c7f779c65433f3c6eb9ca > > > > commit f55c756c17b9567a643c7f779c65433f3c6eb9ca > > Author: zmike <michael.blumenkra...@gmail.com> > > Date: Fri Jan 2 23:21:34 2015 -0500 > > > > curl usage better accounts for timeouts > > > > curl is dumb. it needs to poll its own fd for data, it gets > confused with its own timeouts, and sometimes it forgets that it's supposed > to be doing anything. > > > > this fixes: > > > > * connection timeout processing > > * connection data processing order > > > > also curl_multi_timeout calls are now done from a single function > to handle all of this stupidness in one place > > > > maybe backport after more testing... > > --- > > src/lib/ecore_con/ecore_con_url.c | 49 > ++++++++++++++++++++++++++++++++++----- > > 1 file changed, 43 insertions(+), 6 deletions(-) > > > > diff --git a/src/lib/ecore_con/ecore_con_url.c > b/src/lib/ecore_con/ecore_con_url.c > > index 2c5dc0c..11a2abc 100644 > > --- a/src/lib/ecore_con/ecore_con_url.c > > +++ b/src/lib/ecore_con/ecore_con_url.c > > @@ -276,6 +276,7 @@ static Eina_List *_fd_hd_list = NULL; > > static int _init_count = 0; > > static Ecore_Timer *_curl_timer = NULL; > > static Eina_Bool pipelining = EINA_FALSE; > > +static long last_ms = -1; > > > > static Ecore_Con_Curl *_c = NULL; > > static Eina_Bool _c_fail = EINA_FALSE; > > @@ -343,7 +344,8 @@ _c_init(void) > > goto error; > > } > > _c->curl_multi_timeout(_c->_curlm, &ms); > > - if ((ms >= CURL_MIN_TIMEOUT) || (ms <= 0)) ms = CURL_MIN_TIMEOUT; > > + if ((ms >= CURL_MIN_TIMEOUT) || (ms < 0)) ms = CURL_MIN_TIMEOUT; > > + last_ms = ms; > > _curl_timer = ecore_timer_add((double)ms / 1000.0, > > _ecore_con_url_timer, NULL); > > ecore_timer_freeze(_curl_timer); > > @@ -1519,20 +1521,53 @@ _ecore_con_url_curl_clear(void) > > ecore_main_fd_handler_del(fdh); > > EINA_LIST_FREE(_url_con_list, url_con) > > _ecore_con_url_multi_remove(url_con); > > + last_ms = -1; > > +} > > + > > +static Eina_Bool > > +_ecore_con_url_do_multi_timeout(long *retms) > > +{ > > + long ms = 0; > > + int ret; > > + > > + while (!ms) > > + { > > + ret = _c->curl_multi_timeout(_c->_curlm, &ms); > > + *retms = ms; > > + if ((last_ms > 0) && (ms < 0)) > > + ERR("curl_multi_perform() timeout"); > > + else if (ms <= 0) > > + { > > + last_ms = ms; > > + _ecore_con_url_timer(NULL); > > + DBG("multiperform is still running: timeout: %ld", ms); > > + return EINA_TRUE; > > + } > > + else if ((ret <= 0) && (ms > 0)) break; > > + else > > + ERR("curl_multi_perform() failed: %s", > _c->curl_multi_strerror(ret)); > > + last_ms = ms; > > + _ecore_con_url_curl_clear(); > > + ecore_timer_freeze(_curl_timer); > > + return EINA_FALSE; > > + } > > + return EINA_TRUE; > > } > > > > static Eina_Bool > > _ecore_con_url_fd_handler(void *data EINA_UNUSED, Ecore_Fd_Handler > *fd_handler EINA_UNUSED) > > { > > Ecore_Fd_Handler *fdh; > > - long ms; > > + long ms = 0; > > > > if (!_c) return ECORE_CALLBACK_CANCEL; > > EINA_LIST_FREE(_fd_hd_list, fdh) > > ecore_main_fd_handler_del(fdh); > > - _c->curl_multi_timeout(_c->_curlm, &ms); > > + if (!_ecore_con_url_do_multi_timeout(&ms)) return EINA_FALSE; > > + last_ms = ms; > > if ((ms >= CURL_MIN_TIMEOUT) || (ms <= 0)) ms = CURL_MIN_TIMEOUT; > > ecore_timer_interval_set(_curl_timer, (double)ms / 1000.0); > > + ecore_timer_reset(_curl_timer); > > _ecore_con_url_timer(NULL); > > return ECORE_CALLBACK_CANCEL; > > } > > @@ -1594,6 +1629,7 @@ _ecore_con_url_timer(void *data EINA_UNUSED) > > if (ret == CURLM_CALL_MULTI_PERFORM) > > { > > DBG("curl_multi_perform() again immediately"); > > + ecore_timer_interval_set(_curl_timer, 0.000001); > > return ECORE_CALLBACK_RENEW; > > } > > else if (ret != CURLM_OK) > > @@ -1604,13 +1640,14 @@ _ecore_con_url_timer(void *data EINA_UNUSED) > > } > > if (still_running) > > { > > - long ms; > > + long ms = 0; > > > > _ecore_con_url_fdset(); > > - _c->curl_multi_timeout(_c->_curlm, &ms); > > + if (!_ecore_con_url_do_multi_timeout(&ms)) return EINA_FALSE; > > + last_ms = ms; > > DBG("multiperform is still running: %d, timeout: %ld", > > still_running, ms); > > - if ((ms >= CURL_MIN_TIMEOUT) || (ms <= 0)) ms = > CURL_MIN_TIMEOUT; > > + if ((ms >= CURL_MIN_TIMEOUT) || (ms < 0)) ms = CURL_MIN_TIMEOUT; > > ecore_timer_interval_set(_curl_timer, (double)ms / 1000.0); > > } > > else > > > > > > > > ------------------------------ > > Message: 6 > Date: Mon, 5 Jan 2015 14:49:16 +0100 > From: Thanatermesis <thanatermesis.e...@gmail.com> > Subject: Re: [E-devel] PATCH: duplicated XDG_DATA_DIRS fix > To: Enlightenment developer list > <enlightenment-devel@lists.sourceforge.net> > Message-ID: > <CAPgzchP-h+-FRmtrRYChdWwO4JdNb0cZ87PnPzQ5KJAL5uah= > a...@mail.gmail.com> > Content-Type: text/plain; charset=UTF-8 > > As: http://www.enlightenment.org/ss/e-54aa94e4da9742.14698270.jpg > > Where the real bug happens is where my patch is, but after that there's > hardcoded /usr/share and /usr/local/share in the case that we dont have > XDG_DATA_DIRS set (which the default dirs are these ones, so having them > hardcoded looks correct here for me) > > > 2015-01-03 17:04 GMT+01:00 Sebastian Dransfeld <s...@tango.flipp.net>: > > > > > > Den Jan 2, 2015 kl. 22:18 skrev Davide Andreoli < > d...@gurumeditation.it > > >: > > > > > > 2014-12-31 19:31 GMT+01:00 Thanatermesis <thanatermesis.e...@gmail.com > >: > > > > > >> Right, there's also the case of /usr/local which i have not considered > > >> (only /usr and /usr/local), so this updated patch fixes it: > > >> > > >> http://pastebin.com/1Nn4rETD > > >> > > >> by other side these trailing slashes seems like to be made wrongly by > > your > > >> OS creating the XDG_DATA_DIRS variable, they should not include > trailing > > >> slashes, maybe we should remove the possible trailing slashes from the > > >> variable get > > > I still think that harcoding a path is always the wrong solution, > looking > > > at that > > > file seems to me that the real problem is some line below, in the else: > > > snprintf(buf, sizeof(buf), "%s:/usr/local/share:/usr/share", newpath); > > > > > > Why we are blindly adding /usr/share and /usr/loca/share here ?? > > > plus newpath, that can include the same ?? > > > > > > this seems the real source of the problem, but I'm not in touch with > > > that code, I cannot really understand what was the intention here :/ > > > > > > > Lazyness. It is there to make sure we have all needed paths, with no > check > > for duplicates. The right solution is to parse the env var to a list, > then > > add the missing dirs. > > > > Sebastian > > > > > ------------------------------------------------------------------------------ > > Dive into the World of Parallel Programming! The Go Parallel Website, > > sponsored by Intel and developed in partnership with Slashdot Media, is > > your > > hub for all things parallel software development, from weekly thought > > leadership blogs to news, videos, case studies, tutorials and more. Take > a > > look and join the conversation now. http://goparallel.sourceforge.net > > _______________________________________________ > > enlightenment-devel mailing list > > enlightenment-devel@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > > > ------------------------------ > > > ------------------------------------------------------------------------------ > Dive into the World of Parallel Programming! The Go Parallel Website, > sponsored by Intel and developed in partnership with Slashdot Media, is > your > hub for all things parallel software development, from weekly thought > leadership blogs to news, videos, case studies, tutorials and more. Take a > look and join the conversation now. http://goparallel.sourceforge.net > > ------------------------------ > > _______________________________________________ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > End of enlightenment-devel Digest, Vol 105, Issue 4 > *************************************************** > ------------------------------------------------------------------------------ Dive into the World of Parallel Programming! The Go Parallel Website, sponsored by Intel and developed in partnership with Slashdot Media, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials and more. Take a look and join the conversation now. http://goparallel.sourceforge.net _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
