IRC log for #maemo-ssu on 20121106

02:37.30*** join/#maemo-ssu guly (~why@shivaya.guly.org)
03:40.42*** join/#maemo-ssu amiconn_ (amiconn@rockbox/developer/amiconn)
04:04.22*** join/#maemo-ssu DocScrutinizer05 (~HaleBopp@openmoko/engineers/joerg)
04:32.55*** join/#maemo-ssu Jade (~jade@unaffiliated/jade)
04:35.46*** join/#maemo-ssu Jade (~jade@unaffiliated/jade)
04:40.13*** join/#maemo-ssu Jade (~jade@unaffiliated/jade)
04:53.15*** join/#maemo-ssu Jade (~jade@modemcable021.180-203-24.mc.videotron.ca)
04:53.15*** join/#maemo-ssu Jade (~jade@unaffiliated/jade)
06:41.48*** join/#maemo-ssu povbot (~supybot@office.pov.lt)
06:41.49*** mode/#maemo-ssu [+v povbot] by ChanServ
07:01.42*** join/#maemo-ssu ruskie (ruskie@sourcemage/mage/ruskie)
07:35.50*** join/#maemo-ssu luf (luf@nat/ibm/x-yfwekevzgcdlknbf)
07:59.51*** join/#maemo-ssu dhbiker (~dhbiker@95.87.145.172)
09:02.45*** join/#maemo-ssu X-Fade (~xfade@d5152FFD8.static.telenet.be)
10:05.29lufDocScrutinizer51: no damage for N900 or carkit (unless angry user don't destroy it) :D So no regression :D
11:01.39*** join/#maemo-ssu merlin1991 (~merlin@Maemo/community/cssu/merlin1991)
11:07.36*** join/#maemo-ssu arcean (~arcean@aaes75.neoplus.adsl.tpnet.pl)
11:17.43*** join/#maemo-ssu M4rtinK (~M4rtinK@mail.melf.eu)
11:19.02*** join/#maemo-ssu ivgalvez (~ivgalvez@89.140.113.138.static.user.ono.com)
12:11.12*** join/#maemo-ssu Pali (~pali@unaffiliated/pali)
12:15.07*** join/#maemo-ssu dhbiker (~dhbiker@95.87.145.172)
12:46.06*** join/#maemo-ssu lizardo (~lizardo@79.Red-88-2-49.staticIP.rima-tde.net)
12:46.38DocScrutinizer05luf: OK :-)
13:54.11merlin1991luf: do we know if the patch is good by now?
13:55.30*** join/#maemo-ssu Woody14619a (~Woody@2620:4:4000:1:c95a:a95d:2340:5acd)
13:55.31*** join/#maemo-ssu Woody14619a (~Woody@Maemo/Community/council/Woody14619)
13:55.36lufmerlin1991: The patch seems to be ok now.
13:55.48lufWhen you want to release new -testing?
13:57.12*** join/#maemo-ssu ivgalvez (~ivgalvez@89.140.113.138.static.user.ono.com)
14:01.53*** join/#maemo-ssu dafox (~dafox@ip51cc571d.speed.planet.nl)
14:08.38*** join/#maemo-ssu arcean (~arcean@aaes75.neoplus.adsl.tpnet.pl)
14:20.03*** join/#maemo-ssu krayon (~fallen@pdpc/supporter/28for7/krayon)
14:30.46merlin1991as soon as I can :)
14:41.39lufOps :)
14:42.06lufI hope I'll push the git today or tomorrow with the obexd fix.
14:56.27chem|stmerlin1991: as soon as I am done building that stuff I will shoot you a list to checkback on my doings, I am not really sure it is all right what I do atm...
14:57.26merlin1991take care not to take the new testing stuff ;)
15:09.09chem|stmerlin1991: currently I am at stuff some month old apart from curl
15:09.21merlin1991:)
15:09.23chem|stqt-x11 takes ages to build
15:09.28merlin1991yep
15:09.35merlin1991openssl is almost as bad
15:09.37chem|stlove it!
15:09.55merlin1991though in the case of openssl it's a billion test that are run, whils qith qt it's the friggin html docs
15:10.10chem|st*sigh*
15:11.21merlin1991s/whils qith/whilst with/
15:11.56freemangordonmerlin1991: hmm, iirc it is webkit js that takes ages
15:12.23merlin1991I was up way too long today having fun with kvm, when I finally started to pester people in #virt the my typo to proper words ratio was beyond readable :D
15:12.44chem|st^^
15:15.06merlin1991freemangordon:  possibly webkit aswell, last time I built qt it was the harm qt with meegotouch stuff, back then it was the docs that took forver :D
15:15.17merlin1991*forever*
15:17.46merlin1991btw I have to say qemu-kvm from debian-testing together with virt-manager works quite well to virtualize win stuff (once one removed the shortcomings of the default windows template and installs the virtio drivers from readhat into the guest)
15:23.36chem|stmerlin1991: for the record it took ages for html stuff now it takes ages for webikit js...
15:24.00chem|stI think those ssl test are not even close as boring...
15:37.53*** join/#maemo-ssu dafox (~dafox@ip51cc571d.speed.planet.nl)
15:55.40*** join/#maemo-ssu arcean (~arcean@aaes75.neoplus.adsl.tpnet.pl)
16:14.21chem|stmerlin1991: modest? commit 17e9406 or latest?
16:15.26*** join/#maemo-ssu Jade (~jade@modemcable021.180-203-24.mc.videotron.ca)
16:15.26*** join/#maemo-ssu Jade (~jade@unaffiliated/jade)
16:16.23chem|stI guess the later is not in last testing is it?!
16:17.11chem|stgot it
16:17.35*** join/#maemo-ssu freemangordon (~freemango@130-204-50-168.2074221835.ddns.cablebg.net)
16:18.05merlin1991chem|st: you don't need modest at all
16:18.22merlin1991ah wait you to
16:18.24merlin1991*do*
16:18.33freemangordonyes, he wants it ;)
16:19.30freemangordonchem|st: aiui you should pull latest master in stable, merlin1991, agree?
16:20.34merlin1991freemangordon: yep sounds about right
16:20.54freemangordontinymail as well
16:21.42merlin1991tinymail is later in the alphabet ;)
16:21.51freemangordonaah, ok :D
16:21.56merlin1991btw chem|st don't take obexd
16:22.02merlin1991it's faulty
16:24.21chem|sthmm just build the one from testing
16:24.31chem|stmodest*
16:24.34chem|stnot obex
16:24.39chem|stI am reading the chan
16:25.37kerioi thought obexd was fixed-ish
16:25.43keriooh, not pushed yet
16:26.25chem|stwierd I get a all up to date with git merge master... I am doing something wrong here...
16:27.10chem|stlatest in my log is palis commit not the newer by fmg
16:30.36chem|stanyone?
16:30.58chem|stI merged it now with the commit number but is that the right way?
16:36.23chem|stwonderfull now I am back where I was and it does not built anymore &"$%@
16:38.39chem|staclocal: macro `gl_FUNC_ARGZ' required but not defined
16:38.40chem|stmake: *** [configure-stamp] Error 1
16:42.17chem|stfor god sake I just built libxml2 in the mean time
16:43.48chem|stso that is broken for me now...
16:43.56chem|stany ideas?
16:44.35chem|stI guess something pulled in another version of autoconf or something when I installed python dev stuff for xml2
16:44.36*** join/#maemo-ssu NIN101 (~NIN@p5DD2890A.dip0.t-ipconnect.de)
17:07.21*** join/#maemo-ssu Milhouse (~irc_milho@Maemo/community/contributor/Milhouse)
17:12.26*** join/#maemo-ssu arcean (~arcean@aaes75.neoplus.adsl.tpnet.pl)
17:25.49merlin1991chem|st: a merge is local only
17:26.02merlin1991you need to git fetch changes from the repos first to get everything up to date :)
17:27.14Palimerlin1991, I have new version of battery status menu plugin
17:27.23Palicompatible with BME and with kernel drivers
17:27.30Palibased on MAG version
17:27.37merlin1991psuh
17:27.40merlin1991*push*
17:27.43Paliwhat do you think can it go to CSSU?
17:28.00PaliI will push it into my git branch
17:28.26merlin1991okay, well do we know if the default status menu plugin does anything besides displaying?
17:28.54Paliit play souns & show hildon banners
17:28.58Palibattery is empty
17:29.02Palibattery is full
17:29.07Palicharging
17:29.54PaliI looked into nokia status menu battery binary, exported function calls and there is nothing more I think
17:30.15merlin1991does yours do all the funky stuff aswell?
17:30.19merlin1991(sound and banners)
17:30.27Palicode is here: https://gitorious.org/~pali/community-ssu/pali-status-area-applet-battery
17:30.29Paliyes
17:30.51Palithere is only one small problem: missing space between icon and text
17:31.00Paliso layout is not same
17:31.27freemangordonPali, merlin1991: Seems I am stupid today, look at that
17:31.30freemangordonalarms_events[i++] = alarms_events[i]
17:31.53freemangordonwill that do what I think it will, i.e. alarms_events[i] = alarms_events[i+1]
17:31.59merlin1991am i right saying thats arr[i] = arr[i]; i++; ?
17:32.03Palimerlin1991, read commit message: https://gitorious.org/~pali/community-ssu/pali-status-area-applet-battery/commit/1393753876927073058b5d32c917c0e37a8dc3fb
17:32.17freemangordonmerlin1991: hmm, no
17:32.35freemangordonthe question is if it is ^^^ or alarms_events[i+1] = alarms_events[i+1]
17:32.48merlin1991hm isn't the right side evaluated first
17:32.57PaliI think that specification did not specify order when i++ would be increased
17:32.57freemangordonafaik no
17:33.20freemangordonI think it specifies, that is why there is a difference between i++ and ++i
17:33.45merlin1991no ++i is increment before evaluation
17:33.51freemangordonexactly
17:34.03merlin1991but I'm more wondering if it evaluets left or right from = first
17:34.16freemangordondeffinitely it is left first
17:34.25kerioit's definetely unspecified
17:34.26freemangordon(afaik)
17:34.32Palibut I think it is not specified if i++ will be increased after (alarms_events[i++]) or after (alarms_events[i++] = alarms_events[i])
17:34.35merlin1991hm then you should have arr[i] = arr[i+1]
17:34.38freemangordonkerio: since when you know C?
17:34.46kerioyou're changing and accessing the same variable without a sequence point inbetween
17:35.07freemangordonmerlin1991: yeah, I was trying to do it in the unreadable way :D
17:35.18freemangordonkerio: defien sequence point
17:35.39merlin1991tbh don't count on me being right I could say totall bs atm :D
17:35.49freemangordonwell, will do it in the old-fashined and readable way :D:D:D
17:36.06keriofreemangordon: http://c-faq.com/expr/seqpoints.html
17:36.17keriofrom the comp.lang.c FAQ
17:36.59kerioalso, since when *you* know C?
17:37.05freemangordonkerio: never heard of that term, but if we use it, there IS seq point, as you have to index the array
17:37.21freemangordonkerio: for the last 20 or so years
17:37.37freemangordonany other questions?
17:37.54freemangordonhmm, make them 25
17:37.55kerioand you've never read anything about undefined things in the C standard?
17:38.02freemangordonI forgot how old I am :D:D:D
17:38.09freemangordonnever
17:38.24freemangordon:P
17:39.24merlin1991dafuq how old are you freemangordon?
17:39.45freemangordon37 :(
17:40.27freemangordonwell, not 25, 21-22 then
17:40.31ShadowJKUndefined things are things that one compiler might do something consistent for, over and over again, but another compiler might do something else consistently, and a third compiler might do something random each time :P
17:40.45ShadowJK(and there was that one compiler that wiped your harddrive)
17:41.40merlin1991anybody in here who isn't old in comparison to me?
17:42.00kerioif 91 is your birthyear, i'm younger than you
17:42.11merlin1991yay I'm not the youngest :)
17:42.19merlin1991we kinda have an upside down structure here, the (almost)youngest dude sits on top
17:42.26freemangordonShadowJK: I know what you mean, but if we take C before the modern era with all that extensions and stuff, there was no such thing like "undefined" behaviour in language itself
17:42.36freemangordonjust undefined compiler behaviour
17:42.59kerioANSI C specifies a lot of things as undefined
17:43.03*** join/#maemo-ssu arcean (~arcean@aaes75.neoplus.adsl.tpnet.pl)
17:43.11kerioliterally undefined, as in "this thing is undefined"
17:43.31freemangordonother term for "compiler specific implementation"
17:43.46kerioyes, the correct term
17:44.36freemangordonthough last time I check ++,--, etc was pretty defined in terms of evaluation. I am just not dure if gcc is aware of that :D
17:44.41freemangordon*checked
17:44.56freemangordonthe fuck. *sure
17:45.15freemangordonanyway, I am back to stupid alarm dialog
17:45.28kerioexcept that *they are not*, if you're accessing the value of the variable you incremented somewhere else in the same expression
17:45.49ShadowJKWell your example is good example of undefined behaviour? Compiler is free to evaluate it in whatever order it wants?
17:45.53keriofor instance, i = i++
17:46.19freemangordon^^ makes no sense, a better example is *p++ = *p
17:46.23kerioShadowJK: the compiler is free to show goatse on the screen and then rm -rf
17:46.23freemangordonand it IS defined
17:47.45ShadowJKhttp://stackoverflow.com/questions/11657100/is-this-undefined-behavior-or-implementation-defined :D
17:49.43freemangordonwe were talking about C, case closed AIUI :D
17:50.10ShadowJKBut he also quoted C :)
17:50.39freemangordonyeah, but my englis is not good enough to understand that cryptic sentence :D
17:50.43freemangordon*english
17:51.05freemangordonand still, this is C99 which is quoted
17:58.59keriofreemangordon: hold on, *p++ = *p is just *p++, isn't it
17:59.04ShadowJKin any case, I would expect your original expression to be equivalent of: alarms_events[i] = alarms_events[i]; i = i + 1;  ?
18:00.28freemangordonkerio: i think so
18:00.51freemangordonShadowJK: why? we need first [] evaluated
18:01.16freemangordoni.e. as [i], ++ applied and then second [] evaluated
18:01.43ShadowJKI'd expect the ++ to be executed after the ;
18:01.52freemangordonwhy?
18:02.06kerioShadowJK: it's just guaranteed to have been executed once you reach ;
18:02.23freemangordonhow is that different to *p++
18:02.38ShadowJKBut now that I google it, I get the impression that it's undefined and you might get either result, depending on the compiler
18:02.44ShadowJKor no result at all
18:02.56kerioand there's no sequence point in [] or = anyway, so it's probably a moot point
18:05.00ShadowJKwith -Wall, gcc actually gives a warning both with -std=c89 and -std=c99 about your example too
18:05.20ShadowJKAlso with gnu89 and gnu99
18:05.23freemangordonwell, it could be compiler depenedent. I'll try m$ C and gcc wneh I have some free time, to see what will be the output
18:05.43freemangordonwhat waring?
18:05.47freemangordon*warning
18:05.52ShadowJKoperation on ā€˜iā€™ may be undefined
18:05.53freemangordon"undefined..."
18:05.57freemangordonok
18:06.18ShadowJKWhich is different from implementation specific
18:07.04kerioyep, ##c confirms *p++ = *p is undefined
18:09.15freemangordonmsvc gives no warning and executes it as a[i] = a[i];i++;
18:09.40freemangordonok, I stand corrected
18:09.42freemangordon:)
18:10.10ShadowJKJust write what you mean instead
18:10.17freemangordonI already did it
18:10.38ShadowJK(instead of writing things that the compiler can do whatever it wants with)
18:10.42freemangordonbut was curious if it could be written in such a cryptic way
18:11.10freemangordonfor( ; i < alarm_events_cnt-1; i++)
18:11.11freemangordonalarms_events[i] = alarms_events[i+1];
18:12.17ShadowJKBtw, googling this I found one page which said that it's actually defined in java to behave not like msvc did
18:12.22ShadowJK(and gcc)
18:14.02ShadowJKlol, wikipedia says that earlier version of gcc, when encountering undefined behaviour, would try to launch nethack, rogue, or emacs
18:14.47kerioShadowJK: hm? i thought that was just for #pragma
18:15.00ShadowJKyeah
18:15.22*** join/#maemo-ssu MrPingu (~MrPingute@86.92.226.97)
18:20.57*** join/#maemo-ssu toxaris (~toxaris@s83-180-246-172.cust.tele2.se)
18:32.55chem|stmerlin1991: any ideas on the gl_FUNC_ARGZ?
18:33.04chem|stis that needed?
18:33.17chem|stwas that some workaround from last time
18:33.21merlin1991that was the fun with libtool
18:34.16chem|stI broke it again
18:37.12chem|stwell last time I checked I had it working without any fancy hacking...
18:37.37chem|stI installed some python crap in between the builds
18:48.50DocScrutinizer05((alarms_events[i++] = alarms_events[i])) *seriously* insane, you know that
18:58.47*** join/#maemo-ssu deepy\SIGSEGV (~deepy@wrongplanet/deepa)
18:59.22chem|stas long as there is this argz thing there is no modest with me no more
19:00.14DocScrutinizer05a coder honestly considering to ask like 100 lines of IRC questions to make sure he can use a coding shortcut abomination, really.... shoot him with owl shit
19:01.07DocScrutinizer05I gather freemangordon hasn't invented this obfuscated crap
19:01.40freemangordonit was exactly him :)
19:05.19chem|stpali is xterm considered stable (your last commit for portrait mode)
19:05.38chem|stor should I stick with the one around august?
19:10.00Palichem|st, patch which added new options in Font&Options dialog is stable
19:10.13Paliand patch which added portrait mode flag is stable too
19:10.42Palibut ask merlin1991 or freemangordon if we want portrait mode in xterm
19:11.58chem|sto.O
19:13.09chem|stI took "Increase version to 0.14.mh31+0cssu0" now what is the font option thing? you mean the one before with the application settings right
19:14.29Paliincrease version commit from Monday July 16 2012 is stable
19:14.44chem|stok *check* built
19:15.14chem|stPali: any idea what I messed up with my libtool stuff?
19:15.23keriohttp://www.ebay.co.uk/itm/A-piece-of-cardboard-shaped-a-bit-like-an-iPhone-5-USED-/160916451185 oh lols
19:15.37chem|stI know there was that issue with intltools but that one is another
19:16.05DocScrutinizer05freemangordon: a few basic best common practice rules about coding: Don't peephole optimize since you never can outsmart the compiler on that discipline. Never try to safe a few lines or even chars of code, when the result becomes only slightly less readable. Try to write what you planned to do, not what you think is the most terse nifty way to implement it, i.e. your code should be so clear that it's not possible to write a comment
19:16.07DocScrutinizer05making anything more obvious than it's in code. And final one for now: never use operators or constructs that you're not absolutely sure about what and how they do
19:16.48kerioDocScrutinizer05: today i discovered that a ^= b ^= a ^= b; is actually undefined :(
19:18.25Palikerio, why?
19:18.40*** join/#maemo-ssu arcean (~arcean@aaes75.neoplus.adsl.tpnet.pl)
19:18.40keriosame reason, it modifies the same variable twice between sequence points
19:19.23DocScrutinizer05int sourceindex; int destindex; sourceindex = 1; destindex = sourceindex - 1; while (sourceindex < upperlimit) {array[sourceindex++] = array[destindex++]}
19:20.08DocScrutinizer05oops, other way round
19:20.33DocScrutinizer05{array[destindex++] = array[sourceindex++]}
19:20.47keriofor (int i = 1; i < limit; i++) array[i - 1] = array[i];
19:20.59kerioalso known as *a fucking for loop*
19:21.23DocScrutinizer05also known as not as clear as it could be
19:22.20DocScrutinizer05and you do arithmetic in a for loop which is BS (though the compiler will optimize that too, but it's not what you would do when you do it 'by hand')
19:22.31kerioi do what now?
19:22.38keriooh, i see
19:23.00kerioright, i forgot that arithmetic is incredibly slow, and most definetely slower than memory access
19:23.16keriooh wait
19:23.56kerioany vaguely modern cpu can just calculate i - 1 and then do nothing for a lot of time while waiting for the memory to be accessed
19:24.09DocScrutinizer05see point #1 of rules above
19:24.34kerioyour thing is peephole optimization
19:24.51DocScrutinizer05in fact the compiler will probably compile my code to a pretty similar assembler
19:25.14DocScrutinizer05nope, my thing is doing it the way a sane mind would do it manually
19:25.37DocScrutinizer05esp when your distance is not 1 but maybe 4, or 55
19:25.46kerioyou do *two* calculations in a loop
19:25.54DocScrutinizer05nope
19:26.08DocScrutinizer05increment is not a calculation
19:26.57DocScrutinizer05replace 1 by 55 in your and my code, and then think again
19:27.44kerioanyway, the right solution is obviously memmove(array + 1, array, (elements - 1) * sizeof *array)
19:27.54kerio:D
19:27.57DocScrutinizer05whatever, your code is magnitudes more sane than freemangordon's
19:28.03tadzikdoesn't memmove fuck up when areas overlap?
19:28.11keriotadzik: nope, that's memcpy
19:28.21tadzikah, right
19:28.33DocScrutinizer05and ack for memmv
19:29.07DocScrutinizer05though array + 1 wount work usually
19:29.19keriowhy not?
19:29.35DocScrutinizer05since you're missing out on true size of array elements
19:29.59kerioarrays decay to pointers to their first element when passing them or doing arithmetics with them
19:30.01DocScrutinizer05&array[1]
19:30.12kerioDocScrutinizer05: &array[1] is **effectively the same** as array+1
19:30.23DocScrutinizer05probably not always
19:30.40keriohttp://c-faq.com/ptrs/explscale.html
19:30.42DocScrutinizer05though you *might* be right on that one
19:31.14DocScrutinizer05I just wonder what e.g. lint would say about it
19:31.17kerioK&R1 Sec. 5.3 p. 94, K&R2 Sec. 5.4 p. 103, ISO Sec. 6.3.6, H&S Sec. 7.6.2 p. 204
19:31.23keriohm, what's H&S?
19:31.25DocScrutinizer05ok ok
19:31.45keriooh, harbison steele
19:32.02DocScrutinizer05dafaq never heard of
19:37.57*** join/#maemo-ssu NIN102 (~NIN@p5DD28EE5.dip0.t-ipconnect.de)
19:41.11MrPinguregarding xterm
19:41.49MrPinguSomebody noticed xterm does auto pointermode, and then it's stuck in that mode?
19:41.50chem|st?
19:42.00DocScrutinizer05still I for one prefer 'adress of array, element one' over 'arry plus one, as defined by K&R1 Sec. 5.3 p. 94, K&R2 Sec. 5.4 p. 103, ISO Sec. 6.3.6, H&S Sec. 7.6.2 p. 204'
19:42.04MrPinguLet me shoot video, if needed
19:43.37DocScrutinizer05MrPingu: I seem to recall to have seen that when 'history' aka scrollback been < one screen worth of lines
19:44.01DocScrutinizer05<= actually
19:44.45DocScrutinizer05which also applies when you go $foo mode - the one I forgot the name and being used by stuff like less, mc, top etc
19:45.45DocScrutinizer05if you kill -9 mc from another terminal, your screen looks almost normal until you try to scroll back
19:46.02DocScrutinizer05same for less, top etc
19:46.12DocScrutinizer05is it called block mode?
19:46.16DocScrutinizer05raw mode?
19:46.18*** join/#maemo-ssu gregoa (~gregoa@chello212186052066.410.14.vie.surfer.at)
19:46.19DocScrutinizer05nfc
19:46.43kerioDocScrutinizer05: splint doesn't complain about array+1
19:47.25MrPinguWell most of the times, switching fullscreen makes the pointer icon appear again
19:47.34MrPinguThis time not, dafuq ;o
19:50.32MrPinguhmm not that
19:52.13kerioosso-xterm sucks
19:52.27MrPinguYeah :(
20:12.25Palinew status area applet battery is ready for testing!
20:12.27Palihttp://atrey.karlin.mff.cuni.cz/~pali/status-area-applet-battery_1.0-1_armel.deb
20:12.31Palihttps://gitorious.org/~pali/community-ssu/pali-status-area-applet-battery
20:13.00Palithis version is compatible with BME and also with new kernel drivers
20:13.37Paliusefull for usb host mode when BME is stopped and bq27x00_battery can be loaded for battery info
20:14.12Palimerlin1991, freemangordon, DocScrutinizer05, kerio ^^^^
20:14.31kerio:D
20:14.35kerioPali: what does it use?
20:14.42kerioi mean, is the switching automatic?
20:14.47Paliyes
20:14.49Palifully automatic
20:15.00keriocan it use bq27200 data without the modules loaded?
20:15.14Paliof course not
20:15.19kerio:(
20:15.54Palibut when you stop bme and load kernel drivers, you will see battery status
20:16.11Paliso you can use kernel driver for charging and see status in status bar
20:16.23kerioi'll see a *more accurate* battery status, won't i
20:16.57kerioi like it :3
20:17.00Paliplus it show percentage, current and design capacity and time until battery will discharge
20:17.20Pali(or until fully charge)
20:18.25kerioPali: now add a button for host mode! :D
20:18.48Palikerio, host mode button?
20:18.54Paliit is in usbmode package
20:19.07kerioyeah but that's a *new* button
20:19.38Palithere is no place for button in battery status plugin
20:19.40Pali:D
20:19.51kerioof course there is, *the whole thing*
20:19.52kerio:3
20:20.08kerioPali: how do i make the battery indicator show up in the status area without rebooting? :s
20:20.11kerioit's not hildon-home
20:20.27MrPingustatusmenu, i guess
20:20.51kerioMrPingu: ty
20:21.19Pali$ killall -9 hildon-status-menu; sleep 1; killall -9 hildon-status-menu; sleep 1; killall hildon-status-menu
20:21.23Palikerio ^^^
20:21.41kerioPali: hm, can the data from bme be trusted?
20:21.45MrPinguit's working :)
20:21.51kerioyeah, it works here too
20:21.56Palisome plugins sucks in my n900, so need to kill it twice with -9 and once normal
20:22.24MrPingubme is some kind of mystery :P
20:22.38Palikerio, I design capacity from bme is same as from rx51_battery driver (+/- small epsilon)
20:22.40kerioPali: what are the modules to load again?
20:22.49*** join/#maemo-ssu gregoa (~gregoa@chello212186052066.410.14.vie.surfer.at)
20:22.56Palirx51_battery for design capacity
20:23.12Pali(this can be loaded when bme is running too)
20:23.19kerioit's already loaded
20:23.28keriobut it's not as accurate as the bq27200 data, right?
20:23.28Paliand bq2415x_charger & bq27x00_battery when BME is stopped
20:24.00Palibq2415x_charger is needed only if you want to charge battery
20:24.08Palistatus menu plugin do not need it
20:24.14kerioconfirmed working :D
20:24.42Palistatus menu plugin needs bq27x00_battery (+ rx51_battery for design capacity) or BME
20:24.47Palikerio, ok :-)
20:25.02kerioPali: hold on, it doesn't take the capacity from bq27200? :(
20:25.06keriowhen available, i mean
20:25.35Palikerio, it take current mAh from bq2700 if possible
20:25.36MrPinguNever bothered with those bqxx battery things, but BME seems to report fine
20:26.12MrPinguPali, does it take the default graphics?
20:26.22kerioPali: my bq27200-0/charge_full is 1261638
20:26.28kerioapplet reports 1267
20:26.34kerioit doesn't seem right :s
20:26.38Palikerio, then it take design mAh from rx51_battery (if not possible then take last full mAh from bq27200) and calculate percentage
20:26.56Palikerio, it report design capacity if possible from rx51_battery
20:27.01kerio...why? :S
20:27.06Paliif not then it take last full
20:28.09Paliwhy not?
20:28.23keriobecause bq27200 is accurate, rx51_battery is not
20:28.43Palibq27200 do not know design capacity...
20:29.09Paliand rx51_battery reporting now design capacity
20:29.15keriodesign capacity doesn't mattery, what matters is the biggest charge it can hold
20:29.21PaliMrPingu, yes it have default graphics
20:29.54Palikerio, but there is not way how to get biggest charge
20:30.18kerio/sys/class/power_supply/bq27200-0/charge_full
20:30.40Palithis is capacity after *last* charge
20:30.58Palinot biggest
20:30.58kerioafter the last calibration, you mean
20:31.17keriowhich is what we should use, if it's available
20:31.20kerioDocScrutinizer05: confirm/deny?
20:32.20Palikerio, if you really want to use that value rmmod rx51_battery
20:32.31keriothat's what i'm doing :)
20:32.41Palistatus menu plugin will switch to use it :-)
20:32.55keriobtw, what's the status of the bme userland replacement?
20:33.12Palimaybe it is time to try it :-)
20:33.43Palihald-addon-bme worked when I tried it last time
20:34.01Paliand libbmeipc worked when freemangordon tested it
20:34.15keriohm
20:34.22PaliI only added temperature support from rx51_battery to libbmeipc but not tested
20:35.12*** join/#maemo-ssu dhbiker (~dhbiker@95.87.145.172)
20:37.31Palikerio, do you want to try hald-addon-bme?
20:37.47kerioyeah, but let me make a full backup first
20:38.21Paliok
20:44.40*** join/#maemo-ssu _ade_ (~arno@52481E0B.cm-4-1a.dynamic.ziggo.nl)
20:45.24*** join/#maemo-ssu MrPingu (~MrPingute@86.92.226.97)
20:46.29_ade_Pali: perhaps you should make your status-area-applet-battery conflict with Advanced Power? I had a battery icon twice in the status menu and a double battery status in the menu after installing
20:47.03Pali_ade_, my version replace stock package
20:47.14Paliit is same as if you reinstall stock nokia package
20:47.31Palithen it is problem with advanced power...
20:47.45_ade_pali: I guess you are right if you put it that way.
20:48.36kerio_ade_: reinstalling APM will probably fix it
20:48.47kerioalso try rebooting
20:48.51kerioor killing hildon-status-menu
20:49.19_ade_kerio: I removed advanced power before rebooting
20:49.43_ade_will try to reinstall APM...
20:50.19kerioPali: ok, how do i install your bme replacement?
20:51.14Palikerio, full update of packages I did not tested
20:51.33PaliI only tested hald-addon-bme and now I commited more patches to it
20:51.48keriowell, the bme package should be fairly solid, right?
20:51.55kerioit's just a replacement of initscripts
20:52.03PaliI will push all changes to git
20:56.44_ade_pali: reinstalling APM worked. It does not revert any of the changes you made?
20:57.06Pali_ade_ I do not know what APM is doing...
20:57.23_ade_Me neither ;-)
20:57.24Palimy status area plugin is in same package name as Nokia
20:57.33Paliand has same file name
20:57.46Palionly version was increased :-)
20:57.59Palikerio, now all changes are in git on: https://gitorious.org/rx51-bme-replacement/
20:59.44Palikerio, if you want to test hald-addon-bme: backup old hald-addon-bme binary and update it. then run: sudo stop hal; sudo /etc/init.d/hal stop; sudo /etc/init.d/hal start
21:00.08keriodoes the new hald-addon-bme work with bme too?
21:00.14Palino
21:00.22kerioor, at least, does it stay quiet if bme is running?
21:00.22Paliit work only with kernel drivers
21:00.44Paliyes, it can be started, but it report nothing
21:01.02_ade_Pali: then I think I will stick you version solely (without APM) and see if I encounter issues
21:01.24Palikerio, and it can wait for hotplug kernel drivers
21:01.42Paliand it write very long log output to: /tmp/hald-addon-bme.log
21:02.20Pali_ade_, ok
21:05.11Palikerio, and hald-addon-bme binary is in /usr/lib/hal/
21:18.10*** join/#maemo-ssu MrPingu (~MrPingute@86.92.226.97)
21:20.00*** join/#maemo-ssu gregoa (~gregoa@chello212186052066.410.14.vie.surfer.at)
21:24.46*** join/#maemo-ssu MrPingu (~MrPingute@86.92.226.97)
21:29.05Palikerio, _ade_, if you testing new battery status area applet, tell me if all notifications (battery low, battery empty, battery full, charging, discharging) working
21:29.18Palialso if sound notifications too
21:29.37_ade_Pali: okay
21:29.55Paliand if no sound play when silent profile (or disabled system sounds)
21:32.25*** join/#maemo-ssu lizardo (~lizardo@88.red-2-139-206.staticip.rima-tde.net)
21:32.43*** join/#maemo-ssu arcean (~arcean@aacy139.neoplus.adsl.tpnet.pl)
21:33.17Palikerio, hald-addon-bme working
21:33.39kerioare you asking me?
21:35.33kerioPali: also, what deals with the led and the shutdown on EDVF?
21:36.17kerioPali: also i don't have a toolchain to compile hal-addon-bme :)
21:36.21Palihald-addon-bme managing MCE led patterns and also send DSME signal for shutdown
21:36.54keriodoes hald-addon-bme need rx51-battery?
21:37.07Paliyes, for design capacity
21:37.12Palibut working without it
21:37.20Pali(it report design capacity = 0)
21:37.39Palikerio, here is elf binary: http://atrey.karlin.mff.cuni.cz/~pali/hald-addon-bme
21:38.07_ade_Pali: should the battery status now still work once bme is stopped? Or does an alternative driver has to be loaded then?
21:38.19kerio_ade_: you need the modules
21:38.29Pali_ade_, if bme is stopped you need to load bq27x00_battery module
21:38.36keriobq2415x-charger and bq27x00-battery
21:38.42keriooh, maybe you don't need -charger
21:38.51Paliand bq27x00_battery module must be unloaded before bme is started
21:39.27_ade_okay, I thought so, but just checking.
21:40.42kerioPali: hm, stopping hal caused my n900 to shutdown
21:41.00Palikerio, how did you stopped hal?
21:41.03keriostop hal
21:41.25Palihm...
21:42.28kerioanyway, i'm going to try charging
21:43.45kerioPali: hahaha wtf did you do
21:43.54kerioi get the charging message when the charger isn't connected
21:44.08Palifrom HAL?
21:44.17keriook, now it's working correctly
21:44.23keriofor a moment it was like inverted
21:45.12keriook, i'm getting the correct LED pattern when the screen is locked
21:46.13kerioPali: it's horrendously slow at figuring that the charger was plugged in/removed
21:46.30Palikerio, I see that too
21:46.36Palitrying to debug it
21:46.36kerioPali: it's just the status menu i think
21:46.39Palino
21:46.43Paliproblem is in hal
21:46.43keriothe led pattern is instant-ish
21:47.17Paliah... there is another problem: status menu plugin will not read values from bme if bme is not running
21:47.28Palior new version of bme package is not installed
21:47.45Paliso status menu plugin getting charger value from bq27x00_battery
21:47.54Paliand this can take a time
21:47.57keriohm
21:48.02keriois there no way to make that faster'
21:48.02kerio?
21:48.44Paliso I need to add 1) bq2415x_charger support to status menu or 2) use values from HAL bme if bme is stopped or 3) update bme package :-)
21:49.05Palikerio, I think that option 3) will be enought
21:52.38Palikerio, ok problem is not in hal, only in status menu plugin
21:53.11keriowhat does "update bme package" mean?
21:53.17keriothat is, what will it change?
21:53.51Paliupdate bme-rx-51 deb package
21:54.02kerioyeah, but what will change?
21:54.25keriowhy would it make the battery applet update more quickly?
21:54.30Paliif bme-rx-51 deb package version 1.0 (or high) is installed, then status menu plugin thinks that BME is running
21:54.44keriooic
21:54.56Paliif status menu plugin thinks that bme is not running it will not read values from hald-addon-bme
21:55.14Pali(because that values can be out-of-date)
21:55.28keriohold on, is the check actually on the version on the package? :S
21:55.31keriojust... no
21:55.50Paliyes, on the package version
21:56.16kerio:S
21:56.34Palithis is good way how to check if new version of binary in debian package is installed
21:56.49keriohold on, new version of the binary? :o
21:56.57kerioyour bme actually ships a bme binary?
21:57.12Palino bme binary
21:57.25Palionly bme-rx-51 deb metapackage
21:57.49keriocan't you ship an empty /etc/nobme file instead?
21:58.08kerioif anything, because checking a package version is quite expensive
21:58.23Paliwhy? it is check once at startup
21:58.42kerio...oh, so that's why the status menu hangs for like 5 seconds before loading >:C
21:59.18Palinow it show you message when battery is really started charging :-)
21:59.38keriodepending on dpkg is an unnecessary complication
22:00.35kerioand it also makes it quite hard to test, because dpkg is a fucking mess and you can't easily switch between versions
22:02.52keriogregoa: you around?
22:03.59gregoakerio: yes but I'm doing 2 or 3 thing in parallel, so only easy questions please :)
22:04.12Palikerio, if you want I can compile all deb packages for testing :D
22:04.45keriogregoa: Pali's status applet checks dpkg for the version of a package and has a different behaviour depending on it
22:05.24kerioi'd say it's a bad idea - is it?
22:05.44gregoagregoa: at install time? at runtime once? every second? something else?
22:05.48kerioruntime once
22:05.55keriobut it's still silly
22:06.05Palithere is no other way how to check if is installed Nokia BME version or new replacement
22:06.21kerioof course there's not, you haven't put an empty file somewhere :s
22:06.30Paliusing someting in /etc/ is worse...
22:06.32keriosomething that can be replicated *manually* easily
22:06.41gregoahm, not a packaging question :)
22:07.04Paliconffiles are not cleaned automatically after upgrading/downgrading
22:07.16kerio/usr/share/ then
22:07.17kerioor something
22:07.25Palido you remember MAG problem with ke-recv in CSSU?
22:07.33keriohm, i don't
22:07.34gregoaI mean, shelling out obvisouls slows down startup but I have no idea if there's other way to find the version or differentiate between "old" and "new"
22:08.24kerioPali: actually, much easier
22:08.46keriocheck for the existance of /usr/sbin/bme_RX-51
22:08.48kerio:D
22:09.23Palibad idea
22:09.54Paliyou have bme binary, so it will not work
22:10.05kerioyeah but i can divert it easily
22:10.22kerioinstead of installing a new package that will delete it
22:10.48Palikerio, create new shell script which do: while sleep 1000000; do true; done
22:11.19Paliand store it to /usr/sbin/bme_RX-51
22:11.24Paliand start it
22:11.35Pali(and divert old bme somewhere)
22:12.03kerioi'm still firmly against using packaging metadata to discern behaviours
22:12.05Palidirty hack, but testing now without recompiling applet
22:12.17keriothe packaging is about what's installed through apt, not about what's installed
22:12.44kerioanyway, testing it :)
22:13.51kerioshould i do "start bme" at this point?
22:13.52kerioor just run it?
22:15.12keriooh rofl, i didn't make it executable and now everything is botched
22:15.14*** join/#maemo-ssu IronLegend (~IronLegen@ironlegend.student.ipv6.utwente.nl)
22:15.22keriothankfully i have rescueos :D
22:16.27DocScrutinizer05((<Pali> there is no other way how to check if is installed Nokia BME version or new replacement))  if [  "$(md5sum /usr/sbin/bme_RX-51|cut -d ' ' -f 1)" == "c2eb6586292b9afa407c307ebc4894e7" ]; then echo "original bme detected"; fi
22:18.06PaliDocScrutinizer05, I need to check if nokias hald-addon-bme is not installed
22:18.20DocScrutinizer05well, seems similarly easy
22:18.29Paliand your solution is like my...
22:18.43Palicalling md5sum or dpkg
22:18.47DocScrutinizer05not sure about that
22:18.53kerioi'm also firmly against this kind of extremely specific checking
22:19.07Palimd5sum must read full file
22:19.13DocScrutinizer05dpkg is heavy, and depending on a lot of stuff, and might not even always be available
22:19.24Paliand dpkg must read parse /var/lib/dpkg/state file
22:20.09DocScrutinizer05state?
22:20.13kerioPali: why doesn't your bme replacement just ship a fake process that only sleeps in /usr/sbin/bme_RX-51
22:20.34Palibecause it is stupid to have one process
22:20.47Paliwhich will only sleep because of nokia...
22:20.49DocScrutinizer05I recall moving /var/lib/dpkg to last dark corner of my MyDocs, to make room on rootfs
22:20.55DocScrutinizer05occasionally
22:21.04kerioi just optified it :)
22:21.30DocScrutinizer05aq
22:21.32DocScrutinizer05anyway
22:21.36kerioPali: something's wrong, it's still slow
22:21.53kerio<PROTECTED>
22:21.54kerioah crap
22:22.00DocScrutinizer05I don't see which problem was the right problem for dpkg check being the solution
22:22.08Palipgrep -f ^/usr/sbin/bme_RX-51
22:22.38Palikerio, make it executable
22:22.39DocScrutinizer05probably tl;dr for chanlog
22:22.46kerioPali: it's running
22:22.47Paliand start without /bin/sh
22:22.54keriohm
22:22.57Palikerio, ok then restart applet
22:23.07kerioi already rebooted dammit
22:23.49kerioit still appears as {bme_RX-51} /bin/sh /usr/sbin/bme_RX-51
22:23.53kerioit's probably the problem here
22:24.11DocScrutinizer05I'm rather sure about any package being *installed* is not the right flag for anything outside installation process to decide on it
22:24.24Palikerio, did you started without /bin/sh prefix?
22:24.30kerioyes
22:24.36keriosudo /usr/sbin/bme_RX-51
22:24.41kerioi can't do more, i'm afraid
22:24.49Palikerio, run: pgrep -f ^/usr/sbin/bme_RX-51
22:25.00DocScrutinizer05sudo what?
22:25.27kerioPali: nothing
22:25.31DocScrutinizer05don't you think a root process would also need root's environment?
22:25.40kerioDocScrutinizer05: it's a script that does jack shit
22:25.54kerioall i need is something that says "i'm /usr/sbin/bme_RX-51" and does nothing
22:25.57DocScrutinizer05what for?
22:26.00Palikerio, run without root
22:26.16keriopgrep still fails
22:26.25Palikerio, better start from bash
22:26.44Palimaybe busybox doing something stupid if you starting shell script from shell
22:26.45kerioDocScrutinizer05: because Pali's new battery applet does a shitton of weird tests to choose what to do
22:27.02DocScrutinizer05:-/
22:27.20kerioPali: now it's bash /usr/sbin/bme_RX-51
22:27.25*** join/#maemo-ssu nox- (noident@freebsd/developer/nox)
22:27.37PaliDocScrutinizer05, do you have idea how to check if new version of hald-addon-bme is installed?
22:27.39kerioby the way, you could just make it use bq2415x-charger :)
22:28.17DocScrutinizer05Pali: well, for hostmode kernel we invented a version string in some /proc/* output
22:28.35Palibut this is userland part
22:28.44kerioDocScrutinizer05: as a sidenote, bq27200's full charge is the calibrated full charge, right?
22:28.58kerioand you should use that, instead of the design capacity
22:29.14DocScrutinizer05err, yes?
22:30.19kerioDocScrutinizer05: i mean, bq27200 is the most accurate thing you can get, wrt battery status, so it should supersede anything else
22:31.39DocScrutinizer05regarding what?
22:32.31keriocurrent consumption, TTE/TTF, full charge, percentage
22:32.42keriocurrent charge, too
22:32.45DocScrutinizer05bq27200 is accurate only when CI=0
22:33.22DocScrutinizer05on all of those you listed except current consumption
22:33.58keriohm, how do i get CI with the module?
22:33.59DocScrutinizer05CI=1 and they could be off like if you had a car lead acid battery in that thing
22:34.11kerioDocScrutinizer05: yeah but maybe they're not :3
22:35.00keriough, you need to get the value from bq27200-0/registers
22:35.11DocScrutinizer05eeeek
22:35.53DocScrutinizer05look at bq27k-detail2.sh and then ponder why I decided to list stuff that way
22:36.34keriohehehe
22:37.10DocScrutinizer05I as well could have done a hexdump
22:37.27DocScrutinizer05but I thought it's more useful the way it is now
22:38.42DocScrutinizer05btw the bq27000.ko of openmoko actually had /raw sysnode as well, where you in fact could get that hexdump
22:38.48DocScrutinizer05if you really need it
22:38.59Paliin kp too
22:39.42DocScrutinizer05wm8753 audio mixer even allows writing raw hex values to same /raw sysnode in its sys-API
22:40.06kerioPali: actually a "flags" file in bq27200-0 would be neat
22:40.23DocScrutinizer05a flags dir maybe?
22:40.59DocScrutinizer05hell, there aren't that many. make a /CI file, a /VDF1 file etc
22:41.12Palipatches are welcome :-)
22:45.26kerioPali: are you going to add bq24k support to your status applet then?
22:45.28kerioit would be neat
22:49.15Palikerio, it will not help
22:49.26Palibecause current code reading data from HAL
22:49.42kerioyeah, i said *add* (:
22:49.43Paliand HAL updating values only every 10s
22:50.03Paliso this is reason why it is slow for this
22:51.22Palikerio, code for receiving events from bq2415x kernel driver is in hald-addon-bme
22:51.34Paliand I do not want to copy & paste same code
22:51.55keriocan't you always grab the data from hald-addon-bme then?
22:52.32kerioskip the "compatibility" thing and require people to use bme+stock hald-addon or modules+replacement hald-addon
22:52.35Palikerio, I can only from new hald-addon-bme
22:52.59keriohold on, what happens if you install the applet with everything else stock?
23:04.31*** join/#maemo-ssu luf (~luf@ip-89-103-223-164.net.upcbroadband.cz)
23:15.23lufmerlin1991: I pushed the fix to the obexd git repository.
23:21.40*** join/#maemo-ssu wmarone (~wmarone@c-67-174-151-253.hsd1.ca.comcast.net)
23:40.30*** join/#maemo-ssu Timo (~timo@unaffiliated/tiempjuuh)
23:56.34*** join/#maemo-ssu lizardo (~lizardo@88.red-2-139-206.staticip.rima-tde.net)

Generated by irclog2html.pl Modified by Tim Riker to work with infobot.