| 00:27:20 | | Quit lebellium (Quit: Leaving) |
| 00:36:20 | | Quit ZincAlloy (Quit: Leaving.) |
| 00:36:21 | *** | Saving seen data "./dancer.seen" |
| 00:36:57 | foolsh | __builtin: Well wolf3d and duke3d work very well on fuze+, even sound, I'll give quake a test run |
| 01:00 |
| 01:34:32 | __builtin | Good to know |
| 01:34:47 | __builtin | For quake make sure you get the latest patch set |
| 01:35:32 | foolsh | It's weird, cherry picking Quake into my local branch causes lots of warnings touching SDL wold3d duke3d and of course quake |
| 01:35:49 | __builtin | Don't cherry pick it alone |
| 01:36:13 | __builtin | It relies on its parent commits |
| 01:36:30 | __builtin | Best bet is probably to checkout |
| 01:36:35 | foolsh | ah well of course |
| 01:37:13 | __builtin | Actually... don't checkout |
| 01:37:33 | __builtin | Cherry pick g#1794 and then g#1832 |
| 01:37:36 | fs-bluebot | Gerrit review #1794 at http://gerrit.rockbox.org/r/1794 : Add proper float formatting to vuprintf by Michael Sevakis |
| 01:37:36 | fs-bluebot | Gerrit review #1832 at http://gerrit.rockbox.org/r/1832 : Quake! by Franklin Wei |
| 01:37:56 | __builtin | It won't compile if you pull in g#2105 |
| 01:37:57 | fs-bluebot | Gerrit review #2105 at http://gerrit.rockbox.org/r/2105 : system-arm: log PC address via ipod6g piezo on fault (HACK) by Franklin Wei |
| 01:38:03 | foolsh | Yeah I thought as much when I looked at the related changes |
| 02:00 |
| 02:00:01 | __builtin | ok, quake seems to consistently run at 8.6FPS |
| 02:00:06 | __builtin | let's see if I can optimize that |
| 02:36:22 | *** | No seen item changed, no save performed. |
| 03:00 |
| 03:09:01 | | Join krabador [0] (~krabador@unaffiliated/krabador) |
| 03:09:18 | foolsh | __builtin: Quake on fuze+ compiles with a shit metric ton of warnings but no errors, starts normally, exits at first screen with message "Error: Draw_Pic: bad coordinates" then goes to a screen that says "Plugin returned error #leak-file-handles" |
| 03:09:26 | foolsh | I'll get you more to go on later |
| 03:10:25 | __builtin | sounds like an issue with the screen size |
| 03:11:53 | __builtin | if you want a quick but hacky fix, go to vid_sdl.c and set BASEWIDTH=320 and BASEHEIGHT=200 |
| 03:11:58 | __builtin | SDL will scale it automatically |
| 03:12:10 | __builtin | (it'll be slow as all heavens, of course, but it ought to run) |
| 03:30:47 | foolsh | __builtin: actually it runs at a decent speed on fuze+, crashed once something about "drawing plane" and buttons are buggy it's stuck in auto fire mode, wasted all my ammo right at the beginning :) but how ever over all impressive feat for a DAP |
| 03:31:43 | __builtin | hmm... |
| 03:31:56 | __builtin | if you press the fire button a few times it should get unstuck |
| 03:40:22 | | Join massiveH [0] (~massiveH@ool-18e4e27c.dyn.optonline.net) |
| 03:41:26 | foolsh | Was thinking for wold3d and duke3d SOUND_ENABLE could be #ifdef'ed out for targets with low ram but with color lcds, instead of just omitting them |
| 03:44:05 | fs-bluebot | Build Server message: New build round started. Revision 3e2b50e, 280 builds, 9 clients. |
| 03:44:40 | __builtin | foolsh: sure, for wolf3d |
| 03:44:58 | __builtin | What would the threshold be for sound? |
| 03:47:11 | | Quit krabador (Remote host closed the connection) |
| 04:00 |
| 04:36:23 | *** | Saving seen data "./dancer.seen" |
| 04:37:54 | fs-bluebot | Build Server message: New build round started. Revision 5d05b9d, 280 builds, 9 clients. |
| 04:54:49 | foolsh | __builtin: I can spend sometime decreasing the allocated ram on my fuze+ til I hit the limit |
| 04:55:04 | __builtin | Sounds good |
| 04:55:18 | __builtin | it's going to be a very rough estimate no matter what |
| 05:00 |
| 05:13:25 | | Quit TheSeven (Disconnected by services) |
| 05:13:34 | | Join [7] [0] (~quassel@rockbox/developer/TheSeven) |
| 05:23:48 | fs-bluebot | Build Server message: New build round started. Revision b0e9bb1, 280 builds, 9 clients. |
| 05:24:47 | __builtin | that ought to mitigate any worries about core memory usage |
| 05:51:18 | fs-bluebot | Build Server message: New build round started. Revision 9c17734, 280 builds, 10 clients. |
| 06:00 |
| 06:36:27 | *** | Saving seen data "./dancer.seen" |
| 07:00 |
| 07:17:22 | | Quit massiveH (Quit: Leaving) |
| 08:00 |
| 08:25:41 | | Join dys [0] (~dys@tmo-111-100.customers.d1-online.com) |
| 08:31:51 | | Quit [7] (Ping timeout: 264 seconds) |
| 08:35:16 | | Join TheSeven [0] (~quassel@rockbox/developer/TheSeven) |
| 08:36:31 | *** | Saving seen data "./dancer.seen" |
| 09:00 |
| 09:54:57 | | Join lebellium [0] (~lebellium@89-92-69-110.hfc.dyn.abo.bbox.fr) |
| 10:00 |
| 10:01:05 | | Join ZincAlloy [0] (~Adium@2a02:8108:9440:dfc:54c1:7e76:1ad8:2d7d) |
| 10:29:25 | | Quit Huntereb (Ping timeout: 258 seconds) |
| 10:36:32 | *** | Saving seen data "./dancer.seen" |
| 10:53:23 | | Join Huntereb [0] (~Huntereb@d-69-161-102-241.va.cpe.atlanticbb.net) |
| 11:00 |
| 11:36:53 | Bilgus | morning |
| 11:41:24 | | Quit paulk-leonov (Ping timeout: 268 seconds) |
| 11:43:44 | | Join paulk-leonov [0] (~paulk-leo@leonov.paulk.fr) |
| 11:49:09 | fs-bluebot | Build Server message: New build round started. Revision 55eb1c5, 280 builds, 10 clients. |
| 12:00 |
| 12:12:37 | fs-bluebot | Build Server message: Build round completed after 652 seconds. |
| 12:12:38 | fs-bluebot | Build Server message: Revision 6392e05 result: 2500 errors 0 warnings |
| 12:21:59 | amdj | Who do we fire foam missiles at, for breaking the build? |
| 12:36:33 | *** | Saving seen data "./dancer.seen" |
| 12:41:53 | fs-bluebot | Build Server message: New build round started. Revision 14d91ca, 280 builds, 11 clients. |
| 12:52:20 | | Quit ZincAlloy (Quit: Leaving.) |
| 13:00 |
| 13:02:47 | Bilgus | it depends whens the last time you committed? |
| 13:03:33 | Bilgus | speachy, but technically its my fault for pushing it |
| 13:04:28 | gevaerts | Some of the people who broke the build once haven't been seen in years |
| 13:06:31 | * | gevaerts doesn't know if there's a causal link |
| 13:09:06 | Bilgus | so the issue is genlang isn't being run before the apps / plugins are built therefore lang_enum.h is missing |
| 13:09:52 | Bilgus | I tried moving genlang up in the list in root.make but its still not happening at the right time |
| 13:11:34 | Bilgus | hmm make.inc mentions lang.h so maybe I can co-opt it |
| 13:16:22 | | Quit dys (Ping timeout: 268 seconds) |
| 14:00 |
| 14:15:22 | | Join ZincAlloy [0] (~Adium@2a02:8108:9440:dfc:7896:baad:ba3c:acfd) |
| 14:36:37 | *** | Saving seen data "./dancer.seen" |
| 15:00 |
| 15:13:34 | | Join dys [0] (~dys@tmo-100-27.customers.d1-online.com) |
| 15:32:52 | | Join speachy [0] (d102414d@209.2.65.77) |
| 15:33:29 | speachy | Well, time to figure out why Make isn't generating proper dependencies for the language stuff. :/ |
| 15:33:45 | Bilgus | uh yeah I just hacked a fix to get it to build lol |
| 15:34:06 | Bilgus | its not getting generated till half way through the plugins |
| 15:35:08 | Bilgus | in lang.make DUMMY := $(shell touch $(BUILDDIR)/lang_enum.h) lol |
| 15:37:01 | speachy | Just added lang_enum.h to the clean target too |
| 15:37:20 | speachy | I want to say I ran into this exact problem a while back and fixed it. |
| 15:37:47 | speachy | the dependency list doesn't have any reference to lang_enum.h, which is .. wrong IMO. |
| 15:40:14 | Bilgus | make is not my favorite language |
| 15:42:00 | | Quit speachy (Remote host closed the connection) |
| 15:42:48 | | Join speachy [0] (d102414d@209.2.65.77) |
| 15:44:45 | speachy | I think I know what's going on.. |
| 15:45:08 | speachy | when the dependencies are generated, PLUGIN is not defined. |
| 15:45:45 | speachy | let's see.. |
| 15:50:28 | speachy | better, but the puzzles plugin remains pathological |
| 15:52:16 | speachy | because it explicitly includes rbcompay.h on the cmdline.. |
| 15:56:07 | speachy | so mkdepfile won't pick it up |
| 16:00 |
| 16:01:36 | speachy | found the problem. testing.. |
| 16:03:00 | Bilgus | well you certainly got it faster than I |
| 16:06:39 | speachy | g#2121 |
| 16:06:42 | fs-bluebot | Gerrit review #2121 at http://gerrit.rockbox.org/r/2121 : build: Fix puzzles build due to missing dependency by Solomon Peachy |
| 16:06:55 | * | speachy loves make |
| 16:07:27 | speachy | it's nothing more than a dependency resolver, so if something doesn't work, it's because the deps aren't specified correctly. |
| 16:08:47 | fs-bluebot | Build Server message: New build round started. Revision 3cb272c, 280 builds, 11 clients. |
| 16:09:11 | | Quit michaelni (Ping timeout: 245 seconds) |
| 16:09:13 | speachy | okay, let's see how much red is left after this.. |
| 16:09:16 | gevaerts | The issue tends not to be really make but how messy the problem is |
| 16:09:48 | speachy | that's why everything that seeks to replace make ends up being worse, because the problem isn't with make itself. :) |
| 16:10:01 | * | gevaerts nods |
| 16:11:43 | * | speachy hears his build server's fans spin up |
| 16:13:00 | * | gevaerts fondly remembers the time when he had a build client on his phone :) |
| 16:13:07 | speachy | time to go start cooking brunch. from-scratch biscuits & gravy. |
| 16:22:43 | | Join michaelni [0] (~michael@213-47-41-20.cable.dynamic.surfer.at) |
| 16:36:39 | *** | Saving seen data "./dancer.seen" |
| 16:39:40 | | Quit pixelma (Quit: .) |
| 16:39:40 | | Quit amiconn (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.) |
| 16:40:43 | | Join amiconn [0] (jens@rockbox/developer/amiconn) |
| 16:40:43 | | Join pixelma [0] (marianne@rockbox/staff/pixelma) |
| 16:41:07 | speachy | a lot less red, yay. |
| 16:42:25 | * | speachy will fix the rest after the guests leave. |
| 16:58:15 | * | gevaerts is annoyed by that sim snprintf warning |
| 17:00 |
| 17:02:39 | | Quit foolsh (Ping timeout: 272 seconds) |
| 17:12:24 | fs-bluebot | Build Server message: New build round started. Revision 3665eec, 280 builds, 11 clients. |
| 17:25:06 | | Quit Huntereb (Quit: See ya!) |
| 17:26:58 | | Join Huntereb [0] (~Huntereb@d-69-161-102-241.va.cpe.atlanticbb.net) |
| 17:35:11 | | Quit Huntereb (Quit: See ya!) |
| 17:38:32 | | Join Huntereb [0] (~Huntereb@d-69-161-102-241.va.cpe.atlanticbb.net) |
| 17:41:00 | | Quit noobineer (Remote host closed the connection) |
| 18:00 |
| 18:36:41 | *** | Saving seen data "./dancer.seen" |
| 19:00 |
| 19:01:45 | fs-bluebot | Build Server message: New build round started. Revision d1b8b39, 280 builds, 11 clients. |
| 19:03:21 | speachy | okay, it broke on non-RTC targets. that's fixed −− the strings needed to be moved from rtc-only context to global. English fixed, but I don't know if there's an automagic way to fix up the other languages too. |
| 19:03:31 | speachy | really don't want to do it by hand. |
| 19:05:35 | __builtin | speachy: what are you trying to do? |
| 19:07:27 | __builtin | is it just what you did in HEAD, but for all languages? |
| 19:07:48 | __builtin | looks like an awk script should be able to do that |
| 19:09:44 | gevaerts | There's tools/langtool.pl that might do that, but I'm not sure |
| 19:11:11 | speachy | yes, waht I did in head. |
| 19:11:46 | speachy | for about 18-odd strings, make the rtc: target the * target. |
| 19:12:06 | * | __builtin is working on it |
| 19:12:11 | __builtin | seems simple enough |
| 19:23:06 | __builtin | speachy: is this what you want? https://pastebin.com/uer2hEP3 |
| 19:24:50 | speachy | it looks like you updated the entire rtc: subset rather than just the stuff I changed to get the build fixed. |
| 19:25:46 | __builtin | ah, just those? |
| 19:26:09 | speachy | though it's possible I may have missed some. |
| 19:26:23 | __builtin | what are the IDs you want changed? |
| 19:27:25 | speachy | LANG_TIMEFORMAT LANG_12_HOUR_CLOCK LANG_24_HOUR_CLOCK LANG_MONTH_* VOICE_OCLOCK VOICE_PM VOICE_AM VOICE_OH |
| 19:27:32 | __builtin | alright |
| 19:27:54 | speachy | so 19 in all. |
| 19:29:41 | speachy | enough to be able to voice the date and time, and configure the time format. |
| 19:32:30 | __builtin | how's https://pastebin.com/J4v76ER2 |
| 19:33:27 | speachy | looks good |
| 19:33:35 | speachy | only scanned the first (cz) |
| 19:35:24 | speachy | the errors went to warnings. which may be resolved by the update you're cooking |
| 19:36:43 | speachy | but the fm rec went green too, but player went red. wonder if that's a builder quirk or something. |
| 19:36:56 | __builtin | g#2123? |
| 19:36:57 | fs-bluebot | Gerrit review #2123 at http://gerrit.rockbox.org/r/2123 : Move date/time strings out of RTC context by Franklin Wei |
| 19:38:23 | speachy | looks sane to me |
| 19:40:35 | fs-bluebot | Build Server message: New build round started. Revision ee659d4, 280 builds, 11 clients. |
| 19:40:59 | speachy | and the fans spin back up... |
| 19:42:09 | * | __builtin realized that his buildclient was running as root |
| 19:42:19 | __builtin | so no sparta this time |
| 19:43:36 | speachy | yikes. |
| 19:44:06 | | Join petur [0] (~petur@rockbox/developer/petur) |
| 19:44:08 | speachy | hmm. I think I'll set up another build client |
| 19:45:52 | * | __builtin is working on a docker image to containerize the whole thing |
| 19:46:00 | __builtin | it's a real pita getting a dev environment set up |
| 19:46:47 | speachy | the only annoyance is building the toolchains |
| 19:47:51 | __builtin | once I get this working, it should be a matter of 1 or two shell commands |
| 19:48:54 | __builtin | right now the mipsel build is failing because of a conflict installing documentation |
| 19:52:10 | speachy | would be really nice to tie the builders into gerrit so all patches can get build-tested before committing. |
| 19:54:20 | __builtin | I had that idea many moons ago... got shot down for one reason or another |
| 19:54:37 | __builtin | first thing that comes to mind is arbitrary code execution by anyone who pushes a gerrit patch |
| 19:55:47 | speachy | true that. |
| 19:56:32 | __builtin | I guess it could be made developers-only |
| 19:56:42 | __builtin | but even then it'd be hard to actually implement |
| 19:57:01 | __builtin | (and would require action by the Swedes, who hvae largely removed themselves from the project) |
| 19:58:16 | speachy | gerrit rules could help a lot with that distinction. only blessed users could get stuff autobuilt (or trigger an autobuild after a review) |
| 19:58:42 | __builtin | yeah, something like that |
| 19:58:44 | speachy | but yeah, with the current development velocity it's hard to justify that effort |
| 19:59:19 | __builtin | we don't lose that much from fixing the things after they're pushed |
| 20:00 |
| 20:00:28 | speachy | all green except for the player targets! |
| 20:01:03 | __builtin | crap, I didn't actually kill my client |
| 20:02:36 | speachy | I'm going to commit g#2073 and g#2074 once I get the second builder going. |
| 20:02:39 | fs-bluebot | Gerrit review #2073 at http://gerrit.rockbox.org/r/2073 : FS #7912: Voice enable the chessbox plugin by Solomon Peachy |
| 20:02:39 | fs-bluebot | Gerrit review #2074 at http://gerrit.rockbox.org/r/2074 : FS #7705: Voice enable battery bench plugin by Solomon Peachy |
| 20:12:49 | speachy | okay, now to do a manual build to make sure it's sane.. |
| 20:14:20 | fs-bluebot | Build Server message: New build round started. Revision 4edaf3f, 280 builds, 11 clients. |
| 20:14:54 | speachy | okay, two builders going now |
| 20:15:00 | | Quit michaelni (Ping timeout: 245 seconds) |
| 20:16:51 | __builtin | hmm... who/what builds the latex manuals? |
| 20:17:30 | gevaerts | My clients do, although there seems to be something wrong with the html one |
| 20:17:47 | gevaerts | As in, the build succeeds, but uploading the log fails with http 500 |
| 20:19:01 | __builtin | which packages do I need for that? |
| 20:19:35 | gevaerts | Several :) |
| 20:20:30 | __builtin | might be best to stay away then |
| 20:20:33 | gevaerts | I don't know how current https://www.rockbox.org/wiki/LatexInstallation is |
| 20:20:39 | gevaerts | No, go for it! |
| 20:20:56 | gevaerts | It's one of the things where we have very little redundancy |
| 20:21:30 | speachy | I'll get my builders set up for the manuals too |
| 20:22:06 | speachy | blows up for me with 'multirow.sty' not found |
| 20:22:43 | speachy | (fedora, texlive installed and working for other things) |
| 20:23:04 | gevaerts | on debian stretch, that's in texlive-latex-extra |
| 20:24:40 | speachy | fedora it's texlive-multirow |
| 20:25:57 | __builtin | alright, I've got a docker image for sh, m68k and arm going |
| 20:26:01 | __builtin | and sdl/latex |
| 20:26:14 | gevaerts | Still missing mips and android then :) |
| 20:26:27 | __builtin | yeah, those are annoying |
| 20:26:45 | speachy | both of mine have mips-native and mips-hosted |
| 20:27:03 | speachy | don't recall those being particualrly difficult since the move to gcc 4.9 |
| 20:27:47 | __builtin | they seem to conflict with other toolchains |
| 20:27:54 | gevaerts | oh, and application arm |
| 20:30:59 | | Quit speachy (Remote host closed the connection) |
| 20:31:26 | | Join speachy [0] (d102414d@209.2.65.77) |
| 20:36:44 | *** | Saving seen data "./dancer.seen" |
| 20:37:21 | speachy | on the next build my builders will do the latex manuals |
| 20:39:12 | | Join michaelni [0] (~michael@213-47-41-20.cable.dynamic.surfer.at) |
| 20:40:40 | gevaerts | Great :) |
| 20:41:18 | speachy | the archos builds are being flaky. from one to the next it's inconsistent. |
| 20:44:44 | fs-bluebot | Build Server message: New build round started. Revision 78892b5, 280 builds, 11 clients. |
| 20:45:23 | Bilgus | speachy you dun blowed up the rockbox!! |
| 20:46:37 | speachy | strings. :/ |
| 20:46:38 | Bilgus | :P its good for 'everyone' to get a little hands on there was a couple months there that nothing even got committed |
| 20:47:36 | speachy | though looking at the recent commits, Quake added 3-4k by itself. :) |
| 20:48:35 | gevaerts | We had years of debate on whether adding doom was a mistake, and then __builtin came along :) |
| 20:48:51 | __builtin | speachy: to the core? |
| 20:49:25 | __builtin | The increase is limited to quake targets which already have massive ram |
| 20:49:45 | speachy | if one believes the deltas on the dev builds page |
| 20:50:20 | speachy | even clip+ etc saw a 4.4k bump, and there's no way that thing runs quake.. |
| 20:50:55 | gevaerts | Hmmm |
| 20:51:11 | Bilgus | speaking of I fixed all those new warnings expept 1 the rtc one as wodz said it should be sanitized properly and I've no clue what properly is |
| 20:51:26 | Bilgus | one of you wanna take a hack at it |
| 20:51:40 | speachy | ah, I saw the subsquent build shrunk. the printf float thing. |
| 20:52:30 | gevaerts | Yes, but that's still 4K to core for a plugin that I think is of fairly limited appeal |
| 20:52:31 | Bilgus | #g1994 way my try |
| 20:52:42 | Bilgus | ahem was my try |
| 20:53:48 | __builtin | I should be able to move it into the plugin |
| 20:53:52 | Bilgus | IDK geaverys I've seen a bunch of people very interested in those damn games lol |
| 20:54:16 | Bilgus | I must be drunk typing |
| 20:54:25 | gevaerts | Bilgus: yes, those people are loud :) |
| 20:54:56 | speachy | eh, the only game anyone needs is snake! |
| 20:55:21 | Bilgus | and the damn bullet proof nokia to run it on |
| 20:55:40 | Bilgus | So I thake it no takers on g#1994? |
| 20:55:42 | fs-bluebot | Gerrit review #1994 at http://gerrit.rockbox.org/r/1994 : Fix Truncation warning screens.c->set_time_screen by William Wilgus |
| 20:55:55 | gevaerts | Anyway, I don't know how important 4K still is on devices that are relevant today |
| 20:56:40 | gevaerts | Bilgus: oh, I didn't see that you were working on that... |
| 20:57:11 | gevaerts | I did 3665eecf earlier today to fix that, with a different approach |
| 20:57:19 | speachy | 4k probably isn't relevant for anything but the old HWCODEC Archos units.. |
| 20:57:42 | Bilgus | I went thru the last time I fixed all the new warnings and fixed all the warnings for the next gcc version for when someone updated the build system again |
| 20:58:35 | Bilgus | geaverts a different approach was needed as wodz said the rtc should be sanatized properly but Ive no idea what properly should be |
| 20:58:37 | gevaerts | I didn't like the "fix numbers locally" approach because it's extra code just because the compiler doesn |
| 20:58:45 | speachy | I will say this though −− full accessibility/voice is probably Rockbox's best "killer feature" |
| 20:58:54 | gevaerts | t know those numbers have a limited range |
| 20:59:24 | Bilgus | yeah speacht rb is the only accessible dap firmware for the blind |
| 21:00 |
| 21:00:10 | gevaerts | Bilgus: as I said, I did 3665eecf earlier today because I didn't know you were working on it. I went for increasing the buffer for hosted builds (where presumably 40 bytes extra stack is not very much) |
| 21:00:50 | Bilgus | I'm up for whatever as long as that damn warning goes away lol |
| 21:01:10 | gevaerts | Feel free to revert that (or should loudly so I revert it) if I'm on the wrong track there |
| 21:02:32 | gevaerts | speachy: it definitely was a killer feature, but as with all else, smartphones have probably taken over for most people |
| 21:02:47 | Bilgus | up to you wods doesn't like my approach either but I've grown tired of the yellow lol |
| 21:03:00 | gevaerts | OK, up to wodz then :) |
| 21:03:12 | Bilgus | the blind don't like touchscreens |
| 21:03:27 | gevaerts | Some might not |
| 21:03:41 | Bilgus | for that matter neither do I, I really like tactile feedback |
| 21:03:54 | speachy | exactly why the fuze+ was so gawdawful. |
| 21:03:57 | * | __builtin grumbles |
| 21:04:04 | __builtin | "killer feature" = "quake" |
| 21:04:11 | Bilgus | yeah I own one and its my trucks DAP |
| 21:04:23 | speachy | it's hard to play quake on a 160x120 monochrome OLED screen. Just saying. :P |
| 21:04:47 | Bilgus | I really like lua as that allows the people to make their own stuff :p |
| 21:04:49 | gevaerts | One of my coworkers is blind. He basically is a cyborg with an integrated iphone and apple watch |
| 21:05:18 | gevaerts | __builtin: usually people take the "killer" in "killer feature" as something other than "shooting things" :) |
| 21:05:23 | speachy | I cheered inside when the screen on my fuze+ broke. |
| 21:10:18 | Bilgus | gevaerts, aren't you the one with the DAP international museum? |
| 21:10:41 | gevaerts | I don't have *that* many |
| 21:10:58 | Bilgus | I bet gevaerts wouldn't cheer if one of his lesser favorite targets got broken |
| 21:11:07 | speachy | keep seeing connection stalled messages when the builders are trying to upload the manuals. |
| 21:11:08 | Bilgus | lol |
| 21:12:11 | gevaerts | Yes, there's something weird going on there. Tracing stuff said that the cgi on the log upload returned http 500 |
| 21:12:30 | gevaerts | I have *no* idea why |
| 21:13:36 | * | gevaerts asks zagor |
| 21:25:54 | gevaerts | speachy: do you see a hard reason not to just include lang_enum.h unconditionally in plugin.h? It's just an enum declaration, that shouldn't change anything, and I'm fairly sure the issue is caused by the dependency generation not doing the PLUGIN define |
| 21:29:02 | gevaerts | When that's done, the player build breaks because the lrcplayer plugin uses get_codepage_name, which was moved to HAVE_LCD_BITMAP in the talk support commit |
| 21:30:33 | speachy | I don't see why not, no |
| 21:31:07 | * | speachy was about to build an archos toolchain to start digging into this mess. |
| 21:31:34 | gevaerts | ok, I'll commit that then. I expect just the lrcplayer issue to remain after that, which I declare to be yours :) |
| 21:31:57 | gevaerts | Although that's just moving get_codepage_name() back to where it was I think |
| 21:33:32 | fs-bluebot | Build Server message: New build round started. Revision 4d2874f, 280 builds, 11 clients. |
| 21:33:58 | gevaerts | That's the #ifdef PLUGIN fix. We'll see if it's *just* get_codepage_name() stuff leftover |
| 21:34:21 | __builtin | speachy: are the authors of fs#7705 and fs#7912 in docs/CREDITS? |
| 21:34:22 | fs-bluebot | http://www.rockbox.org/tracker/task/7705 Add speech to battery bench (patches, closed) |
| 21:34:22 | fs-bluebot | http://www.rockbox.org/tracker/task/7912 Voice enable chessbox plugin (patches, closed) |
| 21:35:06 | speachy | the get_codepage_name thing had been wtapped in a HAVE_LCD_BITMAP in the original talk patch, I moved it up to consolidate it. |
| 21:35:21 | speachy | I believe so, but I'll double-check. |
| 21:36:16 | speachy | yes, all names were already in CREDITS |
| 21:37:18 | __builtin | agh, I grepped the wrong file... |
| 21:39:34 | gevaerts | speachy: I suspect that was an accident originally. The main get_codepage_name() function is in firmware/common/unicode.c, and doesn't have any such restriction, so just moving it back out should be fine. I'll move it |
| 21:40:10 | gevaerts | I'll keep plugin API version the same. It only changes for the Player, and that one hasn't built yet with the new API version |
| 21:40:29 | * | gevaerts will wait for the current build round to finish first |
| 21:40:56 | speachy | sounds good |
| 21:41:09 | gevaerts | These big changes are always an adventure :) |
| 21:42:00 | speachy | meanwhile twenty years later the Archos Player is still rearing its head. :) |
| 21:42:22 | gevaerts | It keeps people on their toes! |
| 21:42:47 | gevaerts | We don't actually know for sure if there's still a working one anywhere in the world |
| 21:42:58 | * | __builtin wonders if anyone has actually tested git HEAD on one of them |
| 21:43:13 | speachy | mine literally fell apart, as did the recorder I replaced it with. |
| 21:43:35 | * | gevaerts once got one for testing, but then he used a power supply with the wrong polarity |
| 21:43:42 | speachy | ouch. |
| 21:44:16 | speachy | proably wouldn't be that hard to fix. likely just the power regulator that blew.. |
| 21:44:17 | gevaerts | I really don't understand why people didn't just add a few diodes to those things |
| 21:44:39 | Bilgus | I saw one in the wild a few years back but its probably dead by now surely |
| 21:44:52 | speachy | because a diode would add $0.05 to the BOM.. |
| 21:44:58 | Bilgus | the diode drops .5v and generates heat |
| 21:45:18 | Bilgus | the rule of 5's apparently |
| 21:46:41 | speachy | since I'm mucking with my builders anyway, what would it take to add voice capabilities? |
| 21:46:41 | Bilgus | I like the trick of a fuse on the powerline and a reversed biased diode across tyhe power rails so it blows the fuse and the device is dead but repairable |
| 21:47:48 | gevaerts | speachy: you mean to tell you stuff? https://www.rockbox.org/wiki/BuildClient#Fun_stuff_to_do_with_a_client should explain all :) |
| 21:48:37 | __builtin | alright, sparta-builtin is back :) |
| 21:48:42 | __builtin | still as root, but sandboxed |
| 21:49:04 | speachy | that doesn't mention anything about what generates the nightly voice files |
| 21:49:13 | gevaerts | Oh, that sort of voice! |
| 21:49:15 | speachy | and rbbuild.pl is similarly quiet |
| 21:50:04 | gevaerts | I don't know exactly how that does it, but there's tools/release/voices.pl |
| 21:53:07 | gevaerts | I think technically we can get away with adding a new toolchain to rbclient.pl, and then just add lines to the builds file on the server |
| 21:59:04 | * | __builtin can get all but arm-app built in Docker |
| 21:59:50 | __builtin | https://pastebin.com/yPZqAeZH |
| 22:00 |
| 22:01:05 | gevaerts | Great! As expected, just Player left :) |
| 22:02:05 | gevaerts | Umm |
| 22:02:10 | gevaerts | That looks interesting |
| 22:03:03 | gevaerts | speachy: looks like in your cleanup you managed to make two nested #ifdef HAVE_LCD_BITMAP blocks |
| 22:04:46 | gevaerts | Or not, that was already there |
| 22:07:27 | fs-bluebot | Build Server message: New build round started. Revision 859f762, 280 builds, 11 clients. |
| 22:14:43 | * | __builtin is getting buildclient messages like these: |
| 22:14:45 | __builtin | 2019-07-20 21:20:57 Server message: Fatal build error: Command not found. You have been temporarily disabled. |
| 22:16:18 | __builtin | logs don't seem to have anything useful |
| 22:16:30 | gevaerts | Yes, those are fun. Means you're missing something, but the server doesn't tell you what |
| 22:17:10 | __builtin | I can build manuals... manually |
| 22:17:20 | gevaerts | The server saw "not found" in the build log somewhere |
| 22:18:43 | __builtin | and it seems to be fine automatically |
| 22:19:25 | gevaerts | Do a manual build and capture the output, and then grep for "not found" in there |
| 22:19:42 | gevaerts | That should at least give some hints |
| 22:22:26 | __builtin | here's my client image: https://hub.docker.com/r/built1n/rbclient |
| 22:23:49 | __builtin | damn, my client is too fast for me to manually type `tail' the on the log files |
| 22:25:47 | gevaerts | Which build is it? |
| 22:25:51 | gevaerts | pdf or html? |
| 22:25:53 | __builtin | iriverh120htmlmanual |
| 22:25:58 | gevaerts | ok |
| 22:26:12 | gevaerts | That runs ../tools/configure −−target=iriverh120 −−type=s && make MAKEFLAGS= manual-zip|grep -v 'TeX4ht warning.*not found' |
| 22:26:42 | gevaerts | Can you run (../tools/configure −−target=iriverh120 −−type=s && make MAKEFLAGS= manual-zip|grep -v 'TeX4ht warning.*not found') > logfile 2>&1 |
| 22:27:01 | gevaerts | and then put the logfile on a pastebin? |
| 22:27:31 | __builtin | no output on stdout |
| 22:27:52 | gevaerts | Well, everything should be going to that file if you run that |
| 22:28:51 | gevaerts | That's exactly what the client runs, except it uploads it to the build server instead of to a pastebin |
| 22:29:20 | __builtin | https://pastebin.com/91yjXPFc |
| 22:32:13 | __builtin | The grep -v ought to eliminate all the TeX4ht* lines, right? |
| 22:32:32 | __builtin | ahh... it's wrapping onto a new line for some reason |
| 22:32:42 | __builtin | so the "not found" is slipping through |
| 22:32:44 | gevaerts | Yes, it now also does that gere |
| 22:32:46 | gevaerts | *here |
| 22:33:04 | * | gevaerts tries something |
| 22:33:13 | __builtin | grep multiline? |
| 22:33:31 | gevaerts | First COLUMNS=300 |
| 22:33:48 | gevaerts | Maybe I can trick it into not wrapping. I think that would be the best solution |
| 22:35:46 | __builtin | it seems this build is the one left holding up the build round |
| 22:35:51 | gevaerts | Yes |
| 22:36:03 | gevaerts | I'm not convinced it's exactly this issue, but it might be |
| 22:36:47 | *** | No seen item changed, no save performed. |
| 22:44:10 | gevaerts | OK, if you set max_print_line = 1000 in texmf.cnf (which for me is /etc/texmf/web2c/texmf.cnf, but I had to make a file in /etc//etc/texmf/texmf.d/ with that bit in it), it works |
| 22:45:37 | gevaerts | So on debian-like systems, as root, "echo 'max_print_line = 1000' > /etc/texmf/texmf.d/linelength.cnf && update-texmf" |
| 22:48:49 | gevaerts | hmm |
| 22:53:01 | gevaerts | Wait, is it now triggering on finding "not found" in "grep -v 'TeX4ht warning.*not found'"? |
| 22:54:21 | gevaerts | I've changed the build command to try to avoid that. We'll see at the next build... |
| 23:00 |
| 23:17:49 | speachy | wow, a completely green build. |
| 23:19:50 | * | speachy is back from unpacking/organizing the garage.. |
| 23:19:54 | * | __builtin doesn't understand gevaerts's patch |
| 23:20:02 | __builtin | how does n->[n] change anything? |
| 23:20:59 | gevaerts | __builtin: easy :) |
| 23:22:15 | gevaerts | The server (in rbmaster.pl) looks for literally "not found", and the build command included that. Now it has "[n]ot found", which in regular expression terms is exactly the same, but won't trigger the server |
| 23:22:57 | __builtin | ah, it was triggering on the build command itself? |
| 23:23:10 | gevaerts | It's the same thing you do when running "ps -ef|grep some_command", which will find your grep line, so if that's inconvenient you do "ps -ef|grep [s]ome_command" |
| 23:23:22 | gevaerts | I think so. I'm not entirely sure |
| 23:23:31 | __builtin | did you get around the tex wrapping too? |
| 23:24:02 | speachy | well, I can kick off another build by voice-enabling another plugin |
| 23:24:08 | gevaerts | Yes, see above. I didn't find how to disable that for the command, but I did find how to do it system-wide |
| 23:24:24 | __builtin | oh, duh |
| 23:24:45 | * | __builtin should generate some screenshots for quake and wolf |
| 23:25:08 | __builtin | seeing as I now have a decently fast machine, it shouldn't take a week as it used to :) |
| 23:25:31 | gevaerts | It will |
| 23:25:48 | gevaerts | I mean, if you start those plugins, you might find yourself distracted :) |
| 23:26:06 | Bilgus | ha. |
| 23:26:25 | __builtin | think I could port mesa and quake 3? |
| 23:27:32 | gevaerts | Depends. What framerate are you aiming for? |
| 23:28:01 | gevaerts | RAM isn't an issue if you have sufficient disk to swap things out to, and you have time |
| 23:28:22 | speachy | heh, some of the newer targets have a 3d engine in their SoC. |
| 23:39:53 | fs-bluebot | Build Server message: New build round started. Revision 5a84239, 280 builds, 12 clients. |
| 23:40:21 | speachy | let's see if the iriver manual build succeeds this time.. |
| 23:46:51 | __builtin | gigabeatswps fails on my client |
| 23:46:56 | __builtin | same "command not found" error |
icon identifies that the person is a core developer (has commit access).