--- Log for 15.01.117 Server: nylund.freenode.net Channel: #rockbox --- Nick: logbot Version: Dancer V4.16 Started: 10 days and 3 hours ago 00.03.17 # massa: you can include it in your patch for cross compile 00.03.54 Quit johnb3 (Ping timeout: 240 seconds) 00.04.32 # yes I'll do - after that I see some more warnings; e.g. that UNALIGNED is already defined - is using UNALIGNED as name for a macro a good idea? 00.09.11 # it's not a standard macro name, I don't see why it should be predefined somewhere 00.09.28 # where is it already defined ? 00.11.30 # it's in /usr/i686-w64-mingw32/sys-root/mingw/include/_mingw.h 00.12.13 # it's defined either as __unaligned or as "nothing" 00.12.40 # (the first in win64 environments, the latter in win32)... 00.16.01 Quit robertd1 (Ping timeout: 245 seconds) 00.16.06 # ah, that's very unfortunate 00.19.25 Quit lebellium (Quit: ChatZilla 0.9.93 [Firefox 50.1.0/20161208153507]) 00.21.46 Join robertd1 [0] (~root@186-90-12-124.genericrev.cantv.net) 00.24.59 # massa: the way I see it, you could add code to #undef UNALIGNED if it is defined in the offending files 00.25.05 # with a comment saying why 00.25.27 Quit n3m9 (Read error: Connection reset by peer) 00.25.37 # it seems it's only one place affected - in firmware/asm/memset16.c 00.25.54 # and there I just rename UNALIGNED in RB_UNALIGNED ;-) 00.30.07 # (at least for my target device - when I search for UNALIGNED, there are a lot of places which define it...) 00.30.49 Quit cc___ (Ping timeout: 240 seconds) 00.30.56 # <[Saint]> grepping for it across the entire tree shows it is used rather extensively. 00.31.38 # yeah, that's what I wanted to say with "a lot of places" :-) 00.32.47 # <[Saint]> You're doing the work that no one really bothered with at the moment. 00.32.47 # <[Saint]> Hard to argue that it isn't important, but it evidently wasn't anyone's pressing priority. 00.33.14 # <[Saint]> I never pay any attention to defined but unused or already defined warnings. 00.33.33 # <[Saint]> I usually go out of my way to get the compiler to just shut up about it. 00.33.33 # * pamaury thinks that defining UNALIGNED in mingw.h is big mistake 00.34.06 # <[Saint]> How so? 00.34.08 # but that might be a Windows thing because windows.h defining a awful lot of things 00.34.42 # because it's a name you migth be tempted to use, that has no obvious meaning in C nor any standard construct 00.34.59 # <[Saint]> Oh. Right. 00.35.02 Quit robertd1 (Ping timeout: 260 seconds) 00.35.29 # there is another one like it in POSIX that I run into from time to time, can't remember it 00.35.37 Join robertd1 [0] (~root@186-90-12-124.genericrev.cantv.net) 00.36.01 # <[Saint]> UNINT? 00.36.01 # [Saint]: you're right - actually it's totally unimportant to know if a define is already defined and gets redefined 00.36.10 # for example, say you create an enum for errors: 00.36.16 # enum err_t { ..., UNALIGNED, ... }; 00.36.19 # boom, compiled error 00.36.41 # that's the level of badness I envision 00.36.52 # pamaury: good point 00.37.53 # <[Saint]> In other projects I'll generally do what massa is doing now and use a project-specific prefix. 00.38.01 # <[Saint]> like PROJECT_UNALIGNED 00.38.03 # ah yeah, the POSIX one is OVERFLOW 00.38.15 # which is #define OVERFLOW 1 or something like it 00.38.22 # <[Saint]> pamaury: ah, right - yeah I've bounced off that one too I believe. 00.38.48 # [Saint]: that's what I do, too - or if in C++, use namespaces :-) 00.39.24 # <[Saint]> I like mine to be verbose bordering on lunacy, on the other end of the scale some people get annoyed with that. 00.40.15 # <[Saint]> as an example instead of RB_UNALIGNED I'd likely do ROCKBOX_UNALIGNED to remove any doubt that RB wasn't come acrnym that refers to another esoteric function elsewhere. 00.40.31 # <[Saint]> bah - typing 00.40.36 # <[Saint]> *some acronym 00.41.00 *** Saving seen data "./dancer.seen" 00.41.40 # <[Saint]> Though in this context I sincerely doubt anyone would find RB to be an ambiguous prefix. 00.43.15 # <[Saint]> massa: I wouldn't say it's /totally/ unimportant, for what it's worth. 00.43.19 # what if I add "-UUNALIGNED" to the GCCOPTS - would that help? 00.43.29 # <[Saint]> 99.99% of the time it is absolutely intentional. 00.43.35 # <[Saint]> but that .01%... 00.44.10 # massa: I would go for rename to RB_UNALIGNED 00.44.49 # <[Saint]> for GCCOPTS my usual method is "throw shit at it from stackoverflow top comments until it compiles". 00.44.55 # <[Saint]> It's a very scientific method. 00.46.06 # don't bother - that -UUNALIGNED would not help as it's defined in an include... 00.46.21 # * chrisjj finds that inserting the ZEN blanking plate in the SD slot causes RB to *PANIC*. 00.46.35 # chrisjj: that's not new 00.46.41 # * pamaury leaves 00.46.57 # pamaury: you mean - search and replace every occurance of UNALIGNED to RB_UNALIGNED (or ROCKBOX_UNALIGNED)? 00.47.09 Quit robertd1 (Ping timeout: 240 seconds) 00.47.43 # New to me and RB Flyspray, I think. 00.48.43 Join robertd1 [0] (~root@186-90-12-124.genericrev.cantv.net) 00.51.04 Quit pamaury (Ping timeout: 240 seconds) 00.53.43 # * massa leaves too 00.54.04 # * massa says: good night - for those where it's night :-) 00.54.14 Quit massa () 00.56.26 # <[Saint]> soooooo many 'bugs' on flyspray that are PEBCAK, honest mistake, or hardware error. 00.56.32 # <[Saint]> God I regret looking... 00.58.26 Part robertd1 00.59.41 # <[Saint]> need to go on a shitkicking spree and close out a bunch and ask a bunch of users if they can still reproduce. 01.16.53 Join jhMikeS [0] (~jethead71@d192-24-173-177.try.wideopenwest.com) 01.18.33 Quit girafe (Read error: Connection reset by peer) 01.25.49 Quit shmibs (Quit: leaving =o) 01.26.25 Join shmibs [0] (~shmibs@shmibbles.me) 01.38.01 Join CrashBash-Kun [0] (~CrashBash@unaffiliated/crashbash-kun) 02.08.43 Quit ender` (Quit: Hardware, n The part of the computer that you can kick.) 02.31.10 Quit ZincAlloy (Quit: Leaving.) 02.41.03 *** Saving seen data "./dancer.seen" 03.38.33 Join Senji_ [0] (~Senji@85.187.103.250) 03.40.59 Quit Senji (Ping timeout: 240 seconds) 04.14.36 Join Senji [0] (~Senji@85.187.103.250) 04.17.05 Quit Senji_ (Ping timeout: 240 seconds) 04.38.18 Quit Senji (Ping timeout: 260 seconds) 04.41.06 *** Saving seen data "./dancer.seen" 05.47.08 # <__builtin> wow, people still use flyspray? 05.47.23 # <__builtin> I don't even remember how to get there 05.54.22 Join idonob_ [0] (~Owner@S0106602ad08ec3dd.vs.shawcable.net) 06.25.05 Quit [7] (Ping timeout: 256 seconds) 06.25.22 Join TheSeven [0] (~quassel@rockbox/developer/TheSeven) 06.38.44 Quit CrashBash-Kun (Read error: Connection reset by peer) 06.40.51 Join JanC_ [0] (~janc@lugwv/member/JanC) 06.40.51 Nick JanC is now known as Guest24786 (~janc@lugwv/member/JanC) 06.40.51 Quit Guest24786 (Killed (adams.freenode.net (Nickname regained by services))) 06.40.51 Nick JanC_ is now known as JanC (~janc@lugwv/member/JanC) 06.41.10 *** Saving seen data "./dancer.seen" 07.35.44 Join CrashBash-Kun [0] (~CrashBash@unaffiliated/crashbash-kun) 07.36.08 Quit CrashBash-Kun (Client Quit) 08.41.12 *** Saving seen data "./dancer.seen" 08.49.25 Join CrashBash-Kun [0] (~CrashBash@unaffiliated/crashbash-kun) 08.49.49 Quit CrashBash-Kun (Client Quit) 08.54.28 Join johnb3 [0] (~johnb2@p57B459F7.dip0.t-ipconnect.de) 08.59.13 Quit johnb3 (Ping timeout: 256 seconds) 09.33.05 Join paulk-collins [0] (~paulk@gagarine.paulk.fr) 09.36.04 Join ender` [0] (krneki@foo.eternallybored.org) 09.43.33 Join johnb3 [0] (~johnb2@p57B459F7.dip0.t-ipconnect.de) 10.09.48 Quit johnb3 (Quit: Nettalk6 - www.ntalk.de) 10.19.44 Quit krnlyng (Ping timeout: 252 seconds) 10.20.30 Join xorly [0] (~xorly@57.249.broadband3.iol.cz) 10.33.02 Join krnlyng [0] (~liar@178.112.144.37.wireless.dyn.drei.com) 10.33.06 Join ZincAlloy [0] (~Adium@2a02:8108:8b80:1700:19e:a62c:a52f:7163) 10.38.32 Join Senji [0] (~Senji@85.187.103.250) 10.41.13 *** Saving seen data "./dancer.seen" 10.45.37 Quit xorly (Ping timeout: 245 seconds) 11.03.16 Join cc___ [0] (~ac@2001:910:113f:1:6a05:caff:fe1c:1627) 11.05.36 Join johnb3 [0] (~johnb2@p57B459F7.dip0.t-ipconnect.de) 11.07.08 # Mihail: can you please provide the source / patch for rockbox-clip+-cvdd2_scaling-8_sd_only.zip (the sd_only part), so I can do this on my own in the future? 11.08.15 # With this build however I have zero reception on FM! That's a variant 1. 11.10.17 Join lebellium [0] (~chatzilla@89-93-179-5.hfc.dyn.abo.bbox.fr) 11.15.25 # __builtin: people just click on the Development>bugs from the home page. 11.18.10 Join Mihail [0] (25d48052@gateway/web/cgi-irc/kiwiirc.com/ip.37.212.128.82) 11.18.55 # No FM reception also with scaling-9, so probably with all of them. It does work well with the current dev builds. 11.19.52 # With rockbox-clip+-cvdd2_scaling-9 on my clip+ variant 0, FM works as normal! 11.20.26 # http://knk.square7.ch/rb/rockbox-dont_mount_internal_drive.patch 11.20.55 # we try fix fm later 11.20.58 Join smoke_fumus [0] (~smoke_fum@dynamic-vpdn-93-125-15-142.telecom.by) 11.21.46 # thanks a lot! 11.24.42 # How trivial magic can be ;-) 11.25.27 # just hack :) 11.28.42 Quit cc___ (Ping timeout: 258 seconds) 11.29.26 Quit Senji (Quit: KVIrc 4.9.2 Aria http://www.kvirc.net/) 11.29.50 Join Senji [0] (~Senji@85.187.103.250) 11.43.05 Join pamaury [0] (~pamaury@rockbox/developer/pamaury) 11.45.04 Quit johnb3 (Ping timeout: 255 seconds) 11.47.07 # pamaury: when "do_fw_upgrade", there is an empty "Result:" line. Is it supposed to display something when upgrade completed or is it a generic line for every scsitool command? 11.55.08 # it's normal iirc 11.58.27 # But I can look/fix it if it's confusing 11.59.16 # that's a minor thing 11.59.40 # Better look on the sound issues :P 11.59.45 # at* 12.03.45 # I think it's because the old code used to display the error code unconditionally (OK being success) but the new one only does on error 12.13.54 Join bertrik [0] (~bertrik@rockbox/developer/bertrik) 12.23.54 Quit bertrik (Quit: Lost terminal) 12.26.24 Join bertrik [0] (~bertrik@5354321C.cm-6-5a.dynamic.ziggo.nl) 12.26.24 Quit bertrik (Changing host) 12.26.24 Join bertrik [0] (~bertrik@rockbox/developer/bertrik) 12.41.17 *** Saving seen data "./dancer.seen" 12.53.10 Join MrZeus_ [0] (~MrZeus@81.144.218.162) 12.54.57 Quit MrZeus (Ping timeout: 256 seconds) 13.01.19 Quit Mihail (Quit: http://www.kiwiirc.com/ - A hand crafted IRC client) 13.03.34 Quit idonob (Ping timeout: 240 seconds) 13.07.53 Join skapazzo [0] (~skapazzo@151.9.205.1) 13.11.04 Join idonob [0] (~Owner@S010610c37b922980.vs.shawcable.net) 13.14.44 # pamaury, any particular reason why your battery benchmark procedure https://www.rockbox.org/wiki/CreativeZENPort#Battery_Calibration charging specified the power sources as PC rather than charger? 13.21.21 # no 13.22.33 # Thanks. Any particular reason why it specifies charging from PC is to be by 'holding a key (so that usb mode is not triggered)'? 13.23.10 Quit idonob (Ping timeout: 240 seconds) 13.23.45 # yes, you don't want usb mode because otherwise you can't enable battery_bench 13.24.14 # (since supposedly, when you finish the discharge part, you're device cannot be powered from battery) 13.24.40 # also battery_bench requires disk access 13.24.48 # thus it won't work with usb mode 13.24.58 # (I think) 13.25.25 Join idonob [0] (~Owner@S010610c37b922980.vs.shawcable.net) 13.28.49 Join johnb3 [0] (~johnb2@p57B459F7.dip0.t-ipconnect.de) 13.32.35 # s/usb mode/usb disk mode/ 13.33.24 # Figures. Perhaps (despite that I am indeed avoiding USB mode) 'requires disc access' is the clue to why I'm not getting more than one battery-bench data point from this. Here's the end of file: http://i.imgur.com/MjKInaj.png . It could be me bungling it somehow, or again some unknown required variable. 13.33.26 Join JanC_ [0] (~janc@lugwv/member/JanC) 13.34.11 # For this memDFS benchmarking I've skipped this charge phase, and will investigate another time. 13.34.37 Quit JanC (Killed (orwell.freenode.net (Nickname regained by services))) 13.34.37 Nick JanC_ is now known as JanC (~janc@lugwv/member/JanC) 13.36.35 # 'usb disc mode'? Is this to imply that this mode does something else with USB data? I'd hoped it was doing nothing with USB data i.e. was using the USB for only power. 13.37.30 Quit johnb3 (Ping timeout: 240 seconds) 13.41.41 Quit smoke_fumus (Quit: KVIrc 4.2.0 Equilibrium http://www.kvirc.net/) 13.43.42 Join massa [0] (4fe8d32c@gateway/web/freenode/ip.79.232.211.44) 13.44.15 # Hi everybody! 13.46.21 # pamaury: I finished the configure changes and also managed to upload the changes for review to Gerrit; as I'm new to this - how do I trigger a review now? Which URL shall I present for reviews? 13.48.17 # massa: for reviews I think you add people to the list of reviewers 13.48.25 # but it's usually better to just come on IRC and ask people 13.48.37 # for the link, you can write g#n where n the number of the task 13.48.49 # g#1497 13.48.51 # 3Gerrit review #1497 at http://gerrit.rockbox.org/r/1497 : 3Renamed defines UNALIGNED to ROCKBOX_UNALIGNED - UNALIGNED is already by Matthias Mohr 13.48.58 # g#1496 13.49.00 # 3Gerrit review #1496 at http://gerrit.rockbox.org/r/1496 : 3Fix configure for windows simulator builds in cygwin / mingw cross-compiling environm... by Matthias Mohr 13.50.10 # chrisjj: usb disk mode as in you get a disk over usb and thus the player cannot be operated. This is contrary to when you hold a key on usb plug and you don'get usb disk mode 13.52.06 # pamaury: you mean the change numbers? it's 1496 and 1497 - reachable as http://gerrit.rockbox.org/r/1496 http://gerrit.rockbox.org/r/1497 13.52.15 # yes 13.53.10 # OK, got it. 13.53.28 # massa: I'll try to review your patches today 13.54.34 # pamaury: thanks - the patch for the puzzle function conflict (vsscanf) which is needed for compilation is not included - I thought it's __builtin's part ;-) 13.55.02 # ok 13.56.39 # My ZEN doing the memDFS benchmark has just done a Black Screen of Power Off, as before. (<< True not false alarm.) LED (that was on) is off; audio (that was sounding) is silent. Any other observations you'd like me to do? 14.04.47 # (This a clean boot of a clean install to which has been added only audio files + battery_bench.txt. Unit Q.) 14.05.06 # pamaury: This brings me to problem with the compiled 64-bit simulator. In the first place it works - but if you try e.g. to start playing some music it does not load the codec files - a SDL error appears, saying that the DLL initialising method failed... 14.05.41 # So, how does the codecs get loaded in simulator environment? 14.06.12 # on linux it is compiled to .so (dynamic library), on windows it should be a dll 14.07.49 Quit paulk-collins (Remote host closed the connection) 14.10.36 # pamaury: the file format must be O.K. - otherwise it would say "wrong file format" or something similary; it seems it tries to load it but the initialization fails... 14.10.38 Join n3m9 [0] (~n3m9@ANantes-652-1-64-223.w90-59.abo.wanadoo.fr) 14.11.36 # I enabled DEBUG and LOGF in my build - but I don't see a lot messages. Doesn't this work in simulator buidls? 14.13.18 # logf needs to be enabled per file (by defining #define LOGF_ENABLED before #include "logf.h"). I think DEBUG should print the console 14.13.27 # However I wouldn't compile with DEBUG if I were you 14.14.02 # because last week I noticed that codecs seem broken with DEBUG, I couldn't identity why (it was in a real target, not simulator) 14.14.21 # I'd say, either use a debugger or printf 14.14.33 # I'll try it in my VM 14.18.41 # massa, If you'd like a test of your build of the Windows sims for ZEN and iPod Video, do post them. 14.18.43 # pamaury: did you install the 64-bit mingw compiler chain? Lately you only had the 32-bit versions 14.19.00 Join petur [0] (~petur@rockbox/developer/petur) 14.20.14 # massa: I installed both, but right now I'm busy, I'll debug that later if I can 14.24.23 # I don't really know what is done on DLL init, I am not sure we register any action on it, so it's mostly libc job I think 14.25.11 # chrisjj: you mean to upload the binary packages to some cloud? 14.25.24 # Yes. 14.28.02 # massa: you want to load into firmware/target/hosted/sdl/load_code-sdl.c 14.28.06 # *look 14.28.20 # that's the code used to load a codec/plugin/code 14.28.41 # (on SDL) 14.28.47 # pamaury: I'm already there - but there's only the call to SDL_LoadObject and that gives back a NULL handle -... 14.28.59 # have to find out why... 14.29.11 # ah, that's going to be tough to debug 14.29.17 # I think you need a debugger 14.29.40 # pamaury, this BSoPO'd ZEN showed no response to any key except power-on. Power-on launched RB as normal. So I assume the cause was a reset by angry watchdog, since no other potential cause is known. 14.31.24 # ZEN tester's Pro Tip: Remember that BSoPO loses the session's settings changes e.g. Repeat All for battery benchmarking. 14.31.54 # massa: it works on mine, but I did a 32-bit build, let me check with a 64-bit one 14.33.15 # pamaury: that's what I mean the 32-bit version is working, the 32-bit one is _new_, so maybe there is some loading or generating code which does not work for 64-bit... 14.33.31 # I meant: the 64-bit one is _new_ 14.35.25 Join paulk-collins [0] (~paulk@no3.u-bordeaux.fr) 14.36.25 # compiling a 64-bit build, this will take a while 14.39.12 # one possibility (I'm not sure), is the "long" confusion on linux/windows 14.39.44 # pamaury: the 32-bit version also works with DEBUG enabled - so that cannot be the reason... 14.39.45 # cause on Windows, long is still 32-bit in 64-bit mode 14.40.21 # massa: don't be so sure though, I've got target that work in DEBUG and some that don't 14.40.22 # pamaury: well, there were some pointer cast warnings in 64-bit 14.40.39 # better not silence them 14.41.20 *** Saving seen data "./dancer.seen" 14.41.32 Quit tchan (Quit: WeeChat 1.6) 14.42.53 # there might be some fixing needed 14.46.25 # hum there are some suspicious ones 14.48.25 # * pamaury looks into it 14.51.16 Join johnb3 [0] (~johnb2@p57B459F7.dip0.t-ipconnect.de) 14.53.50 Join tchan [0] (~tchan@c-50-129-174-2.hsd1.il.comcast.net) 14.53.51 Quit tchan (Changing host) 14.53.51 Join tchan [0] (~tchan@lunar-linux/developer/tchan) 14.55.42 # arg, why people use cast to long from pointers??? 14.55.45 # there intptr_t for that 15.02.39 # * pamaury fixes things 15.05.18 # so far only one of them is really dangerous, the one in filesystem-win32.c 15.05.49 # BSoPO again. And this time the first few restarted BSoPOed after the BL text and before the RB logo. I've seen this before. Goodness only knows where the causal state is residing. 15.08.07 # Perhaps battery_bench is itself the cause. 15.08.17 # Perhaps a battery_bench.txt that is failing to take further entries, as reported above for the charging phase. FWIW, I've deleted battery_bench.txt before starting the plugin this time. 15.08.55 # If that too BSoPOs, I'll retry with the plugin not running. 15.09.20 # s/the first few restarted BSoPOed after the BL text/the first few restartes BSoPOed after the BL text/ 15.10.11 # * pamaury touches the skin_parser codes and tries not to break it 15.19.04 Quit bertrik (Remote host closed the connection) 15.22.37 # massa: any progress ? I am fixing a bunch of warnings and although some are dangerous, they would simply crash the simulator if they were triggered 15.29.23 # arg, seriously libtremor, casting a pointer to unsigned long, I expected better 15.34.14 # pamaury: no, I did some real world (tm) activity :-) 15.34.47 Join MT_ [0] (~MT@2601:482:4402:7b60:a8e4:1c64:d2d1:4ba6) 15.35.09 Quit munch (Quit: Let's blast this shit and get naked) 15.35.56 Join munch [0] (pls@gateway/shell/elitebnc/x-hjjqdjfndwontqqf) 15.36.25 Nick MT_ is now known as _mt_ (~MT@2601:482:4402:7b60:a8e4:1c64:d2d1:4ba6) 15.37.17 Quit johnb3 (Ping timeout: 245 seconds) 15.38.18 # haha ok :-p 15.38.26 # I'm almost done 15.38.49 # I've added a comment to your gerrit task 15.38.58 # chrisjj: which ZEN simulator do you want? Just "Zen"? 15.40.34 # Yes please. 15.41.00 Join johnb3 [0] (~johnb2@p57B459F7.dip0.t-ipconnect.de) 15.41.30 # pamaury: how can I update an existing patch at gerrit? Somehow with "git --ammend", but how do I tell it which change# to use? 15.41.55 # I meant "git commit --ammend" 15.42.24 # * chrisjj could seriously believe --ammend in git 15.43.47 # massa: you have two options: 15.43.47 # - use git --amend and don't touch the Change-Id: .... line at the bottom of the message, then upload again 15.43.47 # - create a completely different commit but copy-paste the old ChangeId: ... line 15.44.10 # this line: 15.44.10 # Change-Id: Ib5dd288c3f9a11be9b66e53e3406eadfa65f7604 15.44.21 # that's the only thing that gerrit uses to track tasks 15.46.09 # * pamaury spots unsigned int wasted = (-(long) buffer) & 3; ---> VERY suspicious 15.46.16 # ZEN Unit Q memDFS: Again BSoPO. After 3min. This time with a short battery_bench.txt, exonerating long battery_bench.txt. 15.47.37 Quit johnb3 (Ping timeout: 255 seconds) 15.50.20 # pamaury: and how do I tell --amend which change-Id / earlier commit I mean? E.g. if I do different changes on the same file? Or add changes to another file and want it to get to the same change#? 15.50.20 # is it me or: 15.50.20 # #define PTRDIFF(pe, ps) ((int) (((long) (pe) - (long) (ps)) / sizeof(*pe))) 15.50.20 # is exactly equivalent to: 15.50.20 DBUG Enqueued KICK pamaury 15.50.20 # pe - ps 15.51.17 # massa: if you want to change just the last commit, use git --amend. If you want to change an ealier commit (like the one before the last), it's more complicated, you need interactive rebase 15.53.22 Join cc___ [0] (~ac@2001:910:113f:1:6a05:caff:fe1c:1627) 15.53.33 # I suggest you document yourself on the internet, but basically it goes like this: 15.53.34 # git rebase -i # for example git rebase -i HEAD^^ (the ^ means "one before") 15.53.34 # then you get a text editor: change "pick" to "edit" for the commit(s) you want to change 15.53.34 # now git does stuff and you get into a state where the first commit you want to "edit" is HEAD: do you changes, git commit --amend 15.53.34 *** Alert Mode level 1 15.53.34 # then continue by doing git rebase --continue 15.53.45 # it's very pwoerful but a bit intimidating the first times 15.58.56 # as expected things like z80 and doom are horrible offenders 16.03.35 *** Alert Mode OFF 16.04.21 # pamaury: I read some rebase documentation - but I'm lost :-( 16.05.27 # I did an interactive rebase as suggested, marked the corresponding commit as edit, put my changes in, commited it again and then tried to "rebase --continue" 16.06.05 Quit tchan (Quit: WeeChat 1.6) 16.06.05 # but then it complains about "You have uncommitted changes in your working tree. Please, commit them first and then run 'git rebase --continue' again." 16.07.03 # sorry, forget the last message - here's the correct one: "tools/configure: needs update You must edit all merge conflicts and then mark them as resolved using git add" 16.07.11 # massa: you cannot rebase --continue if you have uncommitted chanes 16.07.22 # ie if git diff shows something 16.07.50 # ah you might have a conflict 16.07.56 # type git status 16.08.03 # it tells you what is the problem 16.10.33 # if you have a conflict, solve it (ie look at the file, correctr what you want). Then explicitely git add and then commit 16.11.29 Join tchan [0] (~tchan@lunar-linux/developer/tchan) 16.11.49 Quit Moarc (Quit: i znowu NADMUCHAƁ BALONA) 16.12.16 # pamaury: but it did commit already... 16.13.11 # I now did a git add tools/configure - and then it wanted that I commit that again - but without a change id; is that correct? 16.14.27 # Again, ZEN memDFS BSoPO. 16.14.35 # if it's a conflict because of rebase --continue, I think it's correct. In the worse case, we can fix it later 16.14.54 # chrisjj: I think we understood that there is a problem, no need to spam the channel 16.15.28 # Perhaps someone can explain this 'good practice' of code that auto-resets a stuck device, preventing me seeing the info on the screen that might show what went wrong. 16.16.18 # chrisjj: for the FUCKING MILLIONTH time, there is NO INFO TO SEE, all right ? 16.16.44 Join Moarc [0] (~chujko@a105.net128.okay.pl) 16.16.59 # If you can't see the info, how do you know there's none to see? 16.17.05 # you'll get a frozen screen, great 16.17.18 # what kind of info do you get from that ? 16.17.23 # A frozen screen will tell me what the device was doing at the freeze. 16.17.44 # What's the advantage in hiding it. 16.17.45 # sure, as in "it was playing sound" ? 16.18.11 # pamaury: Hmm, I did commit it and then the rebase --continue works - but trying to push it to gerrit leads to a "! [remote rejected] HEAD -> refs/for/master (no changes made)" 16.18.18 # I don't know it was playing sound. I think you don't know either. 16.18.41 # I told you the advantage already: if the reset button is hard to reach, you device auto resets so you don't end up with a brick until you can get access to something small enough to reset the device 16.19.08 # massa: it means you did not change anything compared to previously 16.19.08 # If it was playing sound, the screen will show me which file, where in the file, and (apropos fails in my previous batch of tests) the state of the backlight. 16.19.19 # massa: if you type "git diff HEAD", does it show anything ? 16.19.58 # Can't see much the advantage of making it easy to run the device to freeze again. 16.20.11 # Anyway, the reset button is easy to reach in debugging. 16.20.36 # The information hidden by the reset is currently impossible to reach in debugging. 16.20.41 # chrisjj: the problem is that you consider the situation only from your point of view. The watchdog is not a debug mecanism 16.20.49 Quit _mt_ (Read error: Connection reset by peer) 16.21.04 # if you want to disable it from debug, go get a VM and disable it 16.21.08 # I don't consider the watchdog to be a debug mechanism. 16.21.09 # *for 16.21.20 # pamaury: no it does not show anything - but the file content of tools/configure is the changed one (I removed the -Wno flags for win64) 16.21.20 # It is a production mechanism which is a PITA during debugging. 16.22.12 # How can I disable it with a VM? You've not released the sources that would allow me to build it. 16.22.33 # chrisjj: I'll release them if you do builds 16.22.47 # since at the moment you don't do any build, I'm not wasting any time 16.23.00 # and by the way this is not always true, all your recent tests are on gerrit 16.23.38 # massa: this is strange, you can use "git log" and "git log -p" to review the changes recorded in each commit 16.23.40 # How about releasing them so that anyone can do builds? As we'd expect in this collaborative project. 16.24.43 # g#1483, g#1485, g#1489, g#1490 16.24.49 # 3Gerrit review #1483 at http://gerrit.rockbox.org/r/1483 : 3imx233: add small framework for LED by Amaury Pouly 16.24.49 # 3Gerrit review #1485 at http://gerrit.rockbox.org/r/1485 : 3zen/zenxfi: avoid white flash on backlight on by Amaury Pouly 16.24.50 # 3Gerrit review #1489 at http://gerrit.rockbox.org/r/1489 : 3zen/zenxfi: make backlight sequence closer to that of the OF by Amaury Pouly 16.24.51 DBUG Enqueued KICK fs-bluebot 16.24.51 # 3Gerrit review #1490 at http://gerrit.rockbox.org/r/1490 : 3zen/zenxfi: hack to make white screen go away by Amaury Pouly 16.24.51 # Where's lcd_fix? 16.25.04 # g#1428 16.25.06 # 3Gerrit review #1428 at http://gerrit.rockbox.org/r/1428 : 3imx233: workaround hardware dma bug by Amaury Pouly 16.26.18 Join _mt_ [0] (~MT@2601:482:4402:7b60:a8e4:1c64:d2d1:4ba6) 16.26.19 # arg no, why does cygwin changes the permissions of all files :'( 16.26.22 # Thanks. So why are these not in git? 16.27.50 # because I don't if they work obviously 16.29.14 # Who says g#1428 doesn't work? Not me for sure. 16.29.16 # 3Gerrit review #1428 at http://gerrit.rockbox.org/r/1428 : 3imx233: workaround hardware dma bug by Amaury Pouly 16.32.26 # We this attitude, I would commit a lot of things that just work for me and I wouldn't try to fix the ZEN. I haven't tested it on all imx233 targets yet 16.32.32 # pamaury: actually cygwin does not change it - it does only map the windows permissions to some linux based - and that does often not match very good (especially the x flag) 16.33.37 # massa: I have uploaded g#1499 that tries to fix all casts 16.33.39 # 3Gerrit review #1499 at http://gerrit.rockbox.org/r/1499 : 3Fix dangerous casts by Amaury Pouly 16.33.59 Quit paulk-collins (Ping timeout: 240 seconds) 16.34.05 # did you solve your git problem ? 16.38.08 # massa: I also get the dll init problem 16.39.26 # pamaury, you've mistaken my attitude. I'm as cautious about testing before release as anyone you'll meet. If you tell me RB uses Gerrit Review for QA system test results (despite that https://review.openstack.org/Documentation/intro-quick.html seems to indicate Review is for source inspection and unit test results only), then I'll submit my results. 16.39.43 Quit dfkt (Disconnected by services) 16.40.19 # chrisjj: you can download my compiled simulators here: http://www.mohrenclan.de/rockbox/simulators/ but don't expect anything special, there's no changed code - just compiled in cygwin environment... 16.40.21 # However http://gerrit.rockbox.org/r/#/c/1428/ does not offer me a Review button, despite that it lists me as a Reviewer. 16.41.13 # i've never seen a reviw button, one uses Reply 16.41.22 *** Saving seen data "./dancer.seen" 16.41.38 # pamaury: no, I didn't solve the git problem - with "git log" I can see the commit message, but not on top... 16.43.06 # massa: can you pastebin the output of git log -p HEAD^^^..HEAD 16.44.31 # Ah, silly me for going by the manual in which one uses Review, and no reply Reply exists! :-) https://review.openstack.org/Documentation/intro-quick.html 16.44.50 # Thanks. 16.45.27 # pamaury: http://pastebin.com/HJdZpQM7 16.46.22 # the shown top commit is the UNALIGNED patch - then comes the new one which I want to push to gerrit... 16.47.48 # pamaury: if you solved the pointer casts and still have the loading problem there must be another cause for this 16.47.54 # massa: ok, I guess gerrit is confused and thinks there is no change. What you can do is merge the two commits that touch configure 16.49.17 Join paulk-collins [0] (~paulk@gagarine.paulk.fr) 16.49.33 # assume your log is: 16.49.33 # unaligned 16.49.33 # configure patch #2 16.49.33 # configure patch #1 16.49.33 # then you can do: 16.49.35 # git rebase -i HEAD^^^^ 16.49.37 # it will show a file like this: 16.49.39 # pick configure patch #1 16.49.41 # pick configure patch #2 16.49.43 # pick unaligned 16.49.59 # now replace the second line by: 16.49.59 # squash configure patch #2 16.50.59 # it will create a single commit for the two configure patches and let you edit the message. Make sure you keep only the original change id: 16.50.59 # Change-Id: Ib5dd288c3f9a11be9b66e53e3406eadfa65f7604 16.50.59 # and now you should be done 16.53.17 # massa, thanks for the sim ZIPs. Did you actually manage to run the content yourself?? ISTM the .exe is missing. 16.53.51 # Here's a compare of the latest working ZEN sim here (from rasher) against your ZIP http://i.imgur.com/cKpl1C7.png . 16.54.55 # massa: I'm trying to use gdb to find the problem 16.55.33 # but it's very weird 16.57.12 # chrisjj: your're right - it seems "make fullzip" does not put the simulator in the zip... - I redo with "make zip" and reupload it... 16.59.02 # pamaury: I'm neary sure it has something to do with the generation of the codecs - there must be some DLLMain or similar for the simulators - maybe that's missing for 64-bit builds 16.59.34 # what's really weird is this: this is the backtrace 17.00.00 # http://pastebin.com/gSci4eTN 17.00.36 # it looks like if _FindPESectionByName (a mingw thing) calls strlen() that calls rockbox strlen that calls ci->strlen() 17.00.42 # but of course ci is not initialsied that early 17.00.47 # so might be a symbol problem 17.01.26 Ctcp Ignored 2 channel CTCP requests in 34 minutes and 31 seconds at the last flood 17.01.26 # * pamaury is installing tons of debug symbols 17.03.13 Quit b-t` (Read error: Connection reset by peer) 17.04.32 # pamaury: there seems to be something wrong with the sections in the codec - I think there's still some alignment problem; when I solved my git problem I'll also have a look at it... 17.04.46 # and in codeclib.h there is this comment: 17.04.47 # * so confuses the cygwin linker, and maybe others. These functions need to 17.04.47 # * be implemented elsewhere */ 17.05.16 # I'm going to rename more function like it's done for malloc 17.06.58 # massa, that'll be some strange meaning of the word 'full' with which we are not familiar... :-) 17.08.20 # chrisjj: neither "make zip" nor "make fullzip" does put the simulator files into the zip - I'll have to either do it by hand or investigate in it - but later, I'm currently busy with other things... 17.09.08 # massa: chrisjj: it makes sense, make zip takes .rockbox and zip it basically. And make fullzip does the same but adds font. Simulator executale is not in .rockbox/ and it's probably an oversight 17.10.21 # arg, developping on windows would be easier if windows update didn't take 100% of the CPU and disk 17.11.17 # If the rasher.dk automatic ZIPping doesn't use a custom script, perhaps there's a standard one you've missed. 'fullerzip' perhaps :-) 17.18.02 # pamaury: now it uploads something to gerrit - but I can't see any difference? Maybe I should start allover and do a fresh clone? 17.21.17 Quit _mt_ (Ping timeout: 256 seconds) 17.22.04 # massa: what si the state of configure in your editor ? Do you still have the warnings removed ? 17.22.15 # (and make sure to reload to file to check what is really on disk) 17.22.24 # pamaury: no, they're gone... 17.23.10 # ok, let's try again, slowly this time: 17.23.10 # git rebase -i HEAD^^ 17.23.10 # it should show two commits: unaligned and configure. Change the configure one to edit 17.23.43 # pamaury: no, it now shows other commits (puzzle, dircache and others)... 17.24.12 # massa: you mean the last two commits are not yours ? 17.24.31 # yes 17.25.27 # ok, let's restart from a clean state (you'll loose any change, except what is saved on gerrit already): 17.25.28 # git checkout master 17.25.28 # git reset --hard origin/HEAD 17.26.07 # now go to g#1497: there is a "Download" meny at the top right, it gives you the command to get a copy of those changes. I suggest you use checkout 17.26.09 # 3Gerrit review #1497 at http://gerrit.rockbox.org/r/1497 : 3Renamed defines UNALIGNED to ROCKBOX_UNALIGNED - UNALIGNED is already by Matthias Mohr 17.27.05 # now if you type "git log", you commits should be the last two, right ? 17.27.57 # pamaury: do I have to remove everything at first? Because when I do a "git checkout master" it says "Already on 'master' Your branch is ahead of 'origin/master' by 2 commits. (use "git push" to publish your local commits)" 17.28.48 # * pamaury is confused. If it prints that, then the last two commits shown in git log should be yours 17.29.14 # can you pastebin the output of "git log" ? 17.29.25 # and "git status" 17.29.34 # pamaury: in "git log" the two last are mine - but not in "git rebase HEAD^^^^" 17.29.54 # ah that's normal: the number of "^" determines how far to go 17.30.00 # HEAD^^: go back two in times ;) 17.30.10 # (and it prints them backwards) 17.30.31 # so if you do git rebase -i HEAD^^ then it should print only two which are yours 17.30.36 # yes 17.32.11 # pamaury: So I could now just edit "tools/configure" again and the "commit -a" and paste the old Change-Id to the commit message? 17.32.50 # did you do the interactive rebase ? 17.33.14 # massa: ^ if so, then edit configure, then: 17.33.14 # git commit -a --amend 17.33.40 # it will add your changes to the previous commit and normally you don't have to edit the message (you get the same message as the previous one). 17.33.49 # then git rebase --continue 17.34.18 # (NB: the -a means "add all changes" and --amend means "do not create a new commit but add it to the previous changes") 17.38.39 # pamaury: now it seems to work - including the push to gerrit :-) 17.38.52 Join johnb3 [0] (~johnb2@p57B459F7.dip0.t-ipconnect.de) 17.39.44 # yes :) 17.40.07 # git can be confusing, I suggest you read a bit online how it works, it's quite logical in the end, when you understand what's going on 17.41.54 # yes, it seems I have to do this 8-) 17.42.12 # I'm still compiling the new build with my fix... 17.42.18 # hopefully it will fix the sim 17.45.39 Quit johnb3 (Ping timeout: 240 seconds) 17.45.45 # pamaury: sorry for bugging you - here's my next question: I added myself as reviewer to your g#1499 - but how do I now officially "review"? And add review comments to some files? 17.45.47 # 3Gerrit review #1499 at http://gerrit.rockbox.org/r/1499 : 3Fix dangerous casts by Amaury Pouly 17.46.34 Quit paulk-collins (Ping timeout: 240 seconds) 17.46.50 # and how do I add your change to my list of "watched changes"? 17.47.03 # massa: Reply button 17.47.34 # I don't know about watched changes 17.50.37 # pamaury: I saw your comment directly at the lines of the source - how did you do this? 17.51.03 # * massa is away for a while 17.51.21 # massa: click on a line and press "c" 17.51.28 # massa: the simulator is working now 17.51.32 # I will upload the fix 17.51.42 Nick massa is now known as massa_ (4fe8d32c@gateway/web/freenode/ip.79.232.211.44) 17.51.51 # pamaury: great! 17.55.09 Join johnb3 [0] (~johnb2@p57B459F7.dip0.t-ipconnect.de) 17.58.46 # massa_: I've split the rbwrapper change and put it into another commit 18.12.03 Join paulk-collins [0] (~paulk@gagarine.paulk.fr) 18.12.14 Join MrZeus [0] (~MrZeus@81.144.218.162) 18.13.11 Quit MrZeus_ (Ping timeout: 260 seconds) 18.22.30 Quit johnb3 (Ping timeout: 240 seconds) 18.25.46 Join Bilgus [0] (~WW@cpe-174-102-17-217.cinci.res.rr.com) 18.27.11 # <__builtin> pamaury: can I push g#1502? 18.27.13 # 3Gerrit review #1502 at http://gerrit.rockbox.org/r/1502 : 3Fix compilation of puzzles on Windows by Amaury Pouly 18.29.00 Nick massa_ is now known as massa (4fe8d32c@gateway/web/freenode/ip.79.232.211.44) 18.29.05 # * massa is here again 18.29.42 # __builtin: yes, but make sure you don't push the dependencies with it 18.29.53 Join TheEaterOfSouls [0] (~souls@unaffiliated/theeaterofsouls) 18.30.54 # Build Server message: 3New build round started. Revision 793c041, 255 builds, 16 clients. 18.39.09 # <__builtin> massa: I'll push the change to rbendian.h that you suggested, too 18.41.25 *** Saving seen data "./dancer.seen" 18.42.45 # Build Server message: 3Build round completed after 712 seconds. 18.42.46 # Build Server message: 3Revision 793c041 result: All green 18.42.47 # Build Server message: 3New build round started. Revision 42e54c5, 255 builds, 14 clients. 18.43.52 Join webguest073 [0] (~5029e23c@www.haxx.se) 18.45.37 # pamaury: your patch g#1499 looks good now! How did you find out the problems in g#1502? 18.45.40 # 3Gerrit review #1499 at http://gerrit.rockbox.org/r/1499 : 3Fix dangerous casts by Amaury Pouly 18.45.40 # 3Gerrit review #1502 at http://gerrit.rockbox.org/r/1502 : 3Fix compilation of puzzles on Windows by Amaury Pouly 18.46.31 # __builtin: thumbs up! Forgot about this ;-) 18.48.48 Quit webguest073 (Quit: CGI:IRC (Ping timeout)) 18.49.06 # massa: you mean g#1501 ? 18.49.08 # 3Gerrit review #1501 at http://gerrit.rockbox.org/r/1501 : 3Fix codecs in simulator builds on Windows by Amaury Pouly 18.51.34 # pamaury: yes, of course - I'm a bit confused :-) 18.52.03 # I used gdb, I posted the offending backtrace 18.52.52 # http://pastebin.com/gSci4eTN 18.53.18 # aah yes, it says something about strlen - I didn't believe it had something to do with this - I would've searched on other places... 18.58.28 # Build Server message: 3Build round completed after 943 seconds. 18.58.29 # Build Server message: 3Revision 42e54c5 result: All green 19.04.24 # pamaury: how can I integrate your changes to my local git repo? Is cherry picking at gerrit the correct way? 19.04.45 # massa: yes cherry pick 19.08.35 # pamaury: "git cherry-pick" or use the button "cherry-pick" at your change in gerrit? If the latter, what do I enter as "Cherry Pick to Branch"? 19.09.28 # massa: re your comment: the #undef is necessary 19.09.44 # as for the other methods, yeah we could rename all of them I guess 19.12.34 # forums.rockbox.org is currently unreachable for me - down? 19.14.04 # <__builtin> massa: yeah, it's a known issue 19.17.39 Join Ishka [0] (Ishka@dial-crgvill-66-18-183-244.adamswells.com) 19.17.51 # massa: the downside of renaming everything is that it increases the risk of clashes like the one with strcmp 19.20.01 # pamaury: so you suggest to keep it as is and only rename those methods if it's necessary? 19.20.14 # yeah 19.20.43 # pamaury, does the 'Code-Review +2 Amaury Pouly' that just appeared on g#1428 mean that this change will now go to the master? 19.20.44 # 3Gerrit review #1428 at http://gerrit.rockbox.org/r/1428 : 3imx233: workaround hardware dma bug by Amaury Pouly 19.21.57 # I also don't like the #defines there - it's a shame we cannot use namespaces - would be perfect for that purpose :-) 19.22.29 # pamaury: did you read my second question about the cherry-picking? 19.23.00 # Is there a problem with the 'Themes' site? http://themes.rockbox.org/ just times out. 19.23.45 # Ishka: forums are also unreachable atm... 19.23.51 # <__builtin> they're on the same server 19.23.59 # massa: which one ? 19.24.03 # <__builtin> chrisjj: not necessarily 19.24.35 # pamaury: "git cherry-pick" or use the button "cherry-pick" at your change in gerrit? If the latter, what do I enter as "Cherry Pick to Branch"? 19.24.59 # massa: those are two different things 19.25.14 # massa: don't use the cerry pick button unless you understand it ;) 19.25.28 # massa: copy the text from the Download > Cherry-pick menu 19.26.30 # <__builtin> chrisjj: it means that a dev has reviewed it, but that doesn't automatically submit it to master 19.28.29 # pamaury: I'm looking for a highly visible theme that I can read without glasses. 19.29.50 # massa: I didn't understand your message. "Ishka: forums are also unreachable atm..." 19.31.13 # <__builtin> Ishka: there's an issue with the server that hosts both the forums and the theme site 19.31.14 # Ishka: I meant the site http://forums.rockbox.org are also unreachable at the moment... 19.32.30 # massa Thank you very much, I'll try later. 19.32.33 Join johnb3 [0] (~johnb2@p57B459F7.dip0.t-ipconnect.de) 19.33.21 # massa: I suggest we push g#1501 as-is 19.33.23 # 3Gerrit review #1501 at http://gerrit.rockbox.org/r/1501 : 3Fix codecs in simulator builds on Windows by Amaury Pouly 19.33.43 # after all, it wasn't that hard to find the problem, should that happen in the future 19.33.52 # for other functions 19.34.58 # pamaury: it's O.K. - I'm currently in the progress to build it at my local machine 19.36.33 # if that finished and works I'll give you my +1 score (that's all I have) :-) 19.36.34 Quit johnb3 (Ping timeout: 240 seconds) 19.38.16 # pamaury: btw, I cannot find any make target to put the simulator binaries into the zip - I assume rasher does use his own script for that... 19.39.23 Join mark___ [0] (6d9f5f8c@gateway/web/freenode/ip.109.159.95.140) 19.39.31 # massa: apparently make zip doesn't put it in the zip, I'll look into it 19.39.45 # pamaury: I still have cast warnings - e.g. in simple_viewer.c 19.40.10 # ah, maybe I missed some of them, can you pastebin all of them ? 19.41.26 # I will double check this is not breaking linux or normal builds 19.42.48 # Can anybody tell me why Themes and the Forums seem to have disapeard from rockbox.org 19.43.13 # Because that server is down 19.43.53 # for maintainance? 19.44.11 # <__builtin> it's a bug IIUC 19.46.18 # pamaury: I'll pastebin it - when it finished the build... 19.46.44 # I don't know what ITUC means. Is this somthing that will be sorted in the next few days? 19.47.12 # <__builtin> = "if I understand correctly" 19.47.24 # <__builtin> and no, probably not 19.50.33 # hum apparently g#1501 is breaking normal builds 19.50.35 # 3Gerrit review #1501 at http://gerrit.rockbox.org/r/1501 : 3Fix codecs in simulator builds on Windows by Amaury Pouly 19.52.06 Quit Rondom (Remote host closed the connection) 19.52.47 # pamaury: http://pastebin.com/3XYKkt76 19.54.38 Join Rondom [0] (~rondom@modo.nonmodosedetiam.net) 19.57.14 # pamaury: your're right - my normal ipodvideo build does not work anymore... 19.57.36 # it should be simple enough to fix 19.59.10 # pamaury: but the 64-bit simulator works now :-) 20.00.45 # pys 20.00.49 # bah, sorry 20.01.01 # shell tab-completion doesn't work here... 20.09.08 # * __builtin really needs to refine his git workflow 20.16.33 # massa: I updadted g#1501 20.16.35 # 3Gerrit review #1501 at http://gerrit.rockbox.org/r/1501 : 3Fix codecs in simulator builds on Windows by Amaury Pouly 20.16.42 # can you redo a windows build to check it still works ? 20.16.54 Quit mark___ (Ping timeout: 260 seconds) 20.18.16 # pamaury: o.k. - but it still compiles the old 32-bit version... 20.19.44 Join johnb3 [0] (~johnb2@p57B459F7.dip0.t-ipconnect.de) 20.21.53 # massa: I don't understand what you mean 20.22.29 Quit johnb3 (Client Quit) 20.22.37 # pamaury: I will start a new compile when the current one has been finished (with the old version of the patch) 20.23.13 # ok 20.23.30 # I will long into the other warnings 20.27.37 # pamaury: the 32-bit build (with old patch version) has finished - it produced one (new?) warning: http://pastebin.com/h7pNMxSU 20.29.53 # Build Server message: 3New build round started. Revision 0bdb255, 255 builds, 16 clients. 20.34.23 Quit JdGordon (Ping timeout: 255 seconds) 20.34.39 Join JdGordon [0] (~jonno@rockbox/developer/JdGordon) 20.34.41 # pamaury: even the new version does not build the ipodvide: http://pastebin.com/cyzWTNDz 20.39.04 Join xorly [0] (~xorly@ip-89-176-102-19.net.upcbroadband.cz) 20.40.44 # g#1501 doesn't touch tlsf.c, it must come from the dangerous cast fix 20.40.46 # 3Gerrit review #1501 at http://gerrit.rockbox.org/r/1501 : 3Fix codecs in simulator builds on Windows by Amaury Pouly 20.40.51 # is it a native build or sim ? 20.41.29 *** Saving seen data "./dancer.seen" 20.43.24 # Build Server message: 3Build round completed after 811 seconds. 20.43.25 # Build Server message: 3Revision 0bdb255 result: All green 20.43.26 # Build Server message: 3New build round started. Revision a724dbb, 255 builds, 16 clients. 20.51.18 # pamaury: native build 20.51.43 # pamaury: the simulator build 32- and 64-bits both compile and work 20.53.54 # __builtin, Does a +2 require that also verification has been made? Or has the RB project removed the verification requirement stated in the docs? http://gerrit.rockbox.org/r/Documentation/intro-quick.html#_trying_out_the_change > I 20.54.22 # massa: I have updated g#1499 to fix the compile issues 20.54.24 # 3Gerrit review #1499 at http://gerrit.rockbox.org/r/1499 : 3Fix dangerous casts by Amaury Pouly 20.54.33 # it works on native and linux sim here 20.54.39 # Build Server message: 3Build round completed after 675 seconds. 20.54.40 # Build Server message: 3Revision a724dbb result: 0 errors 1 warnings 20.56.16 # the lua warnings won't go away, I don't plan to fix them 20.56.33 # pamaury: what does the "Cannot Merge" sign mean at g#1499 20.56.35 # 3Gerrit review #1499 at http://gerrit.rockbox.org/r/1499 : 3Fix dangerous casts by Amaury Pouly 20.58.27 # massa: it just means that gerrit has detected a potential conflict if this fix is merged but don't worry about it 20.58.27 # <__builtin> chrisjj: there's no requirement for anything 20.58.35 # it would be a long to explain the details ;) 21.02.13 # basically rockbox decided to have a "no merge" policy, ie no merge commits. Gerrit enforces that but still let you push commits that are not rebased on HEAD but only if it's obvious that there no conflicts (basically it's going to rebase them and then push). But if it thinks the rebase may fail, it show "Cannot merge" and thus you have to explicitely rebase to convince gerrit that it cleany applies 21.07.31 Join wodz [0] (~wodz@89-74-169-198.dynamic.chello.pl) 21.10.25 # pamaury: and gerrit's right - I'm unable to apply the latest version of g#1499: " * branch refs/changes/99/1499/4 -> FETCH_HEAD error: could not apply de9a212... Fix dangerous casts" 21.10.26 # 3Gerrit review #1499 at http://gerrit.rockbox.org/r/1499 : 3Fix dangerous casts by Amaury Pouly 21.10.32 # pamaury: As you have good karma with gdb today, maybe you could help me with g#1389. It crashes because pointers passed to dsp_process become NULL in the beginning of processing. I tried to set watchpoints to find offender but I get "Watchpoint 2 deleted because the program has left the block in which its expression is valid." and then crash again. 21.10.33 # 3Gerrit review #1389 at http://gerrit.rockbox.org/r/1389 : 3Port of picoTTS by Marcin Bukat 21.10.54 # massa: you should do a checkout for g#1499 21.10.56 # 3Gerrit review #1499 at http://gerrit.rockbox.org/r/1499 : 3Fix dangerous casts by Amaury Pouly 21.13.22 # wodz: I'll see what I can do. I'd like to cleanup a bit the windows simulator situation first 21.13.33 # pamaury: sure 21.13.50 # also it didn't involved the dsp stuff ;) 21.14.03 # bam, valgrind doesn't like rockbox simulator :/ 21.14.31 # pamaury: I have a feeling the crash is related to the threading but can't find why. 21.17.09 # massa: could you cleanup g#1463 ? I'd like to push it but there are some ugly comments 21.17.11 # 3Gerrit review #1463 at http://gerrit.rockbox.org/r/1463 : 3nwztools/scsitool: add option to force/specify series by Amaury Pouly 21.17.16 # like #echo sdl-ccopts="${sdlccopts}" 21.17.20 # hum, it behaves differently with sdl threading model 21.17.40 # <__builtin> wodz: I tried to get the sim to work with valgrind a while back 21.17.56 # <__builtin> ... to no avail 21.18.08 # __builtin: It kinda works - you need to pass configure --sdl-threads 21.18.19 # <__builtin> ooh, really? 21.19.21 # <__builtin> nice... :) 21.19.42 # pamaury: I thought I could keep them - to reenable them when needed; but you're right I should clean up it... 21.20.17 # __builtin: this threading mode makes debugging with gdb easier as well as you don't need to pass handle SIGUSR1 nostop to gdb as well :-) 21.21.00 # wodz: I take it that picotts crashes in sim ? 21.21.23 # pamaury: yes 21.22.55 # pamaury: but not always. Sometimes it simply stops outputing samples 21.23.52 # how do I test it ? 21.23.58 # * pamaury doesn't promise anything 21.25.48 # pamaury: are those few #echo the only lines you want to cleanup? 21.25.57 # massa: yes I think 21.26.12 Join _mt_ [0] (~MT@2601:482:4402:7b60:a8e4:1c64:d2d1:4ba6) 21.27.15 # pamaury: get the g#1389, build sim, copy lang_pico folder with voice definitions to simdisk/.rockbox. Copy some text file for tts (first chapter of alice in wonderland is the standard test :-)). Then fire up simulator, in browser long select on txt file (or whatever key is mapped to open context menu) and select open with -> picotts 21.27.17 # pamaury: finished 21.27.17 # 3Gerrit review #1389 at http://gerrit.rockbox.org/r/1389 : 3Port of picoTTS by Marcin Bukat 21.28.46 # wodz: I don't promise I'll look into it tonight but probably more tomorrow 21.28.52 # I've spent the day debugging stuff ;) 21.29.00 # pamaury: thanks anyway :-) 21.29.41 # massa: should I push g#1496 then ? 21.29.43 # 3Gerrit review #1496 at http://gerrit.rockbox.org/r/1496 : 3Fix configure for windows simulator builds in cygwin / mingw cross-compiling environm... by Matthias Mohr 21.31.03 # just noticed that if I use rockbox simulator in a windows VM, the sound is crap, as if it was constantly underflowing, maybe a problem with sdl audio 21.31.50 # pamaury: yes, and g#1497 also 21.31.52 # 3Gerrit review #1497 at http://gerrit.rockbox.org/r/1497 : 3Renamed defines UNALIGNED to ROCKBOX_UNALIGNED - UNALIGNED is already by Matthias Mohr 21.32.26 # Build Server message: 3New build round started. Revision 955be5b, 255 builds, 16 clients. 21.33.16 # cool thanks :) 21.33.31 # and I'll push g#1501 unless you object ? 21.33.33 # 3Gerrit review #1501 at http://gerrit.rockbox.org/r/1501 : 3Fix codecs in simulator builds on Windows by Amaury Pouly 21.38.47 # rasher: ping 21.40.08 Join girafe [0] (~girafe@LFbn-1-11729-221.w2-7.abo.wanadoo.fr) 21.41.35 Nick __builtin is now known as _ (~xray@rockbox/developer/builtin) 21.42.04 Nick _ is now known as Guest93277 (~xray@rockbox/developer/builtin) 21.42.28 Quit _mt_ (Read error: Connection reset by peer) 21.43.37 Part Guest93277 ("http://quassel-irc.org - Chat comfortably. Anywhere.") 21.45.06 # Build Server message: 3Build round completed after 760 seconds. 21.45.07 # Build Server message: 3Revision 955be5b result: All green 21.45.08 # Build Server message: 3New build round started. Revision d984725, 255 builds, 16 clients. 21.45.13 Join _mt_ [0] (~MT@2601:482:4402:7b60:a8e4:1c64:d2d1:4ba6) 21.50.55 Join __builtin [0] (~xray@rockbox/developer/builtin) 21.56.59 # Build Server message: 3Build round completed after 712 seconds. 21.57.00 # Build Server message: 3Revision d984725 result: All green 21.58.10 # pamaury: of course you can push it... 22.00.01 # * pamaury thinks we need to add a win32 and win64 builds to the build server, at least one of each 22.00.33 # pamaury: I currently fight again with (or better against?) git... :-( 22.00.46 # * massa thinks the same as pamaury... 22.01.47 # massa: what is your git problem ? 22.02.43 # chrisjj: I updated the simulator build binaries for ipod video and zen in win32 and win64 version at http://www.mohrenclan.de/rockbox/simulators/ 22.03.08 # pamaury: well, it's a bit difficult to explain - but I try :-) 22.04.08 Quit wodz (Quit: Leaving) 22.04.47 Quit michaelni (Ping timeout: 245 seconds) 22.04.52 # Build Server message: 3New build round started. Revision 16d1788, 255 builds, 16 clients. 22.05.02 # I initially created my local rockbox repository in a directory rockbox.gitorig - from that I also did the gerrit pushes 22.06.08 # then I created another directory rockbox.gitdev and made a clone of the rockbox.gitorig (a local clone) 22.07.33 # in that directory I cherry-picked all the changes of your gerrit # - and I also used that directory to build (in subdirectories) 22.08.34 # now I'm unable to get them both to the same (last) rockbox change. 22.09.07 # rockbox.gitorig seems to be fine - at least I can see all commits with "git log" 22.10.31 # massa: to get back normal, I suggest: (warning: you'll loose any change): 22.10.31 # git reset --hard HEAD # delete any changes compared to current HEAD 22.10.31 # git checkout master # make sure you are on master branch 22.10.31 # git reset --hard origin/HEAD # make sure you are at the same commit as remote 22.10.34 # # and then possibly update: 22.10.37 # git pull --rebase 22.14.27 # I already did "git reset --hard" but without additional parameter 22.14.42 # and why do I need several reset commands? 22.15.47 # if you reset --hard, you get back to HEAD but 1) HEAD might be different from remote HEAD (for example if you cherry-pick commits). So in general you want git reset --hard origin/HEAD 22.16.30 # BUT, that only works if you have to local changes, thus you need the first reset --hard. Also if you did a checkout previously, you're not on master anymore, thus the checkout to master to make sure you're back on the right branch 22.16.47 # Could you recommend any good documentation about that git basics? Not the basic basics - I know what revision control etc. is - I use it every day, with svn and hg but I'm really confused with that git handling... 22.17.12 Join michaelni [0] (~michael@213-47-41-20.cable.dynamic.surfer.at) 22.18.36 # Build Server message: 3Build round completed after 824 seconds. 22.18.37 # Build Server message: 3Revision 16d1788 result: All green 22.19.26 # massa: as I recall: https://www.atlassian.com/git/tutorials/inspecting-a-repository is quite nice with picture 22.19.53 # (there is a menu on the left with different topics) 22.20.25 # otherwise https://git-scm.com/book/en/v2/ 22.24.52 # pamaury: thanks - will have a deeper look at them - could it be that the windows TortoiseGit is not the best client to use? I'm very used to the Tortoise-Clients (long ago for CVS, now SVN and HG) - but the GIT version seems to be way behind the other... 22.26.14 # Normally I do the daily work with a GUI and only special things with command line - but here it seems that it's better to do everything with command line 22.26.16 # massa: I don't know, I've only ever used the command line git 22.26.28 # git is not very UI friendly imo 22.27.43 # * __builtin sometimes uses `git gui' 22.27.45 # <__builtin> and `gitk' 22.28.39 Quit shmibs (Quit: leaving =o) 22.29.39 Quit TheEaterOfSouls (Quit: ChatZilla 0.9.93 [Firefox 50.1.0/20161208153507]) 22.29.44 # pamaury: currently only g#1499 is need as cherry pick now? 22.29.46 # 3Gerrit review #1499 at http://gerrit.rockbox.org/r/1499 : 3Fix dangerous casts by Amaury Pouly 22.30.51 # massa: yes 22.33.35 Join shmibs [0] (~shmibs@shmibbles.me) 22.40.30 Quit shmibs (Quit: leaving =o) 22.41.30 *** Saving seen data "./dancer.seen" 22.51.12 # pamaury: win64 built has finished - no more warnings and it still works :-) 22.53.33 # good :) 23.00.23 Quit petur (Quit: Leaving) 23.00.46 # pamaury: win32 built has also finished - no warnings and works! 23.06.42 # massa: cool thanks 23.07.12 # I'll still wait a bit more, I sent an email on the ML about it, to make sure other have a chance at reviewing it since it touches many things 23.07.59 Quit lebellium (Quit: ChatZilla 0.9.93 [Firefox 50.1.0/20161208153507]) 23.17.48 # pamaury: native ipodvideo build has also finished without warnings - I'm now tranferring it to my device 23.24.55 # pamaury: the native build does _NOT_ work - it crashes directly after the start :-( 23.27.37 # pamaury: message is: data abort at 0008BF90 (0) 23.27.54 # pc: 0008BF90 sp:4060AE58 23.28.00 # A: 0008D21C 23.28.08 # A: 000347C4 23.28.20 # A: 00033344 23.29.09 # bt end 23.31.10 # massa: without symols the backtrace is useless though. Did you make sure that HEAD works (ie without g#1499) ? 23.31.12 # 3Gerrit review #1499 at http://gerrit.rockbox.org/r/1499 : 3Fix dangerous casts by Amaury Pouly 23.32.37 Quit skapazzo (Quit: leaving) 23.32.48 Quit JanC (Remote host closed the connection) 23.34.07 # massa: I'll try it on my fuze+ to see if I can find the problem 23.35.19 # pamaury: that's what I do now - compile a HEAD ipodvideo build - we'll see (but that's the last action for today - then I'll leave and unfortunately I don 23.35.34 # I don't think I'll find much time next week... 23.35.52 Quit cc___ (Ping timeout: 256 seconds) 23.36.04 # (including next weekend - I'm skiing ;-) 23.37.02 # pamaury: can I force a backtrace / core or similar at the device? 23.37.28 # massa: not sure what you mean 23.38.04 # I mean can rockbox produce a crashlog or a core file or something similar? 23.38.17 # no, just a backtrace 23.38.42 # which is only useful if you have the rockbox.map file, which you did but I stupidly ask you to try another build ;) 23.38.57 # let me see if I can reproduce it on my fuze+, that would be the simplest way to fix it 23.40.46 # it works on the fuze+ :-/ 23.41.48 # massa: what could help is if you redo the fault build, post the backtrace and upload somewhere rockbox.map 23.43.43 # pamaury: the HEAD build works :-) 23.43.57 # pamaury: O.K. I'll redo the build... 23.44.12 # ok good, at least we know g#1499 is faulty 23.44.13 # 3Gerrit review #1499 at http://gerrit.rockbox.org/r/1499 : 3Fix dangerous casts by Amaury Pouly 23.44.33 # or my build was faulty... 23.44.45 # in doubt, make clean 23.45.10 Join JanC [0] (~janc@lugwv/member/JanC) 23.48.32 # pamaury: I always "make clean" to make sure everything got build ;-) 23.48.53 # did you use Rolo when you put the new firmware ? 23.52.37 Quit n3m9 (Read error: Connection reset by peer) 23.55.40 Quit paulk-collins (Quit: Leaving) 23.57.49 # pamaury: Do I use what? :-) I create a Zip and unzip it at the device which is connect as mass storage device... 23.58.42 # massa: rolo is when you see the message "the firmware has been changed, do you want to Reload" (ROLO=reload)