/srv/irclogs.ubuntu.com/2012/04/01/#ubuntu-arm.txt

MMloshMade some testing on the program..   the   arguments parsing loop is defective..   It loops on the "default" case.   Hardcoding the config path makes the program run fine01:08
MMloshThe problem seems to be signed-vs-unsigned shortint..     It returns "255" while the while loops checks for "-1"01:18
MMloshNevermind..   the "ch" variable should be int instead of char..  It's a "normal" application bug01:39
MMloshThe bug was filled as https://bugs.launchpad.net/ubuntu/+source/getstream/+bug/970520  I am sorry for the flood01:50
ubot2Launchpad bug 970520 in getstream "Does not work on ARMv7 at all (pandaboard)" [Undecided,New]01:50
infinityMMlosh: Fixed in precise, thanks for the catch.02:15
MMloshIf I want it fixed in debian, I need to post a bug there, right?02:16
* infinity notes that he missed the word "on" in his changelog entry. Oops.02:16
infinityMMlosh: I can push the patch to Debian in a bit.02:16
MMloshalso: are you sure that my fix is correct?02:16
MMloshthank you02:16
infinityMMlosh: Your fix is correct.  getopt returns an int.  Casting ints and chars interchangeably works on x86 (hence the bug), because x86 uses signed chars by default.  ARM and PowerPC use unsigned chars by default.02:17
infinityOf course... getstream currently doesn't build on precise.  I should have checked that before I uploaded.02:22
infinity*sigh*02:22
stgraberinfinity: and now that you are touched-it-last, congratulations on making this your FTBFS ;)02:23
infinitystgraber: Yeah, it's just glib include whininess, nothing drastic.02:23
infinitystgraber: And I can be all revisionist history and fix the typo in my previous changelog. ;)02:23
MMloshahs3, yes, forgot that it won't rebuild because of that..   I workarounded that by including whole <glib.h> in my testbuilds..02:25
infinityahs3: Ignore that, MMlosh is tab-complete-impaired. ;)02:25
infinityMMlosh: Yeah, I'll fix in a bit, after I go get a drink. :P02:26
infinityMy own dumb fault for thinking an obviously-correct 1-line fix wouldn't need a test build.02:26
MMloshI wondered where that "3" came from..       hitting tab..  sleepy me..   (europe...)02:27
MMloshinfinity,  minor question  --  Any idea why it's set to build with  "-O0" ?02:46
infinityMMlosh: No idea.02:47
infinity(it is?)02:47
MMloshThe only guess I have is.. someone actually trying the binary after building..  and trying to debug02:47
infinityHrm, so it is.02:48
infinityThat's an upstream decision, not a Debian or Ubuntu modification.  No clue why.02:48
MMloshoh..02:49
infinityI'd change it, but if upstream had a legit reason to do it (I suspect not, but maybe), I'd prefer not to break it and find out later. :P02:50
* infinity testbuilds this time, to avoid further embarrassment.02:53
infinityMMlosh: Well, I'll forward the patch to Debian, but the Debian maintainer hasn't uploaded since 2009.  I might NMU for some of the outstanding issues, if I'm feeling keen.03:06
infinityMMlosh: But, anyway, fixed in Ubuntu (no, really, it builds this time!)03:06
MMloshgreat..  I haven't expected such quick reaction..  since there was no update since that 2009...03:06
MMloshbut the coder itself was kinda-active in 201103:08
infinityOh, curious, the maintainer did respond to a bug for a new upstream version only a few days ago.  Maybe he's not dead.03:10
MMloshneat03:21
MMloshpeople want dvb-s2 I guess03:22
infinityOh, and not going to bother forwarding that fix to Debian, looks like the new upstream version fixes the cast.03:38
infinitySo, Debian will get it for free when they rev, which the maintainer seems to be working on.03:38
MMloshinfinity, interesting..   I would bet that the bug was kept    and lose07:37
drinkin_leninadehi guys, I'm looking for an ARM guru that can help me with a non ubuntu related question…07:51
MMloshinfinity, do you know, if the debian maintainer plans to release the current version soon?     (Since the program is unable to acquire "signal lock" on our multiplex with highest frequency)11:45
=== yofel_ is now known as yofel
infinityMMlosh: Not sure, to be honest.18:38
=== calculu5 is now known as calculus

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!