http://permalink.gmane.org/gmane.network.jabber.ekg2.devel hourly 1 1901-01-01T00:00+00:00 http://gmane.org/img/gmane-25t.png http://gmane.org http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1573 <pre> Tak, były baszyzmy, a #!/bin/bash nie pomaga jeśli skrypt jest czytany przez "." :-) Już zmieniłem cronjob żeby deklarował odpowiedni shell. </pre> Marcin Owsiany 2012-04-23T10:39:26 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1572 <pre>Dnia 2012-04-16, pon o godzinie 08:30 +0100, Marcin Owsiany pisze: Nie, nie, ktoś zrobił dist-upgrade, które domyślnym shellem zrobiło dasha. Nie mam dostępu do tego pliku z poziomu zwykłego użytkownika, więc nie wiem, czy zaplątał się tam jakiś bashizm czy może dash za bardzo się różni od ksh. Jeśli to drugie, dajcie znać, przywrócę starą konfigurację. Pozdr, Wojtek </pre> Wojtek Kaniewski 2012-04-16T18:15:05 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1571 <pre> A nie, po prostu domyślna powłoka zmieniła się na dash, który nie łyka bashowej składni definicji funkcji. Dodałem SHELL=/bin/bash do crontaba. </pre> Marcin Owsiany 2012-04-16T18:21:39 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1570 <pre>Wygląda na to, że ktoś wczoraj wieczorem na maszynie na której jest strona WWW wprowadził błąd do pliku common.sh, a nie wysłał zmiany do github.com/porridge/ekg2-website.git... </pre> Marcin Owsiany 2012-04-16T07:30:40 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1569 <pre>Odpisuję teraz, bo byłem na urlopie. On Mon, Feb 13, 2012 at 01:41:16PM +0100, Jakub Zawadzki wrote: Fakt. Hm, nie sądziłem że wyścig jest związany z tą statyczną zmienną, prawdę mówiąc nie zauważyłem jej aż do Twojego maila. Po prostu wydawało mi się, że mutex w tym miejscu rozwiąże wszystkie problemy związane ze współbieżnym wywołaniem ekg_debug_handler. Ale nie pomyślałem, że gdy wątek resolvera wywołuje print_window_w poprzez ekg_debug_handler, to główny wątek ekg2 może wykonywać print_window_w wywołane gdzieś indziej - w tym przypadku rzecz jasna mutex nie pomaga ani trochę. Right, however just the handler needs to thread-safe, which may be easier than making the whole program thread-safe. I can see two possibilities: 1) (your suggestion) change libgadu to (optionally) not call the debug handler in threaded resolver. (+) straightforward solution (-) need to convince libgadu upstream to implement such option (-) users need a new libgadu to have the fix (-</pre> Marcin Owsiany 2012-02-18T16:22:07 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1568 <pre> "display_mode" brzmi dość ogólnie i mało konkretnie. Proponowałbym raczej coś w stylu "indent_lines" albo "display_indent", z wartościami odpowiednio: 0 = multiple_timestamp 1 = indent_prompted 2 = indent_all </pre> Marcin Owsiany 2012-02-18T15:06:28 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1567 <pre> Wiesz, ten kod mial pozwalac aby mozna bylo wywolywac wiele razy debug() bez \n i ekg2 sobie to mialo łączyć w jedną linijke. Jeśli debug handler może być wywoływany z wielu wątków, to przestaje mieć to sens i lepiej ten kod wywalić /a nie dokładać muteksy/. Co nie powinno być takie trudne, bo w większości wypadków jedno wywoływanie debug - jedna linijka. A ta łatka którą proponujesz naprawia jeden z wielu możliwych błędów. Obawiam się, że potem ich trzeba będzie ich szukać np. w handlerach UI_WINDOW_PRINT. Dlatego lepiej: To call debug handler only from main thread, and IMHO it should be fixed in libgadu cause it currently assumes that program is thread-safe. Which for ekg2 is not true. Takie moje 3 grosze (z zaznaczenie, że nie deweloperuje). </pre> Jakub Zawadzki 2012-02-13T12:41:16 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1566 <pre>[...] [...] Mój algorytm: 1) napisz; 2) wyślij; 3) sprawdź; okazał się zawodny. Dzięki za zwrócenie uwagi. Pozdrawiam, wiechu </pre> Wiesław Ochmiński 2012-02-02T12:15:20 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1565 <pre>On Thu, 2 Feb 2012 12:48:34 +0100 Wiesław Ochmiński &lt;wiechu&lt; at &gt;wiechu.com&gt; wrote: Że się tak wyrażę niezrozumiale i skomplikowanie: &lt;stdin&gt;:13:8: error: duplicate case value '1' case mode2: ^ &lt;stdin&gt;:11:8: note: previous case defined here case mode1: ^ </pre> Michał Górny 2012-02-02T12:02:31 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1564 <pre>Kłaniam! Przerobiłem sposób wyświetlania w ncurses i zrobiłem to w osobnej gałęzi, bo to kontrrewolucja, a nie drobne zmiany: - wywaliłem 'struct screen_line' i wszystko co z tym było związane - w związku z tym zawartość okien nie jest przeliczana na zapas, a tylko przy wyświetlaniu danego okna Tak więc przy pokazaniu lub ukryciu okna kontaktów nie musimy przeliczać 10000 linii okna debug, do którego i tak nie zaglądaliśmy i nie mamy zamiaru robić tego w przewidywalnej przyszłości. Wprowadziłem też zmienną 'ncurses:display_mode', która odpowiada za styl wyświetlania zawartości okien. ncurses:display_mode = 0 (classic) -- wszystko tak jak dotychczas. ncurses:display_mode = 1 (mode1) -- (jak to nazwać?) 12:43:50 Długie linie wyświetlane są w ten sposób, że w timestamp jest wyświetlany tylko w pierwszej linii, a w kolejnych już nie. Wyjątek stanowią linie, które mają 'prompt', jak ta następna. 12:45:20 ::: Długa linia z promptem też nie powtarza time</pre> Wiesław Ochmiński 2012-02-02T11:48:34 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1563 <pre> Because it does not solve the general case. Imagine libgadu resolver thread calling ekg_debug_handler at the same time as another plugin calling the same debug handler. I don't see how this could be solved in libgadu? I don't remember the exact rules, but isn't it OK to fork from the main thread? </pre> Marcin Owsiany 2012-01-28T10:05:19 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1562 <pre>Hi, On Fri, Jan 27, 2012 at 09:32:42PM +0000, Marcin Owsiany wrote: Why not put it in gg plugin, i.e: libgadu_debug_handler()? or fixed in libgadu? Actually I believe we have bigger problem when libgadu is configured with pthread, cause AFAIR you SHOULDN'T use both fork() and threads... Regards, Kuba. </pre> Jakub Zawadzki 2012-01-27T21:48:25 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1561 <pre>Please review. </pre> Marcin Owsiany 2012-01-27T21:32:42 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1560 <pre> Naprawione. Administracja przeprasza za zaistniałą sytuację. l. </pre> Leszek Krupiński 2012-01-27T11:41:20 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1559 <pre>:-/ Internal error An error occurred on the page you were trying to access. If you continue to experience problems please contact your Redmine administrator for assistance. If you are the Redmine administrator, check your log files for details about the error. Back </pre> Marcin Owsiany 2012-01-27T10:45:08 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1558 <pre> The key word there was "unreliably". Obviously it won't save anything if it deadlocks :-/ Perhaps periodic or message-triggered commits would fulfil your need better, and additionally cover for things like OOM killer and people tripping on power cables? </pre> Marcin Owsiany 2012-01-01T18:11:43 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1557 <pre>On Sun, 1 Jan 2012 10:54:58 +0000 porridge&lt; at &gt;debian.org wrote: Depends to whom. I personally care more about having my logs being saved (to not lose messages which I haven't read yet), and that involves committing to sqlite. </pre> Michał Górny 2012-01-01T17:59:28 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1556 <pre>From: Marcin Owsiany &lt;marcin&lt; at &gt;owsiany.pl&gt; --- ekg/stuff.c | 4 ++++ ekg/stuff.h | 8 ++++++++ 2 files changed, 12 insertions(+), 0 deletions(-) diff --git a/ekg/stuff.c b/ekg/stuff.c index 4851680..d5522aa 100644 --- a/ekg/stuff.c +++ b/ekg/stuff.c &lt; at &gt;&lt; at &gt; -1817,6 +1817,10 &lt; at &gt;&lt; at &gt; int isalpha_pl(unsigned char c) return 0; } +inline void ignore_result_helper(int __attribute__((unused)) dummy, ...) +{ +} + /* * strcasestr() * diff --git a/ekg/stuff.h b/ekg/stuff.h index ea257d5..5520abb 100644 --- a/ekg/stuff.h +++ b/ekg/stuff.h &lt; at &gt;&lt; at &gt; -289,6 +289,14 &lt; at &gt;&lt; at &gt; int isalpha_pl(unsigned char c); #define xtolower(c) tolower((int) (unsigned char) c) #define xtoupper(c) toupper((int) (unsigned char) c) +inline void ignore_result_helper(int __attribute__((unused)) dummy, ...); + +#ifdef __GNUC__ +# define IGNORE_RESULT(X) ignore_result_helper(0, (X)) +#else +# define IGNORE_RESULT(X) (X) +#endif + const char *ekg_status_label(const int status, const char *descr, const char *prefix); void ekg_update_status(session_</pre> porridge< at >debian.org 2012-01-01T10:55:00 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1555 <pre>From: Marcin Owsiany &lt;marcin&lt; at &gt;owsiany.pl&gt; Currently ekg2 likes to deadlock when it segfaults in a malloc/free because the debug functions try to execute malloc/free again, and they are not necessarily reentrant. Remove a lot of the fancy things that ekg2 does on such signals and keep the bare minimum with the goal of being able to produce usable corefiles again and restoring terminal state to sanity. Also, no longer change the current directory to the config dir - users generally expect programs to dump core in the directory they were started in. --- docs/ekg2book-en/develbook.xml | 2 +- docs/ekg2book/develbook.xml | 2 +- ekg/abort.h | 4 +- ekg/configfile.c | 40 ------------ ekg/configfile.h | 1 - ekg/ekg.c | 129 +++++++++++++-------------------------- ekg/userlist.c | 51 ---------------- ekg/userlist.h | 1 - ekg2.h | 1 + plugins/ncurses/mouse.c </pre> porridge< at >debian.org 2012-01-01T10:55:01 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1554 <pre>From: Marcin Owsiany &lt;marcin&lt; at &gt;owsiany.pl&gt; Includes unit tests. --- Makefile.am | 5 ++- ekg/abort.c | 72 ++++++++++++++++++++++++++++++++++++++ ekg/abort.h | 77 +++++++++++++++++++++++++++++++++++++++++ ekg/plugins.c | 3 ++ plugins/check/check.c | 2 + plugins/check/static-aborts.c | 76 ++++++++++++++++++++++++++++++++++++++++ 6 files changed, 234 insertions(+), 1 deletions(-) create mode 100644 ekg/abort.c create mode 100644 ekg/abort.h create mode 100644 plugins/check/static-aborts.c diff --git a/Makefile.am b/Makefile.am index bcde301..ba9e0b5 100644 --- a/Makefile.am +++ b/Makefile.am &lt; at &gt;&lt; at &gt; -13,6 +13,7 &lt; at &gt;&lt; at &gt; bin_PROGRAMS = ekg2 noinst_HEADERS = \ ekg2.h \ ekg2-config.h \ +ekg/abort.h \ ekg/bindings.h \ ekg/commands.h \ ekg/completion.h \ &lt; at &gt;&lt; at &gt; -48,6 +49,7 &lt; at &gt;&lt; at &gt; noinst_HEADERS = \ ekg2_SOURCES = \ gettext.h \ $(noinst_HEADERS) \ +ekg/abort.c \ ekg/bindings.c \ ekg/commands.c \ ekg/completion.c</pre> porridge< at >debian.org 2012-01-01T10:54:59 http://permalink.gmane.org/gmane.network.jabber.ekg2.devel/1553 <pre>The following patch series fixes bug#124 http://bugs.ekg2.org/issues/124 It's a little brutal solution, but I think reliably getting a usable corefile is more useful then unreliably getting your settings saved. Please review. [PATCH 1/3] Infrastructure for registering and running abort handlers. [PATCH 2/3] Helper macro for ignoring results without warnings. [PATCH 3/3] Make the fatal signal handlers async-signal-safe again. </pre> porridge< at >debian.org 2012-01-01T10:54:58 Search the mailing list at Gmane query http://search.gmane.org/?group=$group=gmane.network.jabber.ekg2.devel