00:00:11 | rarog | ah yes... i noticed this that it crashed when i was in usb bootloader, put new rockbox code on it and then pulled the cable. |
00:00:27 | CIA-44 | 0d78dd1 build result: 0 errors, 2 warnings (Amaury Pouly committed) |
00:01:12 | | Join evilnick__ [0] (~evilnick@5e007ed8.bb.sky.com) |
00:01:45 | pamaury | not my warnings |
00:01:51 | | Nick evilnick__ is now known as evilinick (~evilnick@5e007ed8.bb.sky.com) |
00:01:57 | | Nick evilinick is now known as evilnick (~evilnick@5e007ed8.bb.sky.com) |
00:02:03 | rarog | kugel: has rockbox for android actually real unicode or real unicode font support? |
00:02:12 | | Quit evilnick (Changing host) |
00:02:12 | | Join evilnick [0] (~evilnick@rockbox/staff/evilnick) |
00:02:51 | rarog | pamaury: I'm compiling bootloader and fuze now, will tell you the test results after flashing. |
00:03:20 | CIA-44 | Commit 8c0c844 in rockbox by Amaury Pouly: imx233: always boot when powered up from 5v |
00:03:47 | pamaury | the last commit (8c0c844) requires to rebuild mkimxboot |
00:04:00 | | Quit wodz (Quit: Leaving) |
00:05:15 | | Quit evilnick_ (Ping timeout: 276 seconds) |
00:05:47 | CIA-44 | Commit e958aee in rockbox by Thomas Martitz: root menu ordering: Remove underscores from the config setting. |
00:05:48 | CIA-44 | 8c0c844 build result: All green |
00:06:07 | kugel | rarog: it should |
00:06:19 | | Quit Xerion (Ping timeout: 240 seconds) |
00:08:07 | CIA-44 | e958aee build result: All green |
00:09:49 | | Quit davo (Quit: Lost terminal) |
00:10:05 | rarog | kugel: as for now it seems not to work for me. I tried with the same set of files on my fuze+ and on my android, both are set to default to utf8, but on android it shows question marks instead of every non-latin char. |
00:10:58 | kugel | you need to use a font that has a sufficient glyphcoverage |
00:11:05 | rarog | pamaury: ok, recompiled everything and compiling again. :D |
00:11:22 | | Join Rower [0] (husvagn@v-413-alfarv-90.bitnet.nu) |
00:12:11 | rarog | kugel, is the default font not enough? or is it built differently from target to target? I didn't change anything theme related, so it should be same default settings as far as I see it. |
00:13:20 | | Quit Rower85 (Ping timeout: 255 seconds) |
00:13:41 | | Quit nosa-j (Read error: Connection reset by peer) |
00:15:04 | | Join nosa-j [0] (~m00k@adsl-74-235-42-78.clt.bellsouth.net) |
00:16:08 | Stephen__ | if i need a gigabeat s build to see a drive over 120gb do i need a custom build ? |
00:21:00 | Torne | Stephen__: if you need the gigabeat S to work with a drive over 120GB you need to do a bunch of weird and complicated steps |
00:21:17 | Torne | no build will make it just work; the original firmware doesn't like large drives |
00:21:18 | Stephen__ | i had it all running nicely till i updated to the current build |
00:21:31 | Torne | oh. |
00:21:38 | Torne | and now what? |
00:21:41 | Torne | only one partition shows? |
00:21:45 | Torne | you need a custom build, yes |
00:22:01 | Torne | to handle the weird partition type magic |
00:22:12 | Stephen__ | ah right, will see can i find the original build I had on it |
00:22:30 | Torne | possibly some other stuff too, i forget the details |
00:22:45 | Torne | but if you were the guy we helped do it before then the patches will probably be around in our logs :) |
00:23:05 | Stephen__ | yeah I was, i have my old builds lets see if they work |
00:23:19 | | Quit ender` (Quit: The curious thing about .Net is that it allows you to use any language you want, as long as it's C#.) |
00:23:34 | Torne | anyway, it should probably keep working okay with just the partition that goes up to 128GB with the standard build |
00:23:39 | Torne | so it shouldn't be completely nonfunctional :) |
00:23:57 | Stephen__ | yeah but it's bugging me now haha |
00:24:14 | Stephen__ | the stuf fi listen to is on the 128gb anyway. |
00:27:00 | Stephen__ | my old copy works again. now to not update it till i get git working. thanks for the help Torne |
00:27:28 | Torne | if you still have the patch it should hopefully just apply to the latest version |
00:28:14 | Stephen__ | i dont but then I'm n windows now so it could be on my linux side |
00:28:35 | rarog | pamaury: After flashing new bootloader and putting new rockbox version in usb mode: Data bort at 60077604, FSR 0x1, (domain 0, fault 1), address 0x00000005 |
00:29:07 | pamaury | yeah, but reboot and retry |
00:29:20 | rarog | Yes, after reboot it works of course. |
00:29:28 | pamaury | ok, just like mine then |
00:29:34 | pamaury | thanks |
00:29:51 | rarog | This was the behaviour for some builds. Though I didn#t write down the crashes. |
00:30:10 | rarog | Yet at least it builds and boots now. :) |
00:31:32 | | Quit adnap (Ping timeout: 248 seconds) |
00:32:17 | rarog | kugel: both configurations have Adobe-Helvetica as the only font set... Though with different numbers before the name, but I think this is just the font size or does it mean something else? |
00:32:40 | | Quit enthdegree (Read error: Connection reset by peer) |
00:32:55 | | Join enthdegree [0] (~BitchX@unaffiliated/enthdegree) |
00:33:33 | | Quit tchan (Quit: WeeChat 0.3.7) |
00:38:23 | Stephen__ | bertrik, are you able to get the custom root menu working on the zip ? |
00:38:25 | | Quit pamaury (Read error: Connection reset by peer) |
00:39:36 | | Join adnap [0] (~adnap@rrcs-71-42-140-57.sw.biz.rr.com) |
00:51:55 | dfkt | i just resynced the "shutdown in main menu" patch for RaaA, since /apps/root_menu.c has changed recently - http://pastie.org/3484123 - it works, however i see just the icon for the menu item, not the word/description. what did i miss, or do wrong? |
00:54:24 | | Join evilnick_ [0] (~evilnick@5e007ed8.bb.sky.com) |
00:55:39 | bertrik | Stephen__, I haven't tried |
00:55:45 | | Quit bertrik (Quit: And That, My Liege, Is How We Know the Earth to Be Banana Shaped) |
00:56:16 | Stephen__ | doesnt work for me. |
00:57:11 | JdGordon | dfkt: I had planned on fixing that up shortly too |
00:57:41 | JdGordon | the "problem" with your copy is that most taregst don't want to add it by default |
00:57:51 | | Quit evilnick (Ping timeout: 246 seconds) |
00:57:59 | JdGordon | so that table with the items needs a 3rd column to say if the item should be added by default or not |
00:58:31 | JdGordon | or maybe just prepend the name with a ? or something |
00:58:53 | dfkt | you mean { "shutdown", &do_shutdown_item, Shutdown }, or the like? |
00:59:00 | JdGordon | dfkt: if you see no text you almost certainly need to update your english.lang |
00:59:21 | dfkt | JdGordon, i just did a git hard reset |
00:59:29 | dfkt | so everything should be fresh |
00:59:32 | JdGordon | no, you need to nuke your build dir |
00:59:46 | dfkt | ah, ok |
01:00 |
01:00:01 | dfkt | good thing that goes fast with git now :) |
01:04:56 | | Join saratoga [0] (98032909@gateway/web/freenode/ip.152.3.41.9) |
01:05:41 | saratoga | is there some way we could more cautiously operate the metadata parsers so that theres less risk of a problem there crashing rockbox? |
01:06:21 | saratoga | i was thinking maybe longjmp, but i guess that doesn't really work since we generally dont' know that the parser crashed |
01:06:31 | | Quit rarog (Quit: ChatZilla 0.9.88 [SeaMonkey 2.4.1/20111011102430]) |
01:07:07 | | Join CaptainKewl [0] (captainkew@207-237-110-248.c3-0.nyr-ubr2.nyr.ny.cable.rcn.com) |
01:07:15 | | Quit parafin (Remote host closed the connection) |
01:07:55 | | Join parafin [0] (parafin@paraf.in) |
01:10:25 | JdGordon | dfkt: g#156 |
01:10:26 | fs-bluebot | Gerrit review #156 at http://gerrit.rockbox.org/r/#change,156 : root menu: add the shutdown option to all builds by Jonathan Gordon (changes/56/156/1) |
01:10:52 | JdGordon | you'll need to add it to your config though |
01:12:40 | | Join Rower85 [0] (husvagn@v-413-alfarv-90.bitnet.nu) |
01:15:58 | | Quit Rower (Ping timeout: 272 seconds) |
01:16:41 | JdGordon | gevaerts: you used the new tag? |
01:17:16 | gevaerts | JdGordon: yes, but the theme site refuses to let me upload! |
01:17:26 | JdGordon | how did you use it? |
01:17:36 | * | JdGordon demands screenshots! |
01:18:10 | JdGordon | if I need to procrastinate today I'm going to do a bit of a write up on using it properly as it has some wierdness |
01:19:14 | gevaerts | http://rockbox.hostname.be/wavy/ |
01:19:32 | gevaerts | the wps uses the new layer tag, the sbs uses the "old" skinned lists |
01:19:44 | dfkt | JdGordon, thanks - but i don't know a thing about how gerrit works. i just run the commandline on that page? |
01:20:18 | gevaerts | 240x320. Feel free to play around, there are some glitches that might well be my fault, but I wouldn't mind a second opinion :) |
01:21:05 | JdGordon | dfkt: yes, go back to clean git and run the checkout command from that age |
01:21:27 | dfkt | ok, thanks |
01:21:30 | JdGordon | bahahaha love it! |
01:22:12 | dfkt | gevaerts, that only works for chopin - make one for kraftwerk that's more sawtooth-wave like :) |
01:22:25 | gevaerts | dfkt: go ahead :) |
01:22:53 | gevaerts | It's a good thing the old viewport limit is gone! |
01:23:39 | gevaerts | oh |
01:24:04 | * | gevaerts feels a bit stupid now. He could make the progressbar a *lot* smoother, probably |
01:24:20 | | Quit Stephen__ (Quit: Leaving) |
01:24:39 | gevaerts | It should render to the back buffer. That doesn't work currently because I used transparency, but that's not needed, really, it can just copy the backround image |
01:32:53 | *** | Saving seen data "./dancer.seen" |
01:35:17 | | Quit gartral (Ping timeout: 246 seconds) |
01:36:19 | | Join gartral [0] (~gareth@ip184-189-215-49.cl.ri.cox.net) |
01:41:04 | | Quit Thra11_ (Quit: kthxbai) |
01:44:50 | dfkt | JdGordon, to enable shutdown, i have to change { SHUTDOWN_PREFIX "shutdown", &do_shutdown_item }, to { ? "shutdown", &do_shutdown_item }, |
01:44:51 | dfkt | ? |
01:46:56 | JdGordon | no, add it to your config.cfg |
01:47:12 | JdGordon | "root menu order: files,wps,settings,shutdown" or whatever you want |
01:47:30 | dfkt | well, that is of course a lot more convenient :D |
01:48:03 | JdGordon | well sure, if you want it permenantly then remove the SHUTDOWN_PREFIX from the line |
01:48:16 | JdGordon | but the patch as i did it has far more chance of being commited |
01:48:27 | JdGordon | most people dont want that option in the menu |
01:48:53 | gevaerts | JdGordon: would I be very annoying if I ask for more than two layers? |
01:49:10 | JdGordon | no, but I',m not sure how that would work |
01:49:14 | dfkt | sure, on most players it's useless, and it's against android philosophy too - but i find it easier to shut down rockbox than hunt for the stop button and minimize, or similar |
01:50:18 | JdGordon | right, so add it to your config |
01:50:23 | JdGordon | or hack the patch |
01:52:45 | | Join [Saint] [0] (~Saint]@unaffiliated/saint/x-8516940) |
02:00 |
02:00:57 | [Saint] | You're a bastard JdGordon ;) |
02:01:07 | JdGordon | why this time? |
02:01:17 | [Saint] | Distracting my magpie like personality with shiny things :p |
02:01:29 | JdGordon | sorry! :D |
02:01:39 | [Saint] | The commit log is a damn fine one. |
02:01:57 | [Saint] | Fancy layering, yes please. |
02:02:45 | dfkt | does the layering mean that album art now can actually be the top layer? |
02:02:53 | [Saint] | Cutting out the irrelevant 0,0 from 99% of my %xl calls also pleases me. |
02:03:21 | JdGordon | i actually changed how %x works, so if you use them at all let me know if i broke it |
02:03:31 | [Saint] | dfkt: yes. |
02:03:36 | dfkt | \o/ |
02:03:56 | gevaerts | dfkt: or the bottom layer! |
02:04:22 | dfkt | gevaerts, that feature was already available - also interactive art where album art gets "eaten" by scrolling lines :) |
02:04:25 | [Saint] | JdGordon: I read that, but, I'm a "%xl and %xd just because I can (and I think its neater) man". |
02:04:29 | JdGordon | gevaerts: ok, lets say you had 3 layers? how would u do it? |
02:05:02 | gevaerts | JdGordon: how is it done now? |
02:05:03 | dfkt | gevaerts, ah right - that's new, like your wavy lines over chopin's head |
02:05:55 | JdGordon | gevaerts: the lcd driver really only deals with the framebuffer (what its drawing to) and the backdrop buffer |
02:06:00 | [Saint] | dfkt: errrr ...I'm not sure what you mean by "was already available"...as its simply not true. |
02:06:36 | [Saint] | There was (and still isn't) any concept of layering. Just backdrop, and everything else. |
02:06:41 | dfkt | [Saint], yes, i misunderstood - i meant i always had stuff drawn over my album art that i didn't want there |
02:06:42 | evilnick_ | [Saint]: It was a joke |
02:06:49 | evilnick_ | And that ^ |
02:07:37 | gevaerts | JdGordon: right. This needs more thinking i guess |
02:07:50 | [Saint] | Aaaaaawwwwwwww! Argh! |
02:08:15 | [Saint] | Now I can do what I probably should've done anyway, but couldn't, with the polyps. |
02:08:21 | [Saint] | *popups |
02:08:53 | [Saint] | Ie. Draw the popup *on top* of the text viewport instead of replacing it. |
02:09:14 | KiwiCam | How do. If you have one viewport overlapping the other slightly on the Clip. Is there a way to have the top viewport's white part of the bitmap transparent? ie Twwo colour bitmap with the Black colour representing the Yellow or Blue on the Clip and the White portions showing up as Black areas on the Clip. Is there a transparency option? |
02:09:19 | JdGordon | [Saint]: no! |
02:09:33 | JdGordon | dont ever draw text on the backdrop buffer |
02:09:42 | [Saint] | Wont work? |
02:09:57 | JdGordon | it might work, but probably not how you want it |
02:09:59 | gevaerts | JdGordon: see, that's why we need more layers! :) |
02:11:15 | [Saint] | JdGordon: I could swap it to the backdrop buffer conditionally, no? It wouldn't need to scroll as its mostly covered. It'd just make for a nicer effect (but add four or five new viewports). |
02:11:31 | [Saint] | "It" being the text viewports. |
02:11:52 | * | gevaerts gave up on scrolling. It doesn't work well on skinned lists or vertical text anyway |
02:12:27 | [Saint] | "Vertical scrolling" is a *complete* hack, though :) |
02:12:54 | gevaerts | I wish I knew a way to do it1 |
02:12:55 | KiwiCam | gevaerts: Yes. Even just being able to use subimages in the %X() would be great. |
02:13:19 | [Saint] | gevaerts: I showed you a way to do it. Didn't I? |
02:13:31 | gevaerts | You did? |
02:13:41 | [Saint] | I seem to recall so, yeah. |
02:13:47 | gevaerts | Ah, yes, for fixed text? |
02:14:01 | [Saint] | It can be done with alternating sublines. |
02:14:09 | gevaerts | It can't in general |
02:14:15 | [Saint] | And yeah, the text needs to be known. |
02:15:01 | | Quit funman (Ping timeout: 245 seconds) |
02:16:06 | | Join DerPapst [0] (~Adium@91-64-115-27-dynip.superkabel.de) |
02:24:49 | | Quit kadoban (Read error: Operation timed out) |
02:25:35 | | Join mystica555_ [0] (~Mike@mystica.mobi) |
02:25:36 | | Quit mystica555_ (Excess Flood) |
02:32:19 | [Saint] | KiwiCam: what were you thinking regarding your last statement? |
02:32:57 | [Saint] | I can't see how that would help with what gevaerts wants to do. |
02:34:30 | [Saint] | Regarding "how do you do foo if you have two viewports overlapping"....you don't, basically, as that's wrong for a bunch of reasons. |
02:35:09 | [Saint] | There's no need to have viewports overlap, switch them out conditionally, don't overlap them. |
02:37:40 | [Saint] | You mention "the top viewport", and...there's no concept of top or bottom. There's backdrop, and "not the backdrop", and that's it. |
02:37:55 | JdGordon | well, that now gets murky |
02:38:24 | [Saint] | Not really, if you draw in the backdrop, its still "backdrop" IMO. |
02:43:24 | [Saint] | Hmmmmm....I should be able to make a theme where the entire backdrop is the album art now. |
02:44:06 | [Saint] | Which would probably look terrible, but, that's not the point! :) |
02:44:55 | * | evilnick_ has some great ideas for what to do with the screen real estate of the 1280 x 800 resolution |
02:45:25 | dfkt | JdGordon, root menu order does generally work in my raaa build with 156, but shutdown won't show up |
02:45:41 | dfkt | (funny enough, root menu order doesn't seem to work on the clip zip) |
02:46:33 | [Saint] | dfkt: perhaps I'm wrong...but I thought you could only remove and reshuffle, but not add arbitrary items to, the main menu. |
02:46:47 | [Saint] | (Which is likely why shutdown doesn't work) |
02:46:57 | dfkt | [Saint], g#156 |
02:46:59 | fs-bluebot | Gerrit review #156 at http://gerrit.rockbox.org/r/#change,156 : root menu: add the shutdown option to all builds by Jonathan Gordon (changes/56/156/1) |
02:47:27 | [Saint] | Ah. I see. |
02:47:59 | [Saint] | I didn't realize you were building your own. |
02:48:19 | dfkt | that's why i wrote "in my raaa build with 156" ;) |
02:49:00 | [Saint] | How clean is your tree? |
02:49:18 | dfkt | very. i just went through all that with JdGordon before |
02:49:28 | [Saint] | We had someone in here with a really messed up Clip build of yours yesterday. |
02:49:48 | dfkt | what was messed up? |
02:51:43 | [Saint] | Oh...whoops, that's right. It was (probably) fs woes, but he never confirmed. |
02:51:52 | * | [Saint] just re-read the log. |
02:52:04 | dfkt | my builds work perfectly fine for me, and of the 500+ downloads i didn't hear any bug reports yet |
02:53:57 | [Saint] | Slightly related, why do you need Shutdown for RaaA anyway? |
02:54:10 | [Saint] | If you do, you ROM is messed up. |
02:54:25 | dfkt | JdGordon, very weird... on raaa the cfg entry has to be called "root_menu_order" with underscores to work, on the zip it has to be "root menu order" without underscores |
02:54:35 | JdGordon | evilnick_: i started porting the 480*800 cabbie to the touchpad (1024*768) on landscape, pretty much having two columns |
02:54:56 | gevaerts | dfkt: e958aee? |
02:54:58 | JdGordon | dfkt: yeah, see kugel's commit a few hours ago |
02:55:26 | * | evilnick_ got as far as making the background work. On the .wps but not on the lists themselves. That took about an hour and a half. I've given up now! |
02:56:00 | dfkt | [Saint], just read today's log at 1:49, since i already gave my reasons why i like shutdown in android ;) |
02:56:00 | [Saint] | Are my themes that badly commented? ;) |
02:56:04 | dfkt | gevaerts, yes |
02:56:42 | dfkt | e958aee for the zip build, and e958aee + gerrit 156 for the raaa one |
02:56:44 | fs-bluebot | Gerrit review #156 at http://gerrit.rockbox.org/r/#change,156 : root menu: add the shutdown option to all builds by Jonathan Gordon (changes/56/156/1) |
02:56:50 | [Saint] | dfkt: that's all very well and good, but, IMO its totally unnecessary. |
02:57:02 | dfkt | damn, i wanted to trick gerrit into not spamming :) |
02:57:38 | evilnick_ | [Saint]: Not at all. But the last time I looked into theming must have been about 4 years ago now, before viewports were done. EVERYTHING was aligned to lines of text and it was simple. Now.... a different story! |
02:58:19 | [Saint] | You can still do that. It just might not work particularly well. |
02:58:29 | [Saint] | (And it's horrible :p) |
03:00 |
03:00:09 | [Saint] | JdGordon: does #156 hide the Shutdown entry by default and you enable it if you want it, or show it by default and you hide it if you don't? |
03:00:10 | * | JdGordon suggests that with viewports skinning is simpler now! |
03:00:15 | evilnick_ | I quite agree, but it's a bit of a jaw-drop moment to see how complex .wps files are now |
03:00:22 | JdGordon | [Saint]: hide by default |
03:01:06 | [Saint] | JdGordon: cool. I tried to look myself but my phone spat the dummy trying to load the review. |
03:01:52 | [Saint] | evilnick_: my cabbie probably isn't the best teaching tool it could be ;) |
03:02:15 | [Saint] | It could be worse :) |
03:02:17 | JdGordon | [Saint]: there is no simple way to say "do the whole menu and add the optional items" though which i may have to add if we get more |
03:05:29 | | Quit evilnick_ (Read error: Connection reset by peer) |
03:06:21 | [Saint] | JdGordon: perhaps "root menu order: -*" or something? |
03:06:30 | JdGordon | maybe |
03:06:57 | JdGordon | it becomes less of a problem once we have a ui for it though |
03:09:18 | [Saint] | Its a bit "out there" but it'd be pretty cool for partially sighted or blind users to be able to set/reset menu order through RBUtil. |
03:09:34 | JdGordon | they can |
03:09:58 | JdGordon | there has beeen talk (on and off for the last X years) about adding config support to rbutil |
03:10:27 | [Saint] | So, "they could"? |
03:10:51 | [Saint] | "can" kinda implies its possible now. |
03:14:55 | dfkt | what do i do when git pull spits this cryptogram? - "You are not currently on a branch, so I cannot use any 'branch.<branchname>.merge' in your configuration file." |
03:16:31 | [Saint] | #git ? |
03:16:40 | dfkt | ... |
03:17:32 | dfkt | ah, "git checkout master" appear to be the magic words |
03:18:51 | [Saint] | I'm pointing out that its not really Rockbox specific...not sure what's so astonishing about that. |
03:19:21 | * | [Saint] apologizes for being too obvious. |
03:21:17 | dfkt | yeah, cause i'm the only one who ever asked a git related question in here, which obviously is a very integral part of rockbox lately |
03:32:57 | *** | Saving seen data "./dancer.seen" |
03:33:29 | saratoga | i think asking about git here is fine |
03:38:10 | | Join [Saint_] [0] (~Saint]@unaffiliated/saint/x-8516940) |
03:39:19 | | Quit [Saint] (Ping timeout: 260 seconds) |
03:47:59 | dfkt | JdGordon, shutdown on raaa is now working! \o/ thanks a lot! |
03:48:47 | JdGordon | :) |
03:50:02 | | Quit [Saint_] (Ping timeout: 246 seconds) |
03:50:56 | | Join [Saint] [0] (~Saint]@unaffiliated/saint/x-8516940) |
03:57:15 | | Quit DerPapst (Quit: Leaving.) |
03:57:52 | | Quit dfkt (Quit: -= SysReset 2.55=- Sic gorgiamus allos subjectatos nunc.) |
03:59:08 | | Quit advcomp2019 (Quit: There are two major products that come out of Berkeley: LSD and UNIX. We don't believe this to be a coincidence.) |
04:00 |
04:03:12 | | Quit TheSeven (Disconnected by services) |
04:03:38 | | Join [7] [0] (~TheSeven@rockbox/developer/TheSeven) |
04:07:37 | | Quit ze (Ping timeout: 272 seconds) |
04:09:40 | | Quit anewuser_ (Read error: Connection reset by peer) |
04:13:10 | | Join advcomp2019 [0] (~advcomp20@unaffiliated/advcomp2019) |
04:19:58 | | Join amiconn_ [0] (amiconn@rockbox/developer/amiconn) |
04:19:58 | | Quit amiconn (Disconnected by services) |
04:20:12 | | Quit pixelma (Disconnected by services) |
04:20:14 | | Join pixelma_ [0] (pixelma@rockbox/staff/pixelma) |
04:20:16 | | Nick pixelma_ is now known as pixelma (pixelma@rockbox/staff/pixelma) |
04:20:20 | | Nick amiconn_ is now known as amiconn (amiconn@rockbox/developer/amiconn) |
04:25:33 | | Quit curtism (Quit: Live Long and Prosper) |
04:28:58 | | Join dys` [0] (~andreas@krlh-4d02b303.pool.mediaWays.net) |
04:30:55 | | Quit dys (Ping timeout: 265 seconds) |
04:59:06 | | Quit [7] (Disconnected by services) |
04:59:14 | | Join TheSeven [0] (~TheSeven@rockbox/developer/TheSeven) |
05:00 |
05:19:32 | | Quit passstab (Quit: Leaving) |
05:26:18 | | Quit [Saint] (Ping timeout: 246 seconds) |
05:26:46 | | Join [Saint] [0] (~Saint]@unaffiliated/saint/x-8516940) |
05:32:58 | *** | Saving seen data "./dancer.seen" |
05:39:12 | | Join Rob2223 [0] (~Miranda@p4FFD06D2.dip.t-dialin.net) |
05:43:15 | | Quit Rob2222 (Ping timeout: 272 seconds) |
06:00 |
06:41:53 | | Join ender` [0] (~ender@foo.eternallybored.org) |
06:41:53 | | Quit freqmod (Remote host closed the connection) |
06:55:46 | | Quit Rower85 (Read error: Connection reset by peer) |
06:59:18 | | Quit [Saint] (Ping timeout: 276 seconds) |
07:00 |
07:00:41 | | Join tchan [0] (~tchan@lunar-linux/developer/tchan) |
07:04:11 | | Join hiptobecubic [0] (~john@ip4da83ac6.direct-adsl.nl) |
07:04:11 | | Quit hiptobecubic (Changing host) |
07:04:11 | | Join hiptobecubic [0] (~john@unaffiliated/hiptobecubic) |
07:05:14 | pixelma | KiwiCam: real *1-bit* bmps should be "transparent" for one colour and opaque with the other, maybe your problem there are the overlapping viewports though as already been stated. |
07:07:16 | pixelma | the first part refers to monochrome displays, on colour displays one "colour" of the 1-bit bmp will get the foreground colour and the other the background colour. That's basically how the non-antialiased fonts work |
07:08:43 | | Join Rower85 [0] (husvagn@v-413-alfarv-90.bitnet.nu) |
07:10:54 | | Quit adnap (Ping timeout: 252 seconds) |
07:14:05 | | Quit Rower85 (Read error: Connection reset by peer) |
07:15:06 | | Join [Saint] [0] (~Saint]@101.98.158.103) |
07:15:06 | | Quit [Saint] (Changing host) |
07:15:06 | | Join [Saint] [0] (~Saint]@unaffiliated/saint/x-8516940) |
07:18:44 | | Join adnap [0] (~adnap@rrcs-71-42-140-57.sw.biz.rr.com) |
07:25:21 | [Saint] | There wouldn't be an abundence of either 480x800 or 320x480 RaaA device owners hanging around would there? |
07:27:43 | [Saint] | Just one of either, or both, would be cool too ;) |
07:29:31 | | Join mortalis [0] (~mortalis@77.108.98.176) |
07:30:20 | JdGordon | [Saint]: so, what will you bribe me with to add a "draw rectangle with gradient" tag? |
07:31:10 | | Join kadoban [0] (~kadoban@ip98-165-177-158.ph.ph.cox.net) |
07:31:47 | [Saint] | How open are the terms for submissions? ;) |
07:32:59 | *** | Saving seen data "./dancer.seen" |
07:36:15 | | Quit Scromple (Read error: Connection reset by peer) |
07:43:03 | | Join Rower85 [0] (husvagn@v-413-alfarv-90.bitnet.nu) |
07:44:56 | | Join [Saint_] [0] (~Saint]@unaffiliated/saint/x-8516940) |
07:46:07 | | Join [Saint__] [0] (~Saint]@unaffiliated/saint/x-8516940) |
07:47:56 | | Join Scromple [0] (~Simon@119.225.209.134) |
07:49:06 | | Quit [Saint_] (Ping timeout: 244 seconds) |
07:51:51 | | Join randumbum [0] (~IRCclient@173-8-222-138-Oregon.hfc.comcastbusiness.net) |
07:56:21 | KiwiCam | Thanks Guys. So, Top is Above and Bottom is Below. Viewports. I don't know what sort of layering gevaerts wants. I wasn't thinking about layering until I saw it with my conversation above. Layering would be good. Two Veiwports, One Below, has Bitmap that I want to be in the background (below). Not the actual background, but I.... |
07:58:05 | mortalis | JdGordon: I found that in your g120 you forgot * in some places. Value lcd_framebuffer[y][x] was replaced by pointer to the value FBADDR(x,y). I noticed this bug in lcd-hifiman.c lcd-rk27generic.c lcd-iaudio7.c, but I didn't check everything. |
07:58:07 | fs-bluebot | Gerrit review #120 at http://gerrit.rockbox.org/r/#change,120 : lcd drivers: Convert lcd_[remote_]framebuffer to a pointer by Jonathan Gordon (changes/20/120/10) |
07:59:53 | KiwiCam | [Saint]: would like the actual Background to be able to be changed. Subimages. A Starfield for instance. Moving Neon Pipes. Who knows? As for my two Viewports, I want to be able to move an object past another so they merge, rather than having a Black square outline as per now. No Transparency for the background of the Above Bitmap. I feel stupid :) |
08:00 |
08:01:02 | KiwiCam | I was also hoping that I could use the %ss for moving viewports (change hte x and y values based on time %cS or %cM say.) |
08:01:14 | | Quit CaptainKewl (Ping timeout: 260 seconds) |
08:01:34 | KiwiCam | I'd like to move progress bars and sliders around. |
08:03:05 | KiwiCam | pixelma: Yes. It's the overlapping viewports. Now that layering is an option. I'd like that as that's what I'm looking for. |
08:05:09 | KiwiCam | Which one is the link that can tell me more above this layers thing you're doing? Would I find info in the latest manual? |
08:12:43 | | Join [Saint_] [0] (~Saint]@unaffiliated/saint/x-8516940) |
08:14:25 | | Quit [Saint__] (Ping timeout: 244 seconds) |
08:14:40 | | Quit Scromple (Quit: Leaving) |
08:27:34 | | Join Zagor [242] (~bjst@rockbox/developer/Zagor) |
08:27:42 | | Quit amiconn (Remote host closed the connection) |
08:27:42 | | Quit pixelma (Remote host closed the connection) |
08:28:45 | | Join amiconn [0] (amiconn@rockbox/developer/amiconn) |
08:28:47 | | Join pixelma [0] (pixelma@rockbox/staff/pixelma) |
08:33:15 | JdGordon | mortalis: hmm, damn, can you push a fix either to git or gerrit? |
08:34:44 | mortalis | JdGordon: sure |
08:36:21 | | Quit randumbum (Ping timeout: 246 seconds) |
08:37:35 | | Quit [Saint_] (Remote host closed the connection) |
08:38:33 | | Quit factor (Read error: Connection reset by peer) |
08:38:42 | | Join randumbum [0] (~IRCclient@173-8-222-138-Oregon.hfc.comcastbusiness.net) |
08:40:12 | | Join factor [0] (~factor@r74-195-184-248.msk1cmtc01.mskgok.ok.dh.suddenlink.net) |
08:47:35 | CIA-44 | Commit fbb8c31 in rockbox by Andrew Ryabinin: Fix bugs in lcd drivers after b37e6bc |
08:49:50 | CIA-44 | fbb8c31 build result: All green |
08:54:51 | JdGordon | mortalis: are those not targets not being bult? |
09:00 |
09:00:03 | mortalis | JdGordon: yes |
09:00:16 | JdGordon | ah, thats how they got missed :) |
09:03:58 | | Join nick_p [0] (~nick@82-69-105-120.dsl.in-addr.zen.co.uk) |
09:05:46 | | Join pamaury [0] (~quassel@rockbox/developer/pamaury) |
09:11:55 | | Join wodz [0] (~wodz@89-76-160-35.dynamic.chello.pl) |
09:12:43 | wodz | mortalis: wanna try dualboot code? |
09:12:54 | mortalis | sure |
09:14:23 | wodz | mortalis: http://www.pastie.org/3486449 |
09:14:43 | wodz | this should build rkw directly - no need for rkwpatch |
09:15:24 | wodz | it looks for rockbox.rkw and BASE.RKW in .rockbox dir on SD |
09:15:53 | wodz | in order to boot OF you should hold any other button in addition to ON/OFF |
09:16:12 | wodz | it should also boot OF on USB insert when player is OFF |
09:16:34 | wodz | thats theory, now lets see how this goes in practice :-) |
09:17:57 | wodz | One thing missing is that this changes break rolo for now. |
09:18:13 | | Join nosa [0] (~m00k@adsl-74-235-42-78.clt.bellsouth.net) |
09:20:13 | | Quit nosa (Read error: Connection reset by peer) |
09:20:45 | | Quit nosa-j (Read error: Connection reset by peer) |
09:24:56 | | Quit gartral (Quit: Lost terminal) |
09:25:37 | wodz | funman: with current build (fbb8c31) nano2g doesn't show up on ubuntu 11.10 http://www.pastie.org/3486486 and hangs on usb disconnect (reacts only to hard reset) |
09:28:13 | | Join nosa-j [0] (~m00k@adsl-74-235-42-78.clt.bellsouth.net) |
09:30:44 | mortalis | wodz: in any case OF loaded |
09:31:20 | mortalis | it write loading rockbox.rkw but it loads OF |
09:32:25 | wodz | I'll look at this in a moment, now I am testing nano2g usb for funman |
09:33:00 | *** | Saving seen data "./dancer.seen" |
09:33:55 | wodz | mortalis: does it boot if you plug usb into powered off device? |
09:34:01 | mortalis | yes |
09:35:26 | mortalis | and i think we should call power_init in bootloader, if not I need hold power button too long |
09:35:34 | | Quit [Saint] (Quit: Quit) |
09:35:54 | | Join LinusN [0] (~linus@giant.haxx.se) |
09:36:43 | | Join nosa [0] (~m00k@adsl-74-235-42-18.clt.bellsouth.net) |
09:36:50 | | Quit nosa-j (Ping timeout: 252 seconds) |
09:36:51 | | Nick nosa is now known as nosa-j (~m00k@adsl-74-235-42-18.clt.bellsouth.net) |
09:37:18 | wodz | mortalis: could you add printf() to see what button does it read? |
09:38:00 | | Join Keripo [0] (~Keripo@eng096.wireless-resnet.upenn.edu) |
09:39:07 | mortalis | wodz: yes, but I think it reads, because when I push only power button it says "loading rockbox.rkw", if power + some other it says "loading BASE.RKW" |
09:40:17 | wodz | funman: This what I get with the build you linked yesterday (http://www.rockbox.org/irc/log-20120228#20:33:48): http://www.pastie.org/3486564 |
09:40:35 | wodz | It also hangs on usb disconnect |
09:41:06 | wodz | So to summarize we have pretty much fucked up usb on stable target just before release. |
09:43:30 | mortalis | wodz: stupid me, rockbox.rkw on my sd it's renamed BASE.RKW. I forgot to replace it |
09:44:16 | wodz | mortalis: I was scratching my head how it is possible to always boot OF :-) |
09:44:27 | | Join freqmod [0] (~quassel@2001:700:300:1430:226:18ff:fe82:1a24) |
09:44:46 | wodz | mortalis: so basicaly dualbooting works, right? |
09:45:16 | mortalis | wodz: yes |
09:46:38 | wodz | mortalis: could you test if removing cache disable code from the begining of crt0.S changes something? IMO it is redundant |
09:49:16 | wodz | mortalis: Another thing to test would be to put something else instead of 0 in line 118,119 in tools/rkw.c (in two reserved fields). And see if nand bootloader likes such abused RKW |
09:50:21 | wodz | mortalis: How long does it take to boot? Is it considerably longer than usual? |
09:51:06 | wodz | I guess I need to dig up my nand bootloader dissasm and look for rkw loading routine |
09:52:09 | mortalis | wodz: approximately 1 sec longer |
09:53:23 | wodz | not that bad |
09:53:44 | | Quit factor (Ping timeout: 244 seconds) |
09:53:49 | | Join lebellium_gs2 [0] (~lebellium@tmo-111-186.customers.d1-online.com) |
09:54:31 | | Join factor [0] (~factor@r74-195-184-248.msk1cmtc01.mskgok.ok.dh.suddenlink.net) |
09:58:24 | lebellium_gs2 | Hello JdGordon. I was thinking: could your new skin engine improvement allow me to display the menu viewport fullscreen when mini-player is off in my theme? :D I mean if I put the usual background pic in background + put the pic to hide the mini-player on the backdrop layer |
09:59:15 | | Join antil33t| [0] (~Ahurhurr@101.98.150.103) |
10:00 |
10:00:40 | mortalis | wodz: how do you disable cache? http://pastebin.com/0ZdDEdWa seems pointless to me |
10:01:22 | wodz | mortalis: thats the code it effectively clears half of the register |
10:01:30 | mortalis | ah |
10:01:34 | mortalis | understand |
10:01:57 | wodz | it is stolen from OF dissasm |
10:02:46 | wodz | haha, nand bootloader from my rk27xx looks for RKLDFUCKORD.RKW file first |
10:02:55 | | Quit antil33t (Ping timeout: 272 seconds) |
10:03:10 | | Join [Saint] [0] (~Saint]@101.98.158.103) |
10:03:10 | | Quit [Saint] (Changing host) |
10:03:10 | | Join [Saint] [0] (~Saint]@unaffiliated/saint/x-8516940) |
10:03:53 | mortalis | lol |
10:04:41 | mortalis | wodz: cache disabled two times in crt0. should I remove both? |
10:05:07 | wodz | no, only the first one in .bootloader_reloc section |
10:11:28 | | Join antil33t [0] (~Ahurhurr@101.98.150.103) |
10:12:00 | JdGordon | lebellium_gs2: yes, you dont remember that that was why i did it originally? |
10:14:04 | mortalis | wodz: it doesn't work with removed cache disable |
10:14:21 | | Quit antil33t| (Ping timeout: 246 seconds) |
10:15:24 | wodz | mortalis: ok thanks for testing |
10:17:40 | | Quit kumy (Remote host closed the connection) |
10:19:26 | mortalis | wodz: another values in rkw header works |
10:19:57 | lebellium_gs2 | JdGordon: that was just to make sure ;) so as soon as I get a little free time I'll include this major improvement in my theme and let you know. Thank you so much :) |
10:21:10 | wodz | mortalis: good - I'll polish this a bit, add support for rkw in rolo and I'll push it |
10:25:42 | wodz | mortalis: according to my dissasm, rkw loaded in nand doesn't check this reserved fields so at least theory matches practice |
10:31:16 | | Quit hiptobecubic (Ping timeout: 245 seconds) |
10:31:21 | | Join antil33t| [0] (~Ahurhurr@101.98.150.103) |
10:31:52 | | Join zaphee [0] (~user@ede67-2-82-232-36-5.fbx.proxad.net) |
10:32:11 | | Part zaphee |
10:34:02 | | Quit antil33t (Ping timeout: 244 seconds) |
10:41:46 | | Quit Keripo (Quit: Leaving.) |
10:45:20 | JdGordon | how will builds cope with LANG string going from *: none to a used string again? |
10:45:35 | pamaury | bluebrother^: about the bootloader problem, could we just extended scramble to work with multiple files when using -add. So one could write scramble -add=X <file1> -add=Y <file2>, this is simpler than using a zip file |
10:46:19 | wodz | pamaury: scramble in its current form is a big mess |
10:46:23 | | Quit lebellium_gs2 (Ping timeout: 245 seconds) |
10:46:55 | wodz | pamaury: and warn me before you touches it as I am going to add rkw support to it |
10:47:36 | pamaury | my change would be orthogonal: just enhance scramble to be able to pack several files when using the -add mode. That's just an idea |
10:51:10 | pamaury | in any case, I don't see how we could build several files for the bootloader and pack them without changing the makefiles |
10:53:05 | gevaerts | JdGordon: probably a stupid question, but can %VB be used inside a conditional? |
10:55:02 | [Saint] | What would you want it in a conditional for? if(foo)<draw in background|dont> kinda thing, or...? |
10:55:55 | gevaerts | basically. I dpn' |
10:55:59 | JdGordon | gevaerts: no, its parse time, not run time |
10:56:06 | gevaerts | I don't know right now if that even makes sense though |
10:56:09 | gevaerts | ok |
10:59:31 | wodz | mortalis: ping |
10:59:41 | mortalis | wodz: pong |
11:00 |
11:00:36 | wodz | mortalis: could you change value in line 117 of tools/rkw.c to 0x60000000 and check if this doesn't affect loading of bootloader? |
11:01:22 | wodz | mortalis: nand bootloader does some checks for values readed from address pointed by this field and putting random address doesn't seem like a good idea |
11:08:59 | * | pamaury thinks he has a solution to the mk*boot dualboot problem |
11:09:34 | pamaury | We could just move the dualboot codes to scamble ! |
11:09:42 | mortalis | wodz: it doesn't work |
11:10:18 | wodz | f**k |
11:13:50 | mortalis | wodz: and one important notice: When I boot OF with bootloader it doesn't update firmware if I put it on disk |
11:14:28 | | Join DerPapst [0] (~Adium@91-64-115-27-dynip.superkabel.de) |
11:15:36 | wodz | mortalis: does it expose hidden partition when magic file is present? |
11:25:16 | | Join merbanan [0] (~benjamin@2a00:801:102:104:221:70ff:fecb:d6f9) |
11:28:42 | mortalis | wodz: oh, it was sd card, not internal flash :-). Update works |
11:29:20 | JdGordon | any particular reason lcd_gradient_rect() is not exported to apps/? |
11:31:33 | * | JdGordon curses at whoever decided it would be smart to call it _rect() but pass it line count not pixel height |
11:31:44 | | Join hiptobecubic [0] (~john@unaffiliated/hiptobecubic) |
11:32:41 | wodz | mortalis: various comments in SDK suggest that the address in 0x14 offset of RKW header points to 5 words long struct which contains 1.flags 2.loadversion 3.resetcount 4.5 reserved. In dissasm of nand bootloader from my device I see only it check 1 field |
11:33:01 | *** | Saving seen data "./dancer.seen" |
11:33:14 | wodz | and compares with 0x3df479a |
11:35:44 | | Quit randumbum (Ping timeout: 260 seconds) |
11:38:38 | | Join randumbum [0] (~IRCclient@173-8-222-138-Oregon.hfc.comcastbusiness.net) |
11:49:30 | | Quit hiptobecubic (Read error: Operation timed out) |
11:52:53 | | Quit nick_p (Quit: Leaving) |
11:53:59 | | Quit kadoban (Ping timeout: 260 seconds) |
12:00 |
12:13:43 | pamaury | I think there is a makefile problem with ucl, running make in tools/ gives a compile error about a missing ucl/ucl.h |
12:19:46 | | Join antil33t [0] (~Ahurhurr@101.98.150.103) |
12:20:21 | | Join hiptobecubic [0] (~john@unaffiliated/hiptobecubic) |
12:21:03 | | Quit sinthetek (Ping timeout: 276 seconds) |
12:22:28 | | Join anewuser [0] (~anewuser@190.207.129.184) |
12:22:39 | | Quit anewuser (Changing host) |
12:22:39 | | Join anewuser [0] (~anewuser@unaffiliated/anewuser) |
12:22:53 | | Quit antil33t| (Ping timeout: 272 seconds) |
12:24:22 | wodz | pamaury: you know that tools are usually build from tools.make, right? |
12:30:37 | CIA-44 | Commit f0ecce9 in rockbox by Frank Gevaerts: Fix ucl-related makefiles |
12:30:43 | gevaerts | pamaury: that should help |
12:33:16 | CIA-44 | f0ecce9 build result: All green |
12:33:53 | wodz | We have two crc implementation which differ in polinomial. I am going to add third for RKW. Maybe it would be wise to pass polinomial as argumen and use unified function? |
12:34:10 | | Nick dys` is now known as dys (~andreas@krlh-4d02b303.pool.mediaWays.net) |
12:34:45 | | Join antil33t| [0] (~Ahurhurr@101.98.150.103) |
12:36:24 | | Quit antil33t (Ping timeout: 260 seconds) |
12:39:22 | | Join sinthetek [0] (~sinthetek@unaffiliated/sinthetek) |
12:40:14 | | Join antil33t [0] (~Ahurhurr@101.98.150.103) |
12:42:28 | pamaury | wodz: yes |
12:42:52 | | Quit antil33t| (Ping timeout: 255 seconds) |
12:43:01 | pamaury | I'm wondering if I should add this to scramble or add another tool. scramble is already complicated enough |
12:44:37 | wodz | pamaury: no, scramble is hacky |
12:46:05 | pamaury | but without scramble I will need to add a step after running $boottool, so I will need to modify configure and the makefiles |
12:47:08 | | Join antil33t| [0] (~Ahurhurr@101.98.150.103) |
12:48:07 | | Quit antil33t (Ping timeout: 240 seconds) |
12:51:21 | | Join T44 [0] (~Topy44@f048162088.adsl.alicedsl.de) |
12:51:43 | | Quit antil33t| (Ping timeout: 260 seconds) |
12:52:27 | | Join antil33t [0] (~Ahurhurr@101.98.150.103) |
12:53:21 | | Quit merbanan (Read error: Connection reset by peer) |
12:53:36 | pamaury | gevaerts: thanks for fixing ucl ! |
12:55:30 | | Quit Topy (Ping timeout: 276 seconds) |
12:55:52 | * | JdGordon thinks he is stuffing up something in the maths with the gradient code |
12:55:59 | JdGordon | anyone awake that can help? |
12:57:40 | pamaury | JdGordon: helping with what exactly ? |
12:59:07 | | Join pow4h [0] (~asdf@adsl-74-160-161-156.asm.bellsouth.net) |
12:59:45 | pow4h | Is it possible for a plugin to call another plugin? Like if i have plugin-a.c. How can I tell it to call plugin-b.c? |
13:00 |
13:00:56 | JdGordon | pamaury: http://pastebin.com/0mJEvC0s is my diff, im trying to split out the gradient rectangle fill part to be generic while keeping the text drawn the same way (which is per line) |
13:01:16 | JdGordon | th eproblem is when num_lines > 1 the colours on the second line are wrong |
13:01:33 | JdGordon | if you apply it, use cabbie and go into the "show id3 infor" screen you can see how its broken |
13:01:39 | JdGordon | pow4h: no |
13:01:45 | gevaerts | pow4h: You mean launching another plugin, or calling a function in another file? |
13:05:55 | Torne | launching another plugin would be possible if you didn't care about it returning to the original plugin or retaining any of the original plugin's state, though i don't know that we have a function for htat in the plugin api right now |
13:06:06 | pamaury | JdGordon: the gradient is supposed to be over all lines or is it repeated for each line ? |
13:06:53 | | Join evilnick [0] (~evilnick@5e007ed8.bb.sky.com) |
13:07:34 | | Join antil33t| [0] (~Ahurhurr@101.98.150.103) |
13:09:46 | JdGordon | pamaury: the gradient is done over some number of text lines, the cur_line and num_lines are text lines, h is pixel lines |
13:10:24 | JdGordon | if i switch the second part (from line 81) to a loop to add the *step to the value it works |
13:10:29 | | Quit antil33t (Ping timeout: 252 seconds) |
13:10:32 | JdGordon | so that mul is apparently wrong |
13:13:59 | | Part Zagor |
13:15:29 | pamaury | hum, I will need to test it, looking at the diff is not enough. Let me eat before and then I can help you :) |
13:18:19 | | Join Zagor [242] (~bjst@rockbox/developer/Zagor) |
13:19:15 | JdGordon | awesome :) |
13:25:52 | pamaury | JdGordon: what is show id3 infor ? I need to start to play a file ? |
13:27:00 | pamaury | hum, I thought we had some vector image for the fuze+ ?! |
13:27:11 | pamaury | damn, this simulator never closes correctly |
13:28:27 | JdGordon | pamaury: yeah, context menu in the wps |
13:28:31 | JdGordon | show track info |
13:29:09 | pamaury | hum, the fuze+ simulator really need some work: the keys are not written so I don't remember the one to use and it does close correctly |
13:29:26 | JdGordon | ok, ill give you a hack so it shows in the main menu? |
13:29:37 | pamaury | no, let me try and suceed :) |
13:30:05 | pamaury | ok, I have the track info but no gradient at all |
13:30:32 | pamaury | oh wait, is the gradient for the highlight ? |
13:30:43 | JdGordon | yes |
13:30:54 | JdGordon | apps/menu.c:207, change the 1 to 2 |
13:31:53 | pamaury | what will that do ? |
13:32:23 | pamaury | ah ! indeed your patch is quite broken :) |
13:33:04 | *** | Saving seen data "./dancer.seen" |
13:33:40 | JdGordon | yeah :) i dont see why |
13:34:35 | pamaury | i don't understand what is num_lines and cur_line, it doesn't seem to change when changing the highlighted line |
13:35:13 | JdGordon | num_lines as passed in is the amount of list lines the gradient spans over |
13:35:32 | JdGordon | cur_line is the line being drawn |
13:35:43 | JdGordon | both a * by the line height to get th epixel |
13:35:54 | pamaury | is the root_menu |
13:35:58 | | Quit sinthetek (Ping timeout: 245 seconds) |
13:36:06 | pamaury | in the root menu I always get num_lines=30 and cur_line=0/15 |
13:36:32 | JdGordon | yes, thats what changing the 1 to 2 did in menu.c |
13:36:39 | JdGordon | is the number of lines to span |
13:36:49 | JdGordon | line height is 15, so 2*15 |
13:37:08 | JdGordon | part of the reason im doing this is so we can draw gradients with a sane api :) |
13:37:33 | pamaury | ok, so the problem is that the gradient is not actually linear, it's like a->b->a |
13:38:17 | JdGordon | BAH, yes it was a stupid mistake |
13:38:19 | JdGordon | i figured it out :) |
13:38:37 | JdGordon | cur_line should be h in that last block |
13:39:19 | JdGordon | thanks anyway |
13:39:53 | pamaury | you're welcome |
13:42:03 | gevaerts | JdGordon: could you have another look at g#113? Your suggestion of using DYNAMIC doesn't fix it, and without it skinned lists very often don't work at all (i.e. they fall back to the regular list) |
13:42:58 | JdGordon | not without fs-bluebot linking me :) |
13:43:05 | JdGordon | i mean, one min |
13:43:15 | gevaerts | http://gerrit.rockbox.org/113 |
13:43:16 | gevaerts | :) |
13:43:37 | | Join sinthetek [0] (~sinthetek@unaffiliated/sinthetek) |
13:43:51 | gevaerts | I fully agree that that isn't right, but I don't know which solution is both correct *and* working :) |
13:44:41 | gevaerts | Well, apart from the basic suspicion that skinlist_set_cfg() should be called from all places that are symmetric with the places that call it with NULL |
13:49:26 | gevaerts | JdGordon: with the medieval theme the skinned list goes away when coming back from some "errors" like trying to resume playback without a playlist. With the wavy theme, skinned lists *never* work |
13:50:12 | JdGordon | gevaerts: hang on, did you change both the tag_table and skin_render.c? |
13:50:36 | JdGordon | do_refresh should always be true if tag_table.c was changed though... |
13:51:19 | gevaerts | You mean the one that doesn't work? Only SKIN_REFRESH_DYNAMIC in tag_table.c |
13:52:18 | gevaerts | hm, right. Something must be wrong |
13:52:20 | * | gevaerts retests |
13:55:12 | gevaerts | JdGordon: no, it really doesn't work with just changing SKIN_TOKEN_LIST_ITEM_CFG to SKIN_REFRESH_DYNAMIC in tag_table.c |
13:57:37 | | Quit sinthetek (Ping timeout: 240 seconds) |
14:00 |
14:00:11 | | Join funman [0] (~fun@rockbox/developer/funman) |
14:04:01 | JdGordon | gevaerts: ok, change both |
14:04:14 | JdGordon | at least them there is no surprise when we see it happening too often |
14:04:40 | JdGordon | what tag shoudl i use for "fill a rectangle" tag? |
14:06:31 | JdGordon | I suppose I should just chuck it in with the viewport tags |
14:06:36 | JdGordon | %Vr? |
14:07:27 | funman | [Saint]: ping |
14:08:14 | gevaerts | JdGordon: I've pushed the "both" option to gerrit now. Can you do a quick review to make sure I didn't misunderstand? |
14:08:38 | funman | wodz: thanks, it seems to work on some nano2g only |
14:09:13 | JdGordon | gevaerts: yeah, thats cool, and remove the trailing space in skin_render.c please :) |
14:09:20 | JdGordon | [Saint]: ping2! :) |
14:11:03 | CIA-44 | Commit 8efb8f9 in rockbox by Jonathan Gordon: lcd drivers: split lcd_gradient_rect so it is actually useful |
14:11:05 | * | JdGordon will do the skin part of ^ tomorrow |
14:11:34 | CIA-44 | Commit b9bf6cd in rockbox by Frank Gevaerts: Call skinlist_set_cfg() unconditionally, i.e. also if do_refresh==false |
14:14:29 | CIA-44 | 8efb8f9 build result: All green |
14:16:05 | Zagor | do we want a cia notice when a patch is uploaded to gerrit? |
14:16:19 | Torne | i'd say no |
14:16:41 | Torne | on the grounds that we're trying to get people not to assume that uploading a patch is sufficient to make anyone care about it :) |
14:16:56 | funman | it could increase awareness so why not |
14:17:49 | | Quit enthdegree (Read error: Connection reset by peer) |
14:17:56 | | Join Rob2222 [0] (~Miranda@p4FFD0DB8.dip.t-dialin.net) |
14:17:58 | CIA-44 | b9bf6cd build result: All green |
14:19:24 | | Quit Rob2223 (Ping timeout: 272 seconds) |
14:30:18 | | Join sinthetek [0] (~sinthetek@unaffiliated/sinthetek) |
14:30:24 | | Join mitakas [0] (~mitakas@212.84.240.163) |
14:32:33 | | Part mitakas |
14:33:03 | | Join verbtim [0] (~verbtim@212.84.240.163) |
14:40:23 | JdGordon | [Saint]: if oyu read the logs.. how do you want the skin tag to fill a rectangle to work? initially I thought %xx(x,y,width,height,[start colour, end colour]) where not putting end colour will make is a solid colour, and not having any colour would just draw the outline? |
14:43:05 | funman | JdGordon: i think he doesnt |
14:46:42 | pamaury | do we restrict to sh scripts or can tools/ script can use bash functionality ? |
14:47:19 | funman | which functionality do you need ? |
14:48:20 | wodz | funman: what is the plan with nano2g usb? |
14:48:52 | pamaury | funman: associative array, it's not strictly necessary, just simplifies a bit |
14:48:57 | funman | wodz: not sure, didn't current build work for you yesterday or was it an older build ? |
14:52:57 | wodz | funman: I think the build from ~week ago worked for me BUT I am not so sure now I used usb in rb (and not disk mode) and it was on win7 laptop form work. |
14:53:32 | wodz | on my linux machine the thing clearly refuses to work in rb |
14:55:02 | funman | except reverting usb-s3c6400x.c changes one by one i don't know |
14:57:34 | wodz | is it possible to go back to the state before your changes or it is somehow interconnected with amsv2 usb work? |
15:00 |
15:02:37 | kugel | pamaury: how about perl or something? |
15:02:44 | kugel | I dont think we have any restrictions there |
15:05:14 | funman | wodz: i made a bunch of changes to nano2g code only, merging amsv2 code is only one or 2 commits |
15:09:31 | Zagor | we're in Wired: http://www.wired.com/geekdad/2012/02/rockbox-open-source-media-player-firmware/ |
15:16:39 | | Join incoganon [0] (~quassel@2001:700:300:1470:5e26:aff:fe17:2ae7) |
15:17:58 | | Quit verbtim (Quit: Leaving) |
15:29:38 | | Quit evilnick (Quit: Leaving) |
15:31:16 | | Join WalkGood [0] (~4@unaffiliated/walkgood) |
15:31:38 | pamaury | I'm wondering what is the best option for the dualboot problem: use a known format like zip (which will require to modify the mk*boot to unzip) or just use an ad hoc trivial format. The latter having the advantage that people will be less likely to fiddle with it and is simpler to implement ? (which is important for a bootloader) |
15:33:06 | *** | Saving seen data "./dancer.seen" |
15:34:41 | funman | zip doesn't make it much harder to fiddle with |
15:34:58 | | Quit randumbum (Quit: ChatZilla 0.9.88 [Firefox 10.0.2/20120216101208]) |
15:35:40 | pamaury | that's the contrary: zip is too easy, if you use some other format it's harder :) |
15:39:05 | funman | ah right i misread |
15:40:09 | pow4h | how do I find which file read() is defined in ? int read(int fildes, void *buf, size_t nbyte); |
15:40:35 | funman | git grep ssize_t'.*'read |
15:41:04 | pamaury | funman: what do you think is best: zip ? ad hoc format ? other ? |
15:42:14 | funman | the best is embed in binary and make new releases of said binary when upgrade is needed |
15:42:46 | pamaury | I'm talking about the format of the binary containing the dualboot code + bootloader |
15:42:55 | pow4h | funman is that git command going to make me reinstall rockbox or delete any files or anything? |
15:43:06 | pamaury | you prefer the currently option of embedding it in rbutil ? |
15:43:13 | funman | pamaury: yep |
15:43:25 | funman | pow4h: i suggest reading the man page for it |
15:44:24 | | Join y4n [0] (y4n@unaffiliated/y4ndexx) |
15:44:48 | pamaury | this embedding is problematic, that requires a new release of rbutil even though it's completely unrelated |
15:45:13 | funman | yeah but it also avoids the problem you are talking about |
15:45:36 | funman | and make things simple for the devs (except the need for new releases of course which is annoying) |
15:46:22 | funman | which other targets need to patch OF ? |
15:46:29 | pamaury | fuze+ |
15:46:40 | pamaury | once it's written, that will be dead simple for devs too |
15:46:40 | funman | i mean except fuze+ and ams |
15:46:46 | pamaury | none |
15:47:46 | | Join Thra11 [0] (~thrall@84.93.159.186) |
15:47:56 | funman | other solution would be implement a reboot to USB mode in dualboot code which rockbox bootloader uses |
15:48:39 | funman | reboot to OF i mean |
15:49:07 | pamaury | I don't understand, how is that related to the embedding of dualboot code ? |
15:49:12 | | Join TheLemonMan [0] (~LemonBoy@ppp-90-25.26-151.libero.it) |
15:49:22 | funman | dualboot code is less likely to change? |
15:49:58 | pamaury | it nevertheless changes, as seen recently |
15:50:14 | funman | fuze+ is not stable yet though no? |
15:51:05 | pamaury | no, but rbutil support is implemented and having the possibility to change the bootloader/dualboot code indepently of rbutil is nice |
15:51:16 | pamaury | *independently |
15:51:42 | pow4h | funman thanks. the git grep returned 30 results. how do i know which is the definition? |
15:52:52 | funman | hmm hard question, try reading the files maybe? |
15:54:12 | pamaury | pow4h: git grep -e 'ssize_t[[:space:]]*read(' |
15:55:14 | | Quit sinthetek (Ping timeout: 252 seconds) |
15:55:17 | pamaury | for the plugins, read is implemented as a macro redirecting to some function (see plugin.h I think) |
15:55:31 | pamaury | (and plugin.c) |
16:00 |
16:03:19 | pow4h | how do I include a file? I put a header.h into the same folder as my .c file and did include header.h, but it didnt work |
16:03:33 | pow4h | #include header.h |
16:03:45 | pow4h | #include "header.h" |
16:05:00 | Torne | what was the actual error? |
16:05:13 | pamaury | pow4h: short answer: you can't, put everything in your .c If you want to use several files for your plugin, you will need to create a subdir like some other plugins and add one line to SUBDIRS file |
16:05:30 | Torne | oh, yeah, i guess . is not in the include path? |
16:06:34 | pamaury | apps/plugins/ is not in the include path I think |
16:12:30 | | Quit WalkGood () |
16:24:40 | | Join sinthetek [0] (~sinthetek@unaffiliated/sinthetek) |
16:27:24 | | Join dfkt [0] (dfkt@chello084112032026.1.11.vie.surfer.at) |
16:27:33 | wodz | mi4 crc routine is a bit weird as polynomial used there is even and all valid crc polinomials are odd. It is reflected most probably but this is not stated in comments |
16:27:34 | | Quit dfkt (Changing host) |
16:27:34 | | Join dfkt [0] (dfkt@unaffiliated/dfkt) |
16:37:21 | | Quit Zarggg (Read error: Connection reset by peer) |
16:38:08 | | Join Zarggg [0] (~zarggg@24.229.141.85.res-cmts.sm.ptd.net) |
16:41:13 | | Join passstab [0] (~v@c-68-80-37-73.hsd1.pa.comcast.net) |
16:49:47 | | Quit incoganon (Remote host closed the connection) |
16:51:38 | | Join perrikwp_ [0] (~quassel@cpe-024-163-024-033.triad.res.rr.com) |
16:54:16 | | Quit perrikwp (Ping timeout: 265 seconds) |
16:54:38 | | Join Strife89 [0] (~Strife89@207.144.201.128) |
16:56:20 | | Part Zagor |
17:00 |
17:05:25 | | Part LinusN |
17:10:25 | | Quit DerPapst (Quit: Leaving.) |
17:18:39 | | Join WalkGood [0] (~4@unaffiliated/walkgood) |
17:19:47 | | Join kadoban [0] (~kadoban@ip98-165-177-158.ph.ph.cox.net) |
17:26:16 | | Quit crwl (Read error: Connection reset by peer) |
17:27:18 | | Quit [Saint] (Read error: Connection reset by peer) |
17:33:10 | *** | Saving seen data "./dancer.seen" |
17:33:15 | | Join GodEater_ [0] (93722cc8@rockbox/staff/GodEater) |
17:33:54 | | Quit GeekShadow (Ping timeout: 265 seconds) |
17:33:57 | | Join benedikt93 [0] (~benedikt9@unaffiliated/benedikt93) |
17:41:42 | | Join crwl [0] (~crwlll@dsl-jklbrasgw1-ffb9c300-103.dhcp.inet.fi) |
17:47:23 | | Join remlap1 [0] (~Patrick@190.28.169.217.in-addr.arpa) |
17:48:10 | wodz | Ok. In order to truly cover crc32 algos used in rockbox we would need truly general implementation. This is not a problem in general as for example pycrc can generated code for us. The question is if we really want this. |
17:49:57 | | Quit remlap (Ping timeout: 276 seconds) |
17:50:03 | wodz | we can either put config struct as param to the crc routine or extend parameters list considerably (5 params fully describe crc32 algos) |
17:52:34 | wodz | if we drop some generality we could use two additional params to cover what we use now (polynominal, reflection) but I bet we will trigger sooner or later xored versions or mixed-reflected ones |
17:53:03 | amiconn | Imo we don't need more than one crc32 implementation in the core |
17:53:07 | amiconn | kiss |
17:53:43 | pow4h | how do i make a data file and read from it? int fd=rb->open("/myfile",O_RDONLY); |
17:53:43 | pow4h | char array[10]; |
17:53:43 | pow4h | rb->read(fd,array,1); |
17:53:43 | DBUG | Enqueued KICK pow4h |
17:53:43 | pow4h | rb->splashf(100,"%d",fd); |
17:53:43 | pow4h | rb->splashf(100,"%d",array[0]); |
17:54:26 | pow4h | i made myfile with vi, and typed 1234 :wq. my mp3 player returns -7 and 0 when i run it |
17:55:38 | pow4h | myfile is in the same folder as my .c file |
17:55:43 | | Join domonoky [0] (~Domonoky@rockbox/developer/domonoky) |
17:56:14 | wodz | amiconn: we have two now firmware/common/crc32.c and firmware/common/crc32-mi4.c |
17:56:52 | wodz | amiconn: The letter is fully reflected version of the former. The implementations differ also in size of LUT |
17:57:11 | | Quit remlap1 (Quit: Leaving.) |
17:57:23 | | Join Strife89|AndChat [0] (~Strife89@207.144.201.128) |
17:57:44 | wodz | amiconn: I need another one with different polynomial but not reflected to implement RKW handling in rolo |
17:58:56 | wodz | amiconn: kiss means code duplication as the simplest thing is to tweak one of the existing implementation to suit your needs |
18:00 |
18:00:00 | | Quit factor (Read error: Connection reset by peer) |
18:00:22 | amiconn | Afaik the mi4 version is only for bootloaders, so both versions don't end up in the same binary |
18:00:50 | wodz | amiconn: not true - it is used in rolo as well |
18:00:50 | amiconn | A generic, data driven implementation is most probably slower than what we have now, or am I wrong? |
18:00:54 | | Join n1s [0] (~n1s@nl118-175-223.student.uu.se) |
18:00:54 | | Quit n1s (Changing host) |
18:00:54 | | Join n1s [0] (~n1s@rockbox/developer/n1s) |
18:00:56 | amiconn | Also larger |
18:01:41 | wodz | amiconn: that depends. crc32.c uses 64bytes LUT, crc32-mi4 uses 1k LUT |
18:02:00 | pamaury | pow4h: what is your problem ? |
18:02:28 | pow4h | pamaury I thought it to return the number 1, because i wrote 1234 in myfile |
18:02:36 | pow4h | i thought it should* |
18:03:21 | pamaury | if open returns -7 it means error |
18:03:29 | pamaury | so the file was not actually opened |
18:03:39 | wodz | general implementation will be slower then specialized one for sure, but it may be much smaller then our both implementation together |
18:04:33 | pamaury | pow4h: are you testing this with the simulator or on target ? |
18:04:38 | pow4h | pamaury is there a man page or something where i can see what the errors are? |
18:04:47 | amiconn | Yeah, but is it larger than one single implementation? This wouldn't be lowmem friendly. Most targets only need one crc32 implementation |
18:05:00 | wodz | we have ~15 calls to crc_32() |
18:05:14 | pamaury | pow4h: I don't think so |
18:05:20 | amiconn | Do those different implementations just require different LUTs, or is there more to it? |
18:05:33 | pamaury | pow4h: did you put the file at the right location on the disk ? |
18:05:37 | wodz | amiconn: more |
18:06:13 | amiconn | hrrmmmm |
18:06:29 | pow4h | pamaury i don't know, where should I put it? |
18:06:55 | wodz | amiconn: basically mi4 version reflects bits in bytes of data and reflects final crc (this is not exactly how this is implemented but it works this way) |
18:07:20 | pamaury | pow4h: if you are using the simulator, it's in <your build dir>/simdisk, the same directory where make install put all the files |
18:07:44 | pow4h | pamaury i am not using the simular. how did you learn what the error code -7 stands for? |
18:07:47 | | Join factor [0] (~factor@r74-195-184-248.msk1cmtc01.mskgok.ok.dh.suddenlink.net) |
18:08:43 | pamaury | pow4h: you can't, you can have a look at the source and try to identify what does it mean but I can't say for sure what it means |
18:08:45 | pamaury | wait a minute |
18:09:27 | pamaury | apparently -7 means no file when using dircache |
18:09:42 | | Quit wodz (Quit: Leaving) |
18:09:42 | pamaury | if you are testing on target, be sure to put the file at the root of the disk |
18:10:20 | pow4h | pamaury is root the same thing as cp myfile /myfile ? |
18:10:36 | pow4h | does that copy myfile to the root? |
18:11:11 | pamaury | pow4h: the root of your target, soe cp myfile /media/where/my/player/is/mounted/myfile |
18:11:38 | pamaury | so for example, cp myfile /media/SuperDevice/myfile |
18:12:28 | pow4h | pamaury im using virtual box ubuntu in a windows xp machine. so im not sure i will look around in linux |
18:12:44 | pamaury | it's the same as putting music on your device, except that you put your file |
18:12:55 | pamaury | you can do it with windows or linux |
18:12:58 | pow4h | ok thanks i will try that |
18:13:04 | pow4h | i just copy and paste it onto the player? |
18:14:07 | pamaury | yes |
18:17:45 | | Quit hiptobecubic (Ping timeout: 252 seconds) |
18:18:39 | | Join randumbum [0] (~IRCclient@c-71-193-176-198.hsd1.wa.comcast.net) |
18:19:06 | | Join DerPapst [0] (~Adium@91-64-115-27-dynip.superkabel.de) |
18:19:36 | | Quit n1s (Read error: Connection timed out) |
18:21:42 | | Join thomasjfox [0] (~thomasjfo@rockbox/developer/thomasjfox) |
18:26:55 | pow4h | pamaury thanks now it is returning 1 and 53, but the contents of myfile are 543 so i only want it to return the first character, 5. So did i create the data file wrong or something? i typed vi myfile (insert mode) 543 :wq |
18:28:48 | | Quit mortalis (Quit: KVIrc 4.1.1 Equilibrium http://www.kvirc.net/) |
18:30:22 | pow4h | hi. i want to make a 'data file' so i can read it with a POSIX-style read function. can I just go do vi myfile, 123 :wq, or is there a special way to create the file? |
18:36:12 | gevaerts | pow4h: 53 is '5' in ascii. This is entirely expected |
18:43:18 | | Quit randumbum (Ping timeout: 260 seconds) |
18:43:35 | CIA-44 | Commit 29c7259 in rockbox by Amaury Pouly: imx233: correctly stop charging on topoff (fix battery discharging) |
18:45:03 | pow4h | gevaerts thank you. does anyone know how to type the 'start of heading' character, which is ascii for 1? |
18:45:42 | | Join jlbiasini [0] (~metaphys@d86-32-96-55.cust.tele2.at) |
18:46:27 | CIA-44 | 29c7259 build result: All green |
18:46:57 | | Join n1s [0] (~n1s@nl118-175-223.student.uu.se) |
18:46:57 | | Quit n1s (Changing host) |
18:46:57 | | Join n1s [0] (~n1s@rockbox/developer/n1s) |
18:46:59 | | Join mystica555_ [0] (~Mike@71-211-205-10.hlrn.qwest.net) |
18:48:46 | | Join GeekShadow [0] (~antoine@56.134.197.77.rev.sfr.net) |
18:50:37 | thomasjfox | Something for the ARM gurus out there: [lib/unwarminder/unwarm_thumb.c:399] -> [lib/unwarminder/unwarm_thumb.c:399]: (style) Same expression on both sides of '&&'. |
18:50:47 | thomasjfox | That line is: if(M_IsOriginValid(state->regData[rs].o) && M_IsOriginValid(state->regData[rs].o)) |
18:50:58 | | Quit Strife89 (Quit: Rebooting to Windows.) |
18:51:04 | thomasjfox | I guess the first field should be "state->regData[rd].o"? |
18:54:56 | | Join japc [0] (~japc@194.65.5.235) |
18:55:07 | | Join Strife89 [0] (~Strife89@207.144.201.128) |
18:57:58 | funman | thomasjfox: that's for wodz |
18:58:21 | thomasjfox | funman: I'll just post a patch on gerrit and he can review it |
18:58:50 | Torne | that code is just the upstream library we imported, though |
18:58:55 | Torne | :) |
18:59:00 | thomasjfox | Still in our code base ;9 |
18:59:12 | Torne | yes, but i mean: he may not be able to judge it any better than you :) |
18:59:28 | thomasjfox | Alright, I'll take a look at upstream first and report it there if needed |
18:59:46 | Torne | i suspect he last change upstream was years ago, also |
18:59:46 | thomasjfox | Judging by the code below the statement, it's borked |
19:00 |
19:00:33 | Torne | also no, it's not rd |
19:02:01 | thomasjfox | hmm |
19:02:23 | Torne | the and is just redundant |
19:02:49 | Torne | the format 4 thumb alu operations only have one source register |
19:03:01 | Torne | oh wait |
19:03:04 | Torne | no i'm dumb |
19:03:31 | Torne | yeah, you are right |
19:03:41 | Torne | this code is too damn long :p |
19:03:43 | thomasjfox | :) |
19:03:59 | thomasjfox | I'll send a fix to upstream, let's see what he has to say |
19:04:19 | Torne | we can still fix it in our code, but i suspect it will make no real difference |
19:04:27 | Torne | :) |
19:06:22 | thomasjfox | I'll push a fix |
19:15:35 | | Quit jlbiasini (Remote host closed the connection) |
19:18:31 | | Join jlbiasini [0] (~metaphys@d86-32-96-55.cust.tele2.at) |
19:18:36 | | Join dhrasmus [0] (~dhrasmus@c-76-105-170-114.hsd1.or.comcast.net) |
19:19:16 | | Quit japc (Quit: Ex-Chat) |
19:19:35 | jlbiasini | pamaury: after the sbloader command and then the dd one what should i do to recover the bootloader? |
19:20:04 | jlbiasini | just unplug? |
19:21:01 | passstab | what am i supposed to do with the message ata error -1 ? |
19:21:21 | passstab | it says i should go into disk mode |
19:21:32 | passstab | but not what i do after |
19:21:53 | jlbiasini | and also is the actual bootloader (actual i.e. compiled from git) ok? |
19:23:05 | | Join randumbum [0] (~IRCclient@c-71-193-176-198.hsd1.wa.comcast.net) |
19:30:45 | | Join lebellium [0] (~chatzilla@f052147070.adsl.alicedsl.de) |
19:33:13 | *** | Saving seen data "./dancer.seen" |
19:35:37 | jlbiasini | ok it seems I updated succesfully my bootloader but I still can't boot into of |
19:41:49 | jlbiasini | kugel: would you have a look to my PLA patches please? I know this must be f***... boring but if you could review one or two we'll be going on. I figured out that a few test_ plugins are still relying on local keymaps they will be next! |
19:42:16 | passstab | anyone? |
19:43:13 | jlbiasini | passstab: not sure I'm competent, what are you trying to do? |
19:44:10 | passstab | my ipod g2 says ata error -1 enter disk mode |
19:44:11 | fs-bluebot | No review with id 2 found |
19:44:33 | passstab | but not what i should do then |
19:44:59 | jlbiasini | what are you trying to do? on what occasion did the message appears? |
19:45:21 | passstab | i don't know what causes it |
19:45:27 | passstab | i'm running 3.10 |
19:45:48 | passstab | i can reboot and it works fine |
19:46:10 | jlbiasini | is it on power up? is it on plug USB is it while playing... ??? |
19:46:21 | passstab | but i don't know if it is indicating something importaint |
19:46:35 | | Join fml [0] (~chatzilla@manz-590f009d.pool.mediaWays.net) |
19:46:37 | passstab | i think either of those |
19:48:25 | jlbiasini | this is something related to with the disk but I just can't help anymore :( |
19:49:33 | passstab | the only thing i see that i can do in disk mode is restore to factory |
19:50:07 | pow4h | after i made a data file and used the read() function, my mp3 player says plugin returned error #leak-file-handles SELECT=yes Any other = no. the program still ran though. what should i do? |
20:00 |
20:00:15 | jlbiasini | pamaury: ok my mistake! It works |
20:00:36 | funman | pow4h: close() |
20:03:29 | | Join remlap [0] (~Patrick@190.28.169.217.in-addr.arpa) |
20:06:47 | | Join bertrik [0] (~bertrik@ip117-49-211-87.adsl2.static.versatel.nl) |
20:06:47 | | Quit bertrik (Changing host) |
20:06:47 | | Join bertrik [0] (~bertrik@rockbox/developer/bertrik) |
20:09:18 | passstab | funman: what should i do? |
20:10:44 | funman | passstab: send me your nano2g :) |
20:11:09 | passstab | does restoreing to factory help? |
20:15:02 | CIA-44 | Commit e8c7d10 in rockbox by Thomas Jarosch: Fix mismatching C++ new[] / delete calls |
20:18:08 | CIA-44 | e8c7d10 build result: All green |
20:23:14 | | Quit WalkGood () |
20:25:18 | | Join Horscht [0] (~Horscht@p54946BDE.dip.t-dialin.net) |
20:25:19 | | Quit Horscht (Changing host) |
20:25:19 | | Join Horscht [0] (~Horscht@xbmc/user/horscht) |
20:29:09 | | Quit Rower85 (Quit: Hmmm...) |
20:39:23 | | Quit factor (Read error: Connection reset by peer) |
20:48:51 | | Quit fml (Quit: ChatZilla 0.9.88 [Firefox 10.0.2/20120216080748]) |
20:50:10 | pow4h | I have int fd=rb->open("/myfile",O_RDONLY); char array[10]; rb->read(fd,array,1); rb->splashf(100,"%d",fd); rb->splashf(100,"%d",array[0]);. Is char array[10]; the only line of code responsible for changing the data file's bits into characters? |
20:51:12 | funman | pow4h: #rockbox is not a C tutorial channel |
20:51:53 | passstab | thats c# |
20:52:30 | passstab | funman is there anyone i should wait for to ask? |
20:53:19 | passstab | or did that message get in magically? |
20:53:25 | | Join curtism [0] (~curtis@unaffiliated/programble) |
20:57:03 | | Join factor [0] (~factor@r74-195-184-248.msk1cmtc01.mskgok.ok.dh.suddenlink.net) |
20:57:14 | funman | no clue |
20:59:14 | | Join hiptobecubic [0] (~john@unaffiliated/hiptobecubic) |
20:59:28 | | Quit thomasjfox (Quit: Konversation terminated!) |
21:00 |
21:03:07 | | Quit pow4h (Killed (idoru (Spam is off topic on freenode.))) |
21:04:39 | | Quit thegeek (Read error: Connection reset by peer) |
21:08:18 | | Quit Rob2222 (Quit: Rob2222) |
21:09:24 | | Join Rob2222 [0] (~Miranda@p4FFD0DB8.dip.t-dialin.net) |
21:10:29 | | Quit Rob2222 (Client Quit) |
21:10:50 | | Join Rob2222 [0] (~Miranda@p4FFD0DB8.dip.t-dialin.net) |
21:11:53 | | Quit Rob2222 (Client Quit) |
21:12:21 | | Join Rob2222 [0] (~Miranda@p4FFD0DB8.dip.t-dialin.net) |
21:13:12 | | Quit y4n (Quit: We're fucking 3LN!) |
21:16:47 | | Quit dhrasmus (Quit: Leaving) |
21:21:55 | | Join dhrasmus [0] (~dhrasmus@c-76-105-170-114.hsd1.or.comcast.net) |
21:24:23 | | Quit anewuser (Ping timeout: 244 seconds) |
21:30:13 | | Join saratoga_ [0] (980329b4@gateway/web/freenode/ip.152.3.41.180) |
21:30:38 | saratoga_ | Zagor: the "diff all" buttons on the gerrit website open up one tab PER FILE when you click them |
21:30:52 | saratoga_ | on my system with only 3GB of RAM this is a great way to crash my browser :) |
21:31:03 | saratoga_ | if i click on a very large change |
21:31:18 | saratoga_ | could we have those buttons show all the diffs on one page sequentially? |
21:33:15 | | Join fml [0] (~chatzilla@manz-590f009d.pool.mediaWays.net) |
21:33:16 | *** | Saving seen data "./dancer.seen" |
21:33:47 | | Quit fml (Client Quit) |
21:34:08 | | Join Keripo [0] (~Keripo@drl276.wlan.sas.upenn.edu) |
21:38:55 | | Join fml [0] (~chatzilla@manz-590f009d.pool.mediaWays.net) |
21:39:16 | | Quit pamaury (Ping timeout: 276 seconds) |
21:40:02 | | Quit zenlunatic (Ping timeout: 276 seconds) |
21:41:36 | | Quit factor (Ping timeout: 252 seconds) |
21:41:53 | | Join factor [0] (~factor@r74-195-184-248.msk1cmtc01.mskgok.ok.dh.suddenlink.net) |
21:46:38 | | Join anewuser [0] (~anewuser@190.207.129.184) |
21:46:38 | | Quit anewuser (Changing host) |
21:46:38 | | Join anewuser [0] (~anewuser@unaffiliated/anewuser) |
21:50:09 | | Join zenlunatic [0] (~justin@c-68-48-40-231.hsd1.md.comcast.net) |
21:53:20 | | Quit TheLemonMan (Quit: WeeChat 0.3.7) |
21:55:58 | jlbiasini | Hello, where do we stand on: FS #12584. Is there a special policy before to commit such a patch? |
21:55:59 | fs-bluebot | http://www.rockbox.org/tracker/task/12584 Updated some missing items for Simplified-Chinese translation (patches, unconfirmed) |
21:59:24 | | Part jlbiasini |
22:00 |
22:00:10 | | Quit fml (Quit: ChatZilla 0.9.88 [Firefox 10.0.2/20120216080748]) |
22:00:17 | funman | if you know a native chinese speaker ask him, else just commit |
22:00:34 | bertrik | I don't know if we have a real policy. IMO, the first contribution by someone should be looked at a bit more carefully |
22:00:37 | funman | we could use a list of what languages developers speak natively |
22:00:57 | | Quit passstab (Remote host closed the connection) |
22:01:09 | | Quit Keripo (Quit: Leaving.) |
22:01:16 | bertrik | there are some phrases in there where the translated phrase is the same as the english one, probably wrongly so |
22:02:31 | bertrik | so I think it shouldn't get committed as it is now |
22:05:00 | bertrik | meh! to people asking a question and then leaving soon after |
22:06:55 | | Quit Strife89|AndChat (Ping timeout: 255 seconds) |
22:08:38 | AlexP | saratoga_: That is a gerrit limitation |
22:08:50 | AlexP | I already asked the same question |
22:13:04 | | Quit dhrasmus (Quit: Leaving) |
22:13:05 | | Quit fs-bluebot (Read error: Connection reset by peer) |
22:14:16 | | Quit adnap (Ping timeout: 260 seconds) |
22:16:06 | | Join adnap [0] (~adnap@rrcs-71-42-140-57.sw.biz.rr.com) |
22:16:13 | | Quit Strife89 (Quit: Connection reset by deer.) |
22:16:54 | | Join Strife89|AndChat [0] (~Strife89@207.144.201.128) |
22:17:49 | | Join fs-bluebot [0] (~fs-bluebo@f053155047.adsl.alicedsl.de) |
22:20:54 | | Join pamaury [0] (~quassel@vit94-1-82-67-248-70.fbx.proxad.net) |
22:20:55 | | Quit pamaury (Changing host) |
22:20:55 | | Join pamaury [0] (~quassel@rockbox/developer/pamaury) |
22:21:36 | gevaerts | saratoga_: this won't help with actual review (i.e. adding comments and things), but the gerrit page has a gitweb link where you can look at the entire thing in one go |
22:27:33 | | Quit hiptobecubic (Ping timeout: 252 seconds) |
22:28:25 | | Join jlbiasini [0] (~metaphys@d86-32-96-55.cust.tele2.at) |
22:30:51 | | Join guymann_ [0] (~charles@69.182.30.34) |
22:32:02 | | Quit guymann (Ping timeout: 276 seconds) |
22:37:57 | | Quit jlbiasini (Remote host closed the connection) |
22:38:35 | | Quit benedikt93 (Quit: Bye ;)) |
22:41:28 | | Join Strife89 [0] (~Strife89@207.144.201.128) |
22:42:47 | | Quit dionoea (Changing host) |
22:42:47 | | Join dionoea [0] (~dionoea@videolan/developer/dionoea) |
22:50:54 | | Join Keripo [0] (~Keripo@eng430.wireless-resnet.upenn.edu) |
22:51:56 | | Quit Strife89 (Ping timeout: 255 seconds) |
22:53:58 | | Join anewuser_ [0] (~anewuser@190.207.129.184) |
22:54:39 | dfkt | is there a way to download a combined .patch/.diff from gerrit? i only found links to download single full files |
22:55:21 | gevaerts | dfkt: go through gitweb |
22:55:59 | | Quit anewuser (Ping timeout: 255 seconds) |
22:55:59 | | Join rarog [0] (~chatzilla@p4FF5D3B8.dip.t-dialin.net) |
22:56:26 | | Quit soap (Ping timeout: 255 seconds) |
22:56:40 | dfkt | gevaerts, you mean the linux application, i presume? |
22:56:45 | gevaerts | no |
22:56:50 | | Join soap [0] (~soap@rockbox/staff/soap) |
22:57:03 | gevaerts | I mean the "gitweb" link on the gerrit page |
22:57:21 | dfkt | ah, the tiny one - now i see it :) |
22:57:47 | dfkt | ah, the "parent" diff seems to be the complete one there |
22:57:55 | dfkt | thanks! |
22:58:30 | | Join Strife1989 [0] (~Strife89@207.144.201.128) |
22:58:43 | dfkt | hmm, that still can't be downloaded |
22:58:51 | saratoga_ | the git web interface stuff is ridiculously confusing |
22:58:57 | dfkt | yeh, it's horrible |
22:59:15 | saratoga_ | i guess its because everyone who uses git uses command line tools and stuff, but still |
22:59:20 | dfkt | oh wait, the "patch" link gives a plain text |
22:59:24 | gevaerts | dfkt: "raw" or "patch" |
22:59:25 | saratoga_ | the svn people got their stuff together |
23:00 |
23:00:03 | saratoga_ | also, does anyone want to take a look at Sean's librbcodec stuff on gerrit? |
23:00:18 | saratoga_ | i'm generally in favor of his work but large scale reorganization isn't really something i have a feel for |
23:00:44 | saratoga_ | that said i've been using his work for debugging codecs in the tiny bits of development time i have these days and its quite useful |
23:00:51 | saratoga_ | being able to use rockbox like ffmpeg is really, really nice |
23:02:35 | funman | reusing rockbox codecs would also be easy |
23:03:46 | | Quit adnap (Ping timeout: 245 seconds) |
23:11:41 | | Join adnap [0] (~adnap@rrcs-71-42-140-57.sw.biz.rr.com) |
23:24:14 | | Join Scromple [0] (~Simon@119.225.209.134) |
23:24:33 | | Quit ender` (Quit: The two aspiring triangle peaks arching into an abstracted ‘M’ typified the progressive leadership-minded outlook of the company. -- Motorola exec on their logo) |
23:27:32 | Torne | fml: in case hyou read this in logs, the reason you can't run git show −−pretty=fuller is because your irc client or web browser has turned the hyphens into em dashes so you are typing git show −−pretty=fuller instead which is not the same characters |
23:27:53 | Torne | fml: paste me the *actual error from gerrit* as well as checking what git show says the commit is by :) |
23:28:16 | Torne | i can tell they are em dashes because you copied them back to irc that way, and indeed that's the error you get if you use them :) |
23:30:07 | | Quit domonoky (Read error: Connection reset by peer) |
23:33:17 | *** | Saving seen data "./dancer.seen" |
23:37:36 | | Quit randumbum (Ping timeout: 252 seconds) |
23:47:32 | | Quit pamaury (Remote host closed the connection) |
23:48:00 | | Quit n1s (Quit: Ex-Chat) |
23:48:27 | | Quit saratoga_ (Quit: Page closed) |
23:54:18 | | Quit Horscht (Quit: Verlassend) |
23:54:52 | | Quit rarog (Quit: ChatZilla 0.9.88 [SeaMonkey 2.4.1/20111011102430]) |