Previous day | Jump to hour: 01 02 03 04 05 06 07 08 09 10 11 12 13 14 15 16 17 18 19 20 21 22 23 | Next day

Seconds: Show Hide | Joins: Show Hide | View raw
Font: Serif Sans-Serif Monospace | Size: Small Medium Large

Click in the nick column to highlight everything a person has said.
The Logo icon identifies that the person is a core developer (has commit access).

#rockbox log for 2012-10-05

00:00:02 Quit kevku (Ping timeout: 260 seconds)
00:02:58Buschelneed to take some sleep, see you tomorrow
00:03:01 Quit Buschel (Quit: ChatZilla [Firefox 15.0.1/20120905151427])
00:05:38 Join Horscht [0] (
00:05:38 Quit Horscht (Changing host)
00:05:38 Join Horscht [0] (~Horscht@xbmc/user/horscht)
00:09:45 Quit Horscht (Client Quit)
00:10:27 Quit lebellium (Quit: ChatZilla 0.9.89 [Firefox 16.0/20121002073616])
00:12:42 Join factor [0] (
00:19:34 Quit Rower85 (Quit: Hmmm...)
00:19:56 Quit ender` (Quit: 99% of lawyers give the rest a bad name.)
00:21:34 Join killbillkill [0] (
00:23:22 Part killbillkill
00:38:54 Quit sakax (Quit: Leaving)
00:55:33 Quit bertrik (Remote host closed the connection)
01:29:17 Quit mgottschlag (Ping timeout: 245 seconds)
01:54:55***Saving seen data "./dancer.seen"
02:07:10 Quit saratoga (Quit: Page closed)
02:09:35 Quit Poodlemastah (Ping timeout: 245 seconds)
02:29:33 Quit factor (Read error: Connection reset by peer)
02:37:49 Part LittleCreature ("WeeChat 0.3.8")
02:47:15 Join factor [0] (
02:50:52 Join Prodicus_ [0] (
02:50:56 Quit Prodicus (Ping timeout: 256 seconds)
02:51:09 Nick Prodicus_ is now known as Prodicus (
02:54:30 Quit Thra11 (Ping timeout: 246 seconds)
03:43:00 Join JdGord [0] (~AndChat80@
03:44:33 Join JdG [0] (~AndChat80@
03:46:52 Quit JdG (Read error: Connection reset by peer)
03:47:28 Quit JdGord (Ping timeout: 264 seconds)
03:51:07 Join JdGord [0] (~AndChat80@
03:52:25 Join JdG [0] (~AndChat80@
03:53:50 Join Raptors [0] (
03:54:23RaptorsAnyone know how fuze + rockbox port is going (wiki seems to be the same as I saw it a couple months ago)
03:54:51 Quit dfkt (Quit: -= SysReset 2.55=- Sic gorgiamus allos subjectatos nunc.)
03:54:56***Saving seen data "./dancer.seen"
03:55:00RaptorsI bought it today (missed out on the original fuze because my bro didn't give me his CC and forgot his password for paypal :()
03:55:31 Quit JdGord (Ping timeout: 245 seconds)
03:57:06RaptorsGoing into this very very low expectations (from what I saw)
03:58:19RaptorsCan someone please tell me that the Fuze + is not total garbage?
03:59:57RaptorsI already know I'm going to be disappointed coming off of the amazing exp I had with fuze v2 for the past 3-4 years
04:05:27 Quit JdG (Ping timeout: 244 seconds)
04:11:09 Quit amiconn (Disconnected by services)
04:11:09 Join amiconn_ [0] (amiconn@rockbox/developer/amiconn)
04:11:14 Nick amiconn_ is now known as amiconn (amiconn@rockbox/developer/amiconn)
04:11:39Raptorsamiconn, you there>?
04:12:09 Join pixelma [0] (pixelma@rockbox/staff/pixelma)
04:13:36 Quit pixelma_ (Ping timeout: 246 seconds)
04:20:58 Join amayer [0] (
04:26:19amayerwhen you select a font in settings > theme settings > font.
04:26:21amayerif you have more then one font loaded in your current theme(ie mine has 3) which font(s) will be over writen with the new font you selected from settings?
04:29:24amayerthe manual give no hints
04:35:04amayerhello *hears echo*
04:38:04 Join TheSphinX_ [0] (
04:41:23 Quit TheSphinX^ (Ping timeout: 248 seconds)
04:44:44 Join mikroflops [0] (
04:47:49 Quit mikroflops_ (Ping timeout: 244 seconds)
05:00:04 Quit Epicanis (*.net *.split)
05:00:04 Quit zchs (*.net *.split)
05:00:05 Quit Elfish (*.net *.split)
05:00:05 Quit sc_ (*.net *.split)
05:00:05 Quit gevaerts (*.net *.split)
05:00:05 Quit desowin (*.net *.split)
05:00:05 Quit knittl (*.net *.split)
05:00:33 Quit fyrestorm (Read error: Connection reset by peer)
05:00:54 Join fyrestorm [0] (
05:01:47 Join Rower85 [0] (husvagn@
05:05:30 Join gevaerts [0] (~fg@rockbox/developer/gevaerts)
05:05:30 Join desowin [0] (
05:05:30 Join knittl [0] (~knittl@unaffiliated/knittl)
05:06:19 Join zchs [0] (
05:17:37 Quit [7] (Disconnected by services)
05:17:46 Join TheSeven [0] (~quassel@rockbox/developer/TheSeven)
05:23:03 Quit advcomp2019__ (Ping timeout: 246 seconds)
05:23:23 Quit soap (Quit: soap)
05:30:16 Join advcomp2019 [0] (
05:30:17 Quit advcomp2019 (Changing host)
05:30:17 Join advcomp2019 [0] (~advcomp20@unaffiliated/advcomp2019)
05:34:52 Join soap [0] (
05:34:53 Quit soap (Changing host)
05:34:53 Join soap [0] (~soap@rockbox/staff/soap)
05:55:00***Saving seen data "./dancer.seen"
06:14:34 Join mikroflops_ [0] (
06:17:27 Quit mikroflops (Ping timeout: 240 seconds)
06:46:14 Join pedro_angelo [0] (
06:46:43 Part amayer
06:47:14 Join advcomp2019_ [0] (
06:47:14 Quit advcomp2019_ (Changing host)
06:47:14 Join advcomp2019_ [0] (~advcomp20@unaffiliated/advcomp2019)
06:48:18 Quit advcomp2019 (Ping timeout: 260 seconds)
06:50:42 Quit ranmachan (Ping timeout: 260 seconds)
06:52:59 Join ranmachan [0] (ranma@2a01:4f8:130:9321::2)
06:56:43 Join mikroflops [0] (
06:57:12 Join kevku [0] (
06:59:33 Quit mikroflops_ (Ping timeout: 240 seconds)
07:05:53 Quit pedro_angelo (Remote host closed the connection)
07:06:24 Quit Totalled (Quit: PETTAN PETTAN, TSURUPETTAN!)
07:13:59 Join Totalled [0] (
07:19:12 Quit XavierGr (Ping timeout: 244 seconds)
07:47:43 Quit Prodicus (Ping timeout: 255 seconds)
07:55:03***Saving seen data "./dancer.seen"
07:59:44 Join mikroflops_ [0] (
08:03:33 Quit mikroflops (Ping timeout: 260 seconds)
08:22:30 Join mortalis [0] (
08:27:00 Join Zagor [0] (
08:27:01 Quit Zagor (Changing host)
08:27:01 Join Zagor [242] (~bjst@rockbox/developer/Zagor)
08:45:26 Join ender` [0] (
08:56:09 Join lebellium [0] (
09:00:26 Quit zoktar (Ping timeout: 260 seconds)
09:05:08 Join einhirn [0] (
09:12:14 Join pamaury [0] (
09:12:14 Quit pamaury (Changing host)
09:12:14 Join pamaury [0] (~quassel@rockbox/developer/pamaury)
09:15:53 Join mgottschlag [0] (~quassel@reactos/tester/phoenix64)
09:17:24 Join Buschel [0] (
09:26:29 Quit [Saint] (Remote host closed the connection)
09:27:57 Join [Saint] [0] (~saint@rockbox/user/saint)
09:47:37 Quit pamaury (Ping timeout: 245 seconds)
09:55:04***Saving seen data "./dancer.seen"
10:00:28 Join LinusN [0] (
10:03:43 Quit mgottschlag (Ping timeout: 260 seconds)
10:23:52 Quit einhirn (Ping timeout: 255 seconds)
10:29:10 Join pamaury [0] (~quassel@rockbox/developer/pamaury)
10:33:54 Quit Provel (Quit: Leaving)
10:34:48 Join petur [0] (~petur@rockbox/developer/petur)
10:36:07 Join mgottschlag [0] (~quassel@reactos/tester/phoenix64)
10:40:43 Join wodz [0] (
11:04:10 Quit mortalis (Ping timeout: 260 seconds)
11:17:39 Quit derf (Ping timeout: 248 seconds)
11:24:40 Join mortalis [0] (
11:39:04 Quit mc2739 (Ping timeout: 244 seconds)
11:41:14 Join mc2739 [0] (~mc2739@rockbox/developer/mc2739)
11:42:58 Join mikroflops [0] (
11:46:40 Quit mikroflops_ (Ping timeout: 255 seconds)
11:46:50 Quit Keripo (Quit: Leaving.)
11:47:34 Quit bluebrother (Disconnected by services)
11:47:40 Join bluebrother^ [0] (~dom@rockbox/developer/bluebrother)
11:50:28 Quit fs-bluebot (Ping timeout: 260 seconds)
11:51:53 Join fs-bluebot [0] (
11:55:05***Saving seen data "./dancer.seen"
12:09:58 Quit sciopath (Read error: Connection reset by peer)
12:10:31 Join sciopath [0] (
12:17:56 Join XavierGr [0] (XavierGr@rockbox/staff/XavierGr)
12:19:02 Quit wodz (Quit: Leaving)
12:23:48 Quit [Saint] (Read error: Connection reset by peer)
12:24:11 Join [Saint] [0] (~saint@rockbox/user/saint)
12:26:04 Quit petur (Quit: *plop*)
12:32:03 Quit [Saint] (Ping timeout: 252 seconds)
12:40:08 Join Topy [0] (
12:43:42 Quit T44 (Ping timeout: 246 seconds)
12:48:05Buscheln1s: i have updated the patch in the suggested way. comb_filter() can not overflow now, the performance loss is relatively low (~0.6 MHz on pp). another change now uses a local variable "coef0" within deemphasis. this is faster on pp, maybe also for cf. the patch does also contain changes in mdct and ifft to support more detailed profiling. you should not care about that for now, biut maybe... is of help for you as well. ->
12:50:03 Quit hype (Quit: ["Textual IRC Client:"])
13:04:07 Join zoktar [0] (
13:09:35 Quit zoktar (Ping timeout: 268 seconds)
13:11:37 Join zoktar [0] (
13:13:03 Join Prodicus [0] (
13:19:35 Quit zoktar (Ping timeout: 252 seconds)
13:20:37 Join zoktar_ [0] (
13:23:52 Quit pamaury (Ping timeout: 255 seconds)
13:27:49 Join hype [0] (~hype@
13:28:03 Join dys [0] (~user@2a01:1e8:e100:8296:21a:4dff:fe4e:273a)
13:34:31 Join lorenzo92 [0] (
13:35:23lorenzo92kugel: okay I'm here to complete RDS
13:35:32lorenzo92I need to redownload everythinh
13:39:17 Join pamaury [0] (
13:39:17 Quit pamaury (Changing host)
13:39:17 Join pamaury [0] (~quassel@rockbox/developer/pamaury)
13:39:29kugellorenzo92: what? why that?
13:42:30lorenzo92kugel: no I mean, I need to fix the thing about gpio.c file cleanup
13:42:37lorenzo92need help to setup git
13:42:45 Quit Buschel (Ping timeout: 255 seconds)
13:43:13lorenzo92once I fetched the patch and done the modifications, what to do to send the modification to gerrit again?
13:43:24lorenzo92into the same patch of course
13:45:44mortalisgit push origin HEAD:refs/for/master
13:46:00mortalischange id line should be the same as on gerrit
13:55:09***Saving seen data "./dancer.seen"
13:55:43lorenzo92mortalis: so now that I applied the patch with the link on gerrit, I need to do git push origin ...bla... without the need of doing something else?
13:57:24mortalisapply patch, make some changes, commit them (git add <changed files> & git commit −−amend) then push
13:57:31gevaertslorenzo92: have you read through ?
13:57:48lorenzo92yes but I'm unsure :D
13:58:20lorenzo92mortalis: I applied with this git fetch git:// refs/changes/20/320/2 && git checkout FETCH_HEAD
13:59:13lorenzo92perhaps need to experiment also with the sandbox :)
14:00:48kugellorenzo92: most important is that you keep the change id line, then gerrit will not open a new task
14:05:16lorenzo92kugel: okay, I did git commit −−amend, now I have the same change-id: Change-Id: I1d3cc995ea3350ec7b101438b8f2027130d4a4c9
14:05:45lorenzo92after committing, do I need to type the same: git push origin HEAD:refs/for/master
14:05:59lorenzo92this because last time I don't remember the garbage I did :)
14:06:05lorenzo92this time wanna be sure hehe
14:09:23Torneyes, it's always the same push commant; the change-id line is what matches it up to the previous one
14:12:04lorenzo92Torne: okay, thanks, hope to do things correctly now :) to summarize, I fetch the current repository, then I apply the patch from gerrit, then I make some changes, add the new or modified files,then I do git commit −−amend, and finally push!
14:12:47Tornethe command you used above doesn't "apply" the patch though
14:13:07Torneit entirely replaces your local directory with whatever was current when the previous patch was written
14:13:33Tornei.e. it checks out the old version that the patch was based on
14:13:39Tornenot the current version of rockbox you just downloaded
14:13:56Torneyou probably want to click on "cherry pick" in gerrit
14:14:13Torneto get the version of the command that cherrypicks. this will put the patch on top of whatever you currently have, i.e. the latest, if you just updated
14:16:26lorenzo92ahhh now it's clear, so in fact it is the same repository at the time when I wrote the first patch! now's clear it's not just a patch apply
14:16:43lorenzo92and no I don't have to cherry pick, at least, don't know :)
14:16:52Torneyeah. checkout gives you the exact history that corresponds to the commit SHA you provide
14:17:05Tornewhereas cherrypick just copies the single patch you gave it onto your current history
14:17:34lorenzo92ok many I try to do it :D
14:27:27lorenzo92Torne: uhm I have some problems after applying, git add ... and commit I got 2 files changed while the change was only for one file...before committing I checked with git diff HEAD and the changes were correctly reported
14:28:27lorenzo92is it better to check with git show?
14:28:44 Join amayer_ [0] (
14:29:04Tornegit diff HEAD is comparing to what's already committed
14:29:09Tornei.e. the previous version of the patch
14:29:19Tornewhereas git commit −−amend is showing you everything that will be changed by the new version of the patch
14:29:26Torneincluding things in the old version of the patch that you have not changed since then
14:30:47lorenzo92ok...perhaps I know what I did wrong, since I have simply copied (restored) an old file, perhaps I need to resave it, I'm checking that atm
14:32:25lorenzo92ah by the way
14:33:29lorenzo92is it normal that in the commit comment I see also old changes (i.e. last patch against current code)?
14:34:56TorneWhen you commit −−amend you are looking at what will become the new commit
14:35:02Tornei.e. everything in the old commit plus everything you have added
14:35:16Torneif you want to undo something from the old commit you have to actually undo it, and add that as a change
14:35:56lorenzo92yes this is clear indeed, was wondering if I should see also old changes, and yes that's logical :)
14:36:07lorenzo92but I prefer to ask to avoid doing a mess hehe
14:36:14lorenzo92as I previously did ^^
14:37:30amayer_is someone available to make a change to the manual?
14:39:29 Join Buschel [0] (
14:39:59kugellorenzo92: did you clone the rockbox repo again?
14:41:46lorenzo92kugel: at the moment I simply used the same repo I had on my hdd (the original one) and fetched the gerrit patch
14:41:53lorenzo92and also edited the gpio file
14:42:03lorenzo92now need to do git add or simply git commit?
14:42:16lorenzo92a little confused now :)
14:42:22kugelokay, that's fine. it sounded like you did that because you said "i need to download everything again"
14:42:33lorenzo92ah nono
14:42:34kugelyou always need git add
14:42:44kugelbut git commit -a can do that for you
14:42:59lorenzo92okay now I git commit −−amend?
14:44:25lorenzo92pff I don't see the gpio file :(
14:44:33lorenzo92# Changes to be committed:# (use "git reset HEAD^1 <file>..." to unstage)## modified: firmware/export/config/samsungypr0.h# modified: firmware/target/hosted/ypr0/radio-ypr0.c
14:45:03kugelwell, the gpio changes should be in a separate commit
14:45:10kugelso it's kinda good it's not there
14:45:14lorenzo92ah yes indeed
14:45:26lorenzo92oh right my god xD
14:45:36lorenzo92so I can try to push? :)
14:47:50lorenzo92kugel: fatal: remote error: access denied or repository not exported: /rockbox
14:48:11kugelwhat did you type?
14:48:31lorenzo92git push origin HEAD:refs/for/master
14:50:40kugelpaste your .git/config file please
14:50:59 Join bertrik [0] (~quassel@rockbox/developer/bertrik)
14:51:43bootlfdhfrsaQuick Question :: Why isn't this image format [ ] available in rockbox ??
14:52:41lorenzo92bootlfdhfrsa: I guess because none has implemented it :)
14:53:05kugeldon't know :( ( Torne^ ?)
14:53:55lorenzo92kugel: I can eventually try with a new rockbox fetch
14:54:05lorenzo92maybe some fresh air works ^^
14:54:13kugelno, no need for that
14:54:23kugelperhaps try again
14:55:03Torneyou haven't set the push URL
14:55:09Tornethe git:// URL can only be used for pulling
14:55:22Torne(it's read only because there's no authentication in the git protocol)
14:55:23lorenzo92uh right can be
14:55:39Tornegit remote set-url −−push origin ssh://
14:55:46Torneit's in the instructions :)
14:56:11kugelbut how did you manage to push the first patches?
14:56:28lorenzo92Torne: yes it is, in fact, strange, perhaps I did something I don't even remeber :D
14:57:06Tornekugel: good question, but it's definitely missing in the config :)
14:57:37lorenzo92kugel, Torne: perfect guys! Thanks, I was on the right way but after all better to be sure instead of end up messing gerrit hihi
14:57:45lorenzo92kugel: ready to check and approve
14:57:49Torneyou would have to try quite hard to mess up gerrit :)
14:58:21Torneeven with committer permissions there is very little you can do wrong that isn't trivial to fix
14:58:36Torne(our gerrit is configured not to allow dangerous things, mostly)
14:58:54kugellorenzo92: now you need to make another commit for g#321
14:58:56fs-bluebotGerrit review #321 at : This patch adds RDS capability to Samsung YP-R0 target. by Lorenzo Miori (changes/21/321/1)
14:59:24lorenzo92kugel: that is? couldn't you simply remove it from gerrit?
14:59:31 Quit factor (Quit: Leaving)
14:59:46kugellorenzo92: the gpio fixes are needed right?
15:00:07kugelso you need to provide the commit :)
15:00:16lorenzo92ah well I'd prefer doing them in a separate commit something called like "YP-R0 cleanup"
15:00:23lorenzo92kugel: they're not needed for RDS
15:00:38Torneyou can still upload it over the top of 321
15:00:46Torneyou can just rewrite the commit message when you amend
15:00:55Torneit will replace the title/decription of the change when you upload it.
15:01:03lorenzo92Torne: yes indeed, will do that :)
15:01:29kugeli know they're not needed for RDS, hence I asked to separate them out :p
15:01:51Torne(also, if you upload them from a different branch that doesn't also include the other change, it won't list them as dependent on each other)
15:02:03lorenzo92kugel: :D
15:02:18lorenzo92Torne: ah so I don't need to do anything special, ie. no rebase etc?
15:02:47Torneif you make a new branch from origin/master and cherrypick the GPIO change onto it, then push that branch, it will upload it as a compeltely separate change that doesn't depend on the other one
15:02:58Tornethen they can be reviewed/committed independantly
15:03:18Torneif you don't know how to work with multiple branches in git then this may or may not be a good time to learn :p
15:03:50lorenzo92okay, another question: can I eventually use gerrit to store for example versions for a patch also if the "feature" isn't working yet?
15:04:04TorneJust upload it and don't ask anyone to review it
15:04:11TorneMaybe put a note that it's not finished in the commit message
15:04:15lorenzo92okay, good !
15:04:19Torneor as a review comment you put on your own change]
15:09:20 Join wodz [0] (
15:28:40lorenzo92speaking about lcd optimizations that I need to do on ypr0, what's the difference between HAVE_LCD_ENABLE and HAVE_LCD_SLEEP? In 2 words I have the possibility to shutdown the lcd and I want that rockbox goes sleeping i.e. doesn't update the graphics in the meanwhile (lcd is turned off already but I'm missing this optimization)
15:43:16lorenzo92mortalis: oh! seems I would better search in the past too :D
15:43:28kugellorenzo92: need more help with git? (you didnt upload a new gpio patch yet)
15:44:41lorenzo92well at the moment I was doing something different, anyways...I need to git fetch git:// refs/changes/21/321/1 && git checkout FETCH_HEAD, then make changes to my gpio.c file file and again add and commit. after that the push
15:44:44lorenzo92got it?
15:45:09 Quit mortalis (Quit: Leaving)
15:45:46kugellorenzo92: i think you better redo the changes. the patch that you uploaded actually reverts them
15:45:54 Quit Zagor (Quit: Clint excited)
15:46:36kugeldont you still the changes locally?
15:47:19 Join WalkGood [0] (~4@unaffiliated/walkgood)
15:47:49lorenzo92the only changes I have is the last patch I sumbmitted before
15:48:20kugelso gpio-ypr0.c is the same state as git master?
15:50:08lorenzo92wait...git master is the current code right (repo)? Then yes for the patch I submitted, but no against the actual repo
15:50:19lorenzo92don't know if it's clear ^^
15:51:53lorenzo92mortalis: in case of LCD_SLEEP, graphics behind won't be continuosly updated right?
15:55:11***Saving seen data "./dancer.seen"
15:56:32kugellorenzo92: I don't understand
15:56:40 Part LinusN
15:56:46kugelright now there is no patch that has your gpio fixes
15:57:26amayer_on about half way down there is a line that says:
15:57:29lorenzo92kugel: okay. so the point is. what if I fetch the latest repo, adding the gpio changes and then committing the patch again to gerrit (using the same ID)?
15:57:31amayer_im pretty sure it should say:
15:57:33amayer_%?mp<Stop|Play|Pause|Ffwd|Rew>(take out the % next to play)
15:58:08kugellorenzo92: yes, that sounds like the right way to do it
15:58:42lorenzo92kugel: perfect, last doubt, to add the same ID I must only add it into the commit comment right? nothing else apart updating the remote url?
15:59:29kugelright, the last line should be e.g. "Change-Id: I6db2a2cc0ec640db24e4f462ae14f15bcfa025cb"
15:59:48lorenzo92good now I'm understanding git much more hehe
16:00:00TorneIt needs to be in a separate paragraph
16:00:06Tornei.e. blank line before it
16:01:25lorenzo92Torne: yes I use the same "style" as for the others....
16:03:55wodzDoes SH have iram? If so it is used for plugins?
16:04:06wodzs/it is/is it/
16:05:45wodzok SH do have iram, now I need to figure out if it is available to plugins
16:08:22kugelwodz: it's all in config.h
16:08:47 Quit lorenzo92 (Quit: ChatZilla 0.9.89 [Firefox 15.0.1/20120907231657])
16:08:57wodznot all, a lot is in :-)
16:09:47Torneright, but i meant, it's not a stylistic choice, it's how git tools parse commit messages
16:09:56Tornemetadata is in email style headers in the last paragraph; anywhere else it's ignored
16:09:59Torneoh he's gone
16:14:07wodzgevaerts: ping
16:15:16wodzor other make gurus, I need to run appropriate strip −−strip-unneeded −−strip-debug on plugins elf. I can't manage to do that. Any help welcome
16:16:40 Part amayer_
16:22:04 Join Elfish [0] (amba@2a01:4f8:100:90a1:abc:abc:abc:abc)
16:22:23kugelwodz: you found the place where to do it. what's your question?
16:25:54wodzthe syntax :P $(shell arm-elf-eabi-strip −−strip-unneeded −−strip-debug $(BUILDDIR)/$*.rock) doesn't work
16:26:52wodzdespite the fact that it should be smart enough to take strip appropriate for the target architecture
16:29:57kugeltry $(wildcard $(BUILDDIR)/*.rock)
16:30:25kugelno wait, it should work on a single file at a time right?
16:31:29kugelwhat do you mean by doesnt work?
16:31:47kugelalso a tip: it's helpful to run make V=1 to see what command make runs
16:32:15wodzI know. The error message is like arm-elf-eabi-strip: '/home/wodz/rockbox/build/apps/plugins/lua/lua.rock': No such file
16:32:33wodzbut the file /home/wodz/rockbox/build/apps/plugins/lua/lua.rock do exists
16:33:50kugelwhere did you insert it?
16:34:10kugelI expect $*.rock to produce something like lua.elf.rock or lua.rock.rocj
16:35:39 Join amayer_ [0] (
16:37:49wodzboth versions return the same error
16:40:18kugelwodz: no idea, sorry
16:43:55wodzthis drives me crazy :/
17:09:07 Join eckoit [0] (~ryan@
17:11:07wodzhow to see DEBUGF() output? I compiled the build with debug support and now what?
17:12:37kugelI dont think you can see DEBUGF() on-target
17:13:09wodzso... how to see the output?
17:14:44kugelnot at all?
17:15:27kugelunless DEBUGF() goes into a file since saratoga's logging patch?
17:16:40 Join saratoga [0] (123e0cca@gateway/web/freenode/ip.
17:16:54saratogawodz: DEBUGF doesn't work on target, although you can use the new disk logging system via ERRORF
17:17:30wodzsaratoga: ok - is that somewhere documented how to use new system?
17:17:45saratogai sent an email to the mailing list over the summer explaining it
17:17:59saratogabut basically you call that function like DEBUGF and it ends up in a log file in the root of your player
17:18:17saratoga(provided you don't crash before the next disk spin up on HD targets)
17:18:25wodzit had some additional params, no?
17:18:39saratogaeventually DEBUGF will be folded into that system but i have had no time lately
17:19:07saratogano just like debugf
17:19:24saratogaERRORF maps to something like disklog(WARNINGLEVEL, ....)
17:19:35wodzok, lets try
17:21:50wodzsaratoga: Do I still need debug build to take this into effect?
17:22:14amayer_can someone review g#324
17:22:16fs-bluebotGerrit review #324 at : Corrected typo '%Play' with 'Play' on line 407 by Alex Mayer (changes/24/324/1)
17:22:26saratogawodz: no its in all builds
17:22:30saratogaeven the releases
17:22:43saratogaso any ERRORF code you commit will log for everyone's builds
17:22:54saratogaalthough by default WARNF and NOTEF are ignored in normal builds
17:23:42saratogathe idea is that if you have some condition that should never happen, you use ERRORF so that if it happens users can report it
17:23:56saratogafor example a malloc that fails, a null pointer, etc
17:24:10AlexPamayer_: i submitted that, thanks
17:24:37saratogalong term i was hoping to get the codecs, database, buffering code well logged so that we can actually debug random errors
17:24:42AlexPamayer_: Just personal preference, but I tend to start all manual commit messages with "manual: do blah"
17:24:58 Join bootinfdsds [0] (~Prmhfhfx@
17:25:08 Nick AlexP is now known as AlexP_ (~alex@rockbox/staff/AlexP)
17:25:26amayer_AlexP_: makes sense. ill have to remember that
17:25:45AlexP_Just so people can safely ignore it :)
17:28:00 Quit bootlfdhfrsa (Ping timeout: 240 seconds)
17:29:21 Quit eckoit (Read error: Connection reset by peer)
17:29:50 Join eckoit [0] (~ryan@
17:30:34wodzok if I use ERRORF() and want to see output I need to compile debug build, right?
17:30:53saratogano any build will work
17:31:05wodzsomething is not working then
17:31:27saratogawhere are you putting the ERRORF call?
17:31:43saratogamaybe its a header problem
17:33:10wodzmaybe it bombs out earlier not reaching my logging part
17:35:35 Quit eckoit (Read error: No route to host)
17:37:19 Join eckoit [0] (~ryan@
17:41:42 Join fling [0] (~fling@fsf/member/fling)
17:42:04amayer_if i want to add "show icons: on" to WPSLIST
17:42:05 Quit Topy (Read error: Connection reset by peer)
17:42:06amayer_do i need to make a new variable in
17:42:16flingHello! Will Rockbox work on iRiver T30?
17:44:23wodzsaratoga: The file should be saved on clean shutdown, right?
17:44:29wodzfling: no
17:45:02 Join prof_wolfff [0] (
17:48:02flingwodz: is not it planned/possible in future?
17:50:38wodzfling: nobody did the job, so I can't tell if this is possible or not
17:51:33 Quit hype (Quit: ["Textual IRC Client:"])
17:52:51pamauryamayer_: +1 for prefixing commits if possible, so you can quickly know where it belongs :)
17:53:21amayer_pamaury: should i prefix every commit?
17:53:35wodzsaratoga: I can't make ERRORF() to work :/
17:54:03amayer_pamaury: you mean like AlexP said? putting "manual: " at the begining?
17:54:57pamauryyes, but more generally you'll notice many people put "x:" at the beginning where x can be manual or just anything!
17:55:14***Saving seen data "./dancer.seen"
17:55:25pamaurythere is no rule though
17:56:22AlexP_amayer_: There isn't a rule, and for things like the manual it is obvious what to use - in other places it isn't possible or appropriate, but if something seems to fit it can help others
17:57:51amayer_AlexP_ what about something like would i put "wpsbuild: fixed ...". something like that?
17:58:03AlexP_yeah, that would be good
17:58:28AlexP_It is up to you, but if you can think of a quick prefi that might help people then that would be good :)
17:58:57pamauryanother example: many people who work on targets put the name of the target (rk27xx, imx233, as3525, ...)
17:59:24pamaurysame thing for codecs
17:59:51AlexP_yeah, "mp3: fix seeking in xxx" "clip+: fix snake" etc
18:00:07AlexP_or whatever :)
18:00:47AlexP_Don't worry about trying to force one in if it doesn't fit though
18:01:19amayer_ok will do
18:03:40saratogawodz: you can verify that its enabled for your target by putting a call to it right here:;a=blob;f=apps/main.c;h=6b6566c7ca1b9241ebfff6f477d73784707e8a1c;hb=HEAD#l493
18:04:12saratogaand yes it worked for me on clean shutdown, at least on the sandisk players
18:08:56saratogai don't have a player here to test with though
18:12:33 Quit pamaury (Ping timeout: 246 seconds)
18:12:33 Quit mgottschlag (Ping timeout: 246 seconds)
18:22:27 Quit wodz (Read error: Connection reset by peer)
18:26:45saratogawhat is ifdef (CODEC) actually checkign for?
18:26:49saratogaits impossible to grep that
18:28:04amayer_can someone look at g#325 and tell me if i got all that right?
18:28:05fs-bluebotGerrit review #325 at : wpsbuild: added 'show icons' setting to build by Alex Mayer (changes/25/325/1)
18:42:39 Join Horscht [0] (
18:42:40 Quit Horscht (Changing host)
18:42:40 Join Horscht [0] (~Horscht@xbmc/user/horscht)
18:49:11 Join wodz [0] (
18:49:33*wodz is proud to announce first relocatable elf plugin running on PP
18:52:21 Join pretty_function [0] (~sigBART@
18:57:02amayer_does anyone have a screen with a remote display?
19:00:23wodzamayer_: what you mean?
19:05:33amayer_well there are settings for remote displays. im wondering if there is a "remote show icons:" setting
19:06:02wodzdunno, but you can use sim for targets with remotes to test.
19:06:31wodziriver h100 for example
19:06:35amayer_i dont have any build stuff here.
19:06:37amayer_do you need build stuff for sim?
19:06:49amayer_im on linux ubuntu(at work)
19:07:10lebelliumI have a H320 with LCD remote
19:07:57lebelliumthere is no "remote show icons" setting
19:08:18amayer_did you read that im manual? or test on device?
19:08:43lebelliumthe H320 is in my hand
19:09:17amayer_does the remote screen have icons?
19:10:46lebelliumwait I check
19:11:40lebelliumthe setting applies for both
19:11:42lebelliumno choice
19:11:52lebelliumboth the main unit and remote*
19:12:14amayer_lebellium: thank you for checking
19:12:24amayer_*going to lunch* bbs
19:23:16 Quit WalkGood ()
19:45:09 Quit Rower85 (Quit: Hmmm...)
19:45:33RaptorsDoes anyone know how the Fuze+ port is going?
19:45:56Raptorshas the trackpad been improved or is it still a dpad?
19:46:49RaptorsAsking because (as I said yesterday) I bought a Fuze+ because the Fuze orignal got sold out due to my bro not fixing his paypal. Reluctantly got the Fuze+ in hopes rockbox would fix the UI
19:46:59Raptorsand since it was cheap
19:47:27 Quit Prodicus (Quit: ChatZilla [Firefox 15.0.1/20120905151427])
19:48:49 Join mikroflops_ [0] (
19:50:42wodzRaptors: ask pamaury
19:51:27Raptorswodz, where is he?
19:51:53 Quit mikroflops (Ping timeout: 256 seconds)
19:52:57wodzRaptors: apparently but he is often active
19:55:15***Saving seen data "./dancer.seen"
20:03:32 Join Provel [0] (
20:12:56 Quit amayer_ (Quit: going ~/)
20:18:12 Nick AlexP_ is now known as AlexP (~alex@rockbox/staff/AlexP)
20:44:43 Join amayer_ [0] (
20:58:18amayer_question: i dont want to seem rude but am i allowed to add myself to the credits?
20:59:57wodzyou mean in gerrit ticket?
21:00:50amayer_wodz: ^
21:00:57wodzif you have commit access you can
21:01:00gevaertsWe should recheck CREDITS. I'm not sure if people have maintained it properly since the git switch
21:01:01 Quit ender` (Read error: Connection reset by peer)
21:01:33 Join ender` [0] (
21:01:54wodzgevaerts: could you help with plugins.make? I want to run appropriate strip command on resulting elf
21:01:56amayer_i dont have commit access.
21:01:58amayer_isnt it just a list of everyone that has helped out with the project?
21:02:27wodzamayer_: by help we consider code or major art work
21:03:01AlexPamayer_: It isn't really the contributor who does that
21:03:15AlexPIf the person submitting considers it enough to get in, they should ad it
21:03:28gevaertsThere's been a discussion a while ago about who should be in CREDITS. I can't remember if it was here or on the ML
21:03:41AlexPIt is a bit bebulous
21:03:43gevaertswodz: do you have a full patch somewhere?
21:03:46AlexPer, nebulous
21:04:02wodz g#326
21:04:04fs-bluebotGerrit review #326 at : ELF loader for plugins by Marcin Bukat (changes/26/326/1)
21:04:26amayer_wodz: AlexP: ok well ill wait until i have some more commits under my belt before i do that
21:04:53AlexPamayer_: It is a very blurry line
21:05:15AlexPamayer_: e.g. the typo fix I committed earlier wouldn't be enough
21:05:25AlexPBut it is hard to say what would be
21:06:01amayer_AlexP: i get that
21:06:03amayer_most of the stuff i do is small things here and there. i havent done anything note worth (YET)
21:07:29amayer_well speaking of commits, anyone that knows perl: g#325
21:07:30fs-bluebotGerrit review #325 at : wpsbuild: added 'show icons' setting to build by Alex Mayer (changes/25/325/1)
21:08:33AlexPAnother question is whether that setting should be part of a theme
21:08:47AlexPAnd if so, does cabbie set it when selected?
21:10:43gevaertswodz: I assume the main issue is that plugins.make still needs the bits you pasted earlier today?
21:11:29amayer_AlexP: cabbie doesnt set it. thats why i added it.
21:11:31amayer_on original boot cabbie has icons enabled(default rockbox setting)
21:11:33amayer_but if you load a theme that doesnt use icons then reload cabbie the icons are still disabled
21:12:01AlexPI didn't look at the patch, and the commit message didn't give me that :)
21:12:25amayer_im working on the message thing
21:12:51AlexPI thought (just from your message) that it added the setting to one that is written out hen you select write theme cfg or whatever it is
21:12:55amayer_i dont want my messages to be too long, but i can never think of a short way to explain it
21:13:02AlexPlong is fine
21:13:19AlexPJust have a short summary, then a blank line, then a longer explanation
21:13:22wodzgevaerts: It is needed to run $strip −−strip-unneeded −−strip-debug on all rocks. And this $strip is something like arm-elf-eabi-strip
21:13:29AlexPKeeping to 80 chars long per line would be handy too
21:14:03AlexPand the message could be "Make Cabbie enable icons when selected" or something like that
21:14:06amayer_AlexP: how do you do that in command line?
21:14:08amayer_i type:
21:14:10amayer_git commit -am "this is the comment"
21:14:21AlexPdont' do -m "whatever"
21:14:31AlexPand it'll open your default editor
21:14:39AlexPFor you to type whatever you want in
21:14:44amayer_is that what the gerrit hook thing was?
21:14:51wodzgevaerts: currntly I run find . -name '*.rock' -exec arm-elf-eabi −−strip-unneeded −−strip-debug {} \; by hand after building
21:14:58amayer_totally confused
21:15:02AlexP-m lets you specifiy the message on the command line
21:15:07gevaertswodz: ok. Looking at it now
21:15:17AlexPif you don't use -m then it'll open your editor for you to type the message in
21:15:45amayer_is that built into git?
21:15:54AlexPand svn and ...
21:16:09amayer_sweet. i thought if i didnt put a -m it would just be blank
21:16:27AlexPyou will want to set $EDITOR
21:16:31AlexPunless you like vi
21:16:50amayer_core.editor ??
21:16:52wodzAlexP: ubuntu set it to nano by default :-)
21:17:01amayer_im on ubuntu
21:17:02AlexPwodz: heh, did they? :)
21:17:07amayer_i like nano better
21:17:10AlexPamayer_: then it will use nano apparently
21:17:17AlexP$EDITOR is an environment variable
21:17:23AlexPlike $PATH
21:17:28amayer_oh ok
21:18:10wodzhmm, EDITOR is empty but on ubuntu git uses nano by default
21:18:46AlexPoh, "The editor used to edit the commit log message will be chosen from the GIT_EDITOR environment variable, the core.editor configuration variable, the VISUAL environment variable, or the EDITOR environment variable (in that order)."
21:19:26AlexPamayer_: So yes, core.editor if you want :)
21:20:42 Join mikroflops [0] (
21:23:12amayer_that makes more sense
21:23:45 Quit mikroflops_ (Ping timeout: 240 seconds)
21:23:49amayer_is there a way i can change my commit message and update gerrit? or is that going to be complicated
21:23:58AlexPthere is a way :)
21:24:03AlexPI don't know what it is
21:24:16AlexPI would guess you amend the commit then push it again
21:24:27AlexPbut best wait for a git/gerrit ninja
21:25:25amayer_name: Alex Mayer
21:25:27amayer_level: 1
21:25:29amayer_skills: super basic git(nothing fancy)
21:25:44amayer_idk what amend does
21:25:46amayer_ill look it up on
21:25:53AlexPamend means change
21:27:42 Join mgottschlag [0] (~quassel@reactos/tester/phoenix64)
21:29:56amayer_AlexP: i get that im not sure how to use it
21:30:18amayer_git is slowly coming to me. but i still struggle with the concept
21:30:24AlexPme too
21:30:36AlexPbut to amend a commit just means to change a commit
21:30:45AlexPI can't remember exactly how though :)
21:31:38 Quit Staphylo (Read error: Operation timed out)
21:34:14amayer_is the Download code on gerrit what i would need to "git fetch" to make the ammend?
21:34:29AlexPYou do it on your repo
21:34:40AlexPif it is the last commit you made
21:35:12amayer_ok. so i dont have to be on that specific branch(the gerrit: refs/changes/25/325/1)
21:35:21amayer_is that what i would push too?
21:35:38AlexPI'm going to stop answering as I'm not sure, and I don't want to get it wrong
21:35:40amayer_git push origin HEAD:ref/changes/25/325/1
21:36:12amayer_AlexP: ok
21:38:34wodzamayer_: no. follow strictly UsingGit it says git push origin HEAD:refs/for/master
21:39:18wodzif you have proper Change-Id line in commit message gerrit will manage to put it where it should be
21:40:18amayer_you mean if the SHA is the same?
21:40:28 Quit pretty_function (Ping timeout: 244 seconds)
21:40:29amayer_the git SHA id thing
21:40:51gevaertsThe Change-Id line, not the commit hash
21:42:28amayer_oh i see. git attaches that at the end of a commit message?
21:42:38amayer_i guess thats what the gerrit hook does
21:42:43gevaertsamayer_: the gerrit hook does that, strictly speaking
21:42:57amayer_its all becoming more clear
21:43:14gevaertswodz: I think on top of g#326 might do it.
21:43:16fs-bluebotGerrit review #326 at : ELF loader for plugins by Marcin Bukat (changes/26/326/1)
21:43:28*wodz looks
21:44:25gevaertsThe change to the LD line probably isn't needed, but I get annoyed when $@ isn't used :)
21:46:20gevaertsNote that that entire $(call...) in e.g. the LD line is only to suppress regular output and handle the V=1 thing, but doesn't actually enclose the $(CC) command itself
21:47:04wodzshouldn't there also be line in configure like STRIP=`findtool ${STRIP} −−lit` ?
21:48:01gevaertsHm, yes. I missed that part
21:49:27amayer_gevaerts: so if i amended the commit i should repush it?
21:49:27 Join einhirn [0] (
21:49:35gevaertsamayer_: yes
21:49:42amayer_just put to refs/for/master
21:49:47gevaertsWell, only if you like the new version :)
21:49:49amayer_the line on UsingGit
21:49:57amayer_ok thank you
21:50:13gevaertsYes. The push line for new gerrit patches or modified patches is exactly the same
21:51:26*gevaerts has 'gerrit = push origin HEAD:refs/for/master' as a git alias
21:51:58wodzgevaerts: seems to work
21:52:49amayer_AlexP: would this be a better message?
21:52:51amayer_ Make Cabbiev2 enable icons when selected
21:52:55amayer_ adds 'show icons: on' to cabbiev2.cfg file.
21:52:57amayer_ when you switch theme away from cabbiev2 to a theme
21:52:59amayer_ with 'show icons: off' then switch back to cabbiev2
21:53:01amayer_ the icons are still off which doesnt mimic the default
21:53:03amayer_ settings presented by rockbox.
21:53:05amayer_gevaerts: how do i do that?
21:53:15AlexPamayer_: pastebins are a good concept :)
21:53:28amayer_AlexP: sorry
21:53:34AlexPIt's fine :)
21:54:03AlexPBut yeah, looks fine to me. I'd leave a blank line after the first line
21:54:08gevaertsamayer_: add an [alias] section to .git/config with that line
21:54:47amayer_so then the command "git gerrit" will push to origin?
21:54:59wodzgevaerts: thanks for help
21:55:06 Quit wodz (Quit: Leaving)
21:55:16***Saving seen data "./dancer.seen"
21:55:24gevaertsamayer_: I don't know the proper way to describe what it does, apart from "the right thing, in this case" :)
21:55:46amayer_is that what you type tho "git gerrit" ?
21:56:11gevaertsI *believe* the origin and HEAD bits make sure only stuff on your currently active branch gets pushes
21:59:08gevaertsamayer_: of course, feel free to use another name than gerrit for it
21:59:44amayer_gevaerts: that is soo awesome!
21:59:46amayer_i think im going to use 'torockbox' or something
22:00:16gevaertsYou can do all sorts of fancy stuff with aliases. I'm sure the web has many examples
22:01:00amayer_is that something we should put in the wiki?
22:01:03amayer_it would be very helpful
22:01:31gevaertsIt wouldn't hurt, I guess
22:02:08amayer_im on it
22:04:20 Quit Buschel (Quit: ChatZilla [Firefox 15.0.1/20120905151427])
22:13:56 Quit saratoga (Ping timeout: 240 seconds)
22:16:59 Quit efyx (Ping timeout: 246 seconds)
22:29:39 Join speckmade [0] (
22:31:02 Join efyx [0] (~efyx@
22:33:14 Quit zoktar_ (Quit: ZNC -
22:33:49speckmadeI got a Sansa Clip+ today and I'm so happy about the greatness of Rockbox and how easily it installs.
22:37:15speckmadeI just installed the IT-MAY-EXPLODE daily build version and now enjoy listening to files in the brand-new incredible Opus format.
22:37:23speckmadeThat's impressive!
22:38:10gevaertsspeckmade: glad to hear you're enjoying it :)
22:38:15speckmadeI'd like to thank everyone involved in making this great thing happen!
22:38:22 Join pedro_angelo [0] (
22:41:12*the-kyle is also impressed with the stuff going on in git. I patched opus into my build a day before it went live in git, and I was impressed with the sound then, and am impressed with the stuff that has hit my git build since then as well.
22:41:46*the-kyle never dreamed such small files could sound so good.
22:46:01 Join zoktar [0] (
22:46:08 Part amayer_
22:47:05 Quit kevku (Quit: KVIrc 4.2.0 Equilibrium
22:57:59 Quit ender` (Quit: The human eye is a wonderful device. With a little effort, it can fail to see even the most glaring injustice.)
23:12:14 Quit scorche|sh (Changing host)
23:12:14 Join scorche|sh [0] (~scorche@rockbox/administrator/scorche)
23:13:32 Quit Horscht (Quit: Verlassend)
23:14:56 Quit the-kyle (Quit: Leaving.)
23:22:16 Quit einhirn (Quit: Miranda IM! Smaller, Faster, Easier.
23:51:53 Quit eckoit (Quit: eckoit)
23:55:19***Saving seen data "./dancer.seen"

Previous day | Next day