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

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

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

#rockbox log for 2025-03-07

00:14:07***Saving seen data "./dancer.seen"
02:00
02:14:08***No seen item changed, no save performed.
03:00
03:09:50 Quit othello7 (Ping timeout: 252 seconds)
03:44:50 Quit hactar|ant (Ping timeout: 260 seconds)
03:48:33 Join hactar|ant [0] (~zem@97-115-78-75.ptld.qwest.net)
03:53:59 Join ^Dan [0] (~xxx@89.136.142.218)
04:00
04:04:29 Quit jacobk (Ping timeout: 260 seconds)
04:04:58 Join jacobk [0] (~quassel@47-186-65-73.dlls.tx.frontiernet.net)
04:05:39 Part ^Dan
04:14:11***Saving seen data "./dancer.seen"
06:00
06:14:12***No seen item changed, no save performed.
06:16:19 Quit PheralSparky (Read error: Connection reset by peer)
07:00
07:17:04 Quit thanosengine (Ping timeout: 245 seconds)
07:19:08 Join thanosengine [0] (~thanos@user/thanosengine)
07:38:51 Quit disrel (Ping timeout: 244 seconds)
08:00
08:14:14***Saving seen data "./dancer.seen"
09:00
09:14:55 Join disrel [0] (disrel@user/disrel)
09:28:57 Quit disrel (Ping timeout: 252 seconds)
09:39:21 Join disrel [0] (disrel@user/disrel)
09:53:39speachyso, I've added a warning to checkwps about use of SYSFONT in a theme. it's waaay too prevalent for us to just reject it.
09:55:04speachynot in the sense that themes are actually _using_ sysfont to render text, but it's used in a lot of graphical element viewports..
10:00
10:00:31speachyand the skin_Debug stuff now dumps viewport arguments too
10:07:56_bilgus_ok so I have the Hifiwalker H2 thw scrollwheel is rubberized
10:08:07speachyyep
10:09:19_bilgus_any way I'm a bit confused configure 249) Eros Q / K native (hw4 bl only) I assume it means for the HW4 ver only or does it mean its only for building a bootloader?
10:10:13speachythe latter.
10:10:25_bilgus_ah ok
10:10:53_bilgus_explains why I was getting an error doing -N
10:13:00_bilgus_for some reason when I downloaded the latest daily I can't run most of the lua script because it excludes like 3/4 of the include scripts
10:14:16***Saving seen data "./dancer.seen"
10:15:15_bilgus_probably need to check permissions or something
10:16:57_bilgus_ah its not copying any only the generated ones, wonder how long this has been broken
10:18:25speachyshows how many users there are. :D
10:18:48_bilgus_whats weird is it works here
10:19:18_bilgus_no one will ever use it if they can't start the scripts :/
10:30:35_bilgus_here is the line in the make file '$(LUA_BUILDDIR)/%.lua: $(LUA_INCLUDEDIR)/%.lua | $(LUA_BUILDDIR)
10:30:35_bilgus_$(call PRINTS,CP $(notdir $<))cp $< $@'
10:36:48_bilgus_a few of them have full permission and they don't show either so probably not that
10:38:46_bilgus_weird, guess I'll DO IT IN BUILDZIP
10:45:31_bilgus_I think I can do both without too much issue extra copying if it works but should ensure the files get included no matter what
10:46:42_bilgus_hahah I have to because when I remove the copy part it fails to build for me now wth doesn't the same hold true when its the buildserver :/
10:49:07speachybuildzip is how we install unmodified files, so yeah
10:49:35_bilgus_yeah but a prereq getting skipped is ,, odd
10:49:41speachycould be different umasks or something?
10:50:03speachyor different mtime on the files?
10:50:29_bilgus_oh the former doesn't appear to be but that latter part hmm
11:00
11:07:14_bilgus_this has been a month of bugs, glad to find them now rather than after 4.0
11:07:49_bilgus_or rather Febuary was
11:10:57_bilgus_well doing t in the build script appears to work ok but I know there was some particular reason I didn't do that in the first place but damned if I remember what
11:16:14speachylots of noise?
11:22:55_bilgus_an issue with it not realizing the source changed and keeping the old files perhaps it was something probably come back to bite me
11:24:17sporkwill the commit messages return to irc ?
11:24:36sporki like those as a passive way to see the changes
11:24:57_bilgus_haven't seen bluebrother
11:25:18_bilgus_the bot is here he's just not connected to the server for some reason
11:25:32_bilgus_ g#6418
11:25:35rb-bluebotGerrit review #6418 at https://gerrit.rockbox.org/r/c/rockbox/+/6418 : [BugFix] lua include scripts may fail to copy by William Wilgus
11:25:48speachyyeah, probably a really simple thing to fix but it's his bot running on his system
11:26:05sporkturn on the bluebrother^ signal
11:26:33_bilgus_tbh don't want to bug him too much its his good will afterall
11:27:25speachywe could move it to the rockbox.org server, if he's agreeable.
11:27:38speachyalongside the logbot
11:27:49_bilgus_I suppose you don't notice till its gone :)
11:28:11speachyI've missed it for some time; plus we get instant notification if we break the build. :D
11:28:28sporkwhen _bilgus_ breaks the build
11:28:29_bilgus_yeah i'm back to watching the build page
11:28:44_bilgus_no commits == no breaks?
11:28:56sporkbreakage is progress
11:28:59speachywell, now you're thinking like a politician
11:30:03speachyall of us with commit rights have introduced a sea of "How did this not compile, I'm _running_ it!" red
11:30:45_bilgus_If I had a wider swath of test buils I could ave myself some grief but there are quite a few one offs
11:31:01_bilgus_checkwps is my usual oversight
11:31:33sporkthe battery levels change is much appreciated
11:31:43speachyslowly stumbling towards perfection
11:31:54_bilgus_careful with that one, note the warning in the manual
11:32:07sporkwith batteries degrading over time, we can now re-run it and get something close to reality
11:32:08speachyI try to ensure we have no coverage gaps in the builders.
11:32:38speachywhich reminds me, need to get a winsim in there too somewhere
11:32:54speachy(and finish adding in the full complement of manuals, so we can ditch the nightly manual builds)
11:33:16sporkreading? manual? no way, straight to breaking and blaming
11:33:37_bilgus_my fear is someone sets shutoff to super low on a device that we actually control and wiping out their battery or disksafe super low and corrupt their disk
11:34:16_bilgus_I thought about making the eport comment out disksafe and shutoff or even exclude them
11:34:21speachyIMO we should hardcode in a failsafe level, everything's using li-ion cells these days.
11:34:58_bilgus_we already have that we could just exclude 'shutoff'
11:35:14speachydisksafe though is trickier, because the current outrush will do a lot worse on an old worn out cell than a good one, specially at the lower charge levels.
11:35:26speachyyeah, I think that's prudent
11:36:13_bilgus_I'll leave the import part but remove 'shutoff' frm the exported file
11:36:16sporkdisksafe applies to all storage types?
11:36:36speachyit's per-player, so yes.
11:36:40_bilgus_yeah the idea is harddisks need more power
11:37:07speachywas especially bad for the OG archoses with their 2.5" ATA drives
11:37:07_bilgus_but if you were on an ipod and have iflash that whole dynamic changes
11:37:08sporkyes, but there are major differences between hdd's, sd cards, internal flash, etc.
11:37:54_bilgus_being that the sd etc take much less power therefore can write at lower battery levels (read current requirements)
11:38:21speachyif your device wasn't shipped with a spinning rust drive originally, then drivesave and shutoff are probably the same
11:38:39sporkright
11:39:01sporksafe is good
11:39:16speachywe always err on the side of not destroying data, where possible
11:39:30sporki am going to try it on some of my sansa's. they have never charged up to 100%
11:39:39sporknice goal, speachy
11:40:09sporkonly destroy as a last resort
11:40:16sporknot thinking like a politician
11:41:07_bilgus_with a bit more work on the config file writing we could actually negate the possibility of a partially written config file corrupting the next boot after a drive failure
11:41:27_bilgus_rather a write failure
11:42:17_bilgus_really it just needs an extra line to be parsed crc of the data written or a just a #Success comment at the end
11:42:46_bilgus_if its not there then fallback
11:44:58_bilgus_crc would be a bad idea we want the file easily changed well maybe for the resue info one that would be good
11:45:10_bilgus_resume*
11:48:21 Quit rogeliodh (Quit: The Lounge - https://thelounge.chat)
11:48:28_bilgus_YES the lua includes are now.. included.
11:48:42 Join rogeliodh [0] (~rogeliodh@rogeliodh.dev)
11:57:14_bilgus_really I just wanted to run the 'stars' lua demo with the scroll wheel
11:58:38_bilgus_its only left or right but its pretty neat since you can wind it up
12:00
12:12:55 Quit thanosengine (Quit: -h)
12:14:17***Saving seen data "./dancer.seen"
12:46:04_bilgus_Well I have to leave shutoff at least for reference so I moved both to the end of the file added a warning and commented shutoff https://pastebin.com/9vckbT3n
12:49:00_bilgus_is that just a general message or indicative of an issue? 'WARNING: Do not use SYSFONT (id 0) in viewports!'
12:53:29_bilgus_the latter hmm
12:54:03speachyit's not actually a problem unless that vp is used to display text
12:55:18_bilgus_??
12:56:53_bilgus_so what exactly is the issue when you do?
12:57:24_bilgus_just missing character set or something worse?
12:58:02speachyit just means that your UI element size will change out from underneath you
12:58:25speachyalso means you don't have i18n support
12:58:35speachybecause sysfont is limited to 250 glyphs
12:59:11speachythe more general problem is that we have the statusbar height hardcoded to be sysfont_height
12:59:50speachy(and many places still just assume both are 8px)
13:00
13:06:34_bilgus_very old issue too
13:07:46speachyreally the only theme to use font ID 0 should be failsafe and cabbie.
13:07:52_bilgus_could we leave it alone (the size) and scale it (the whole rendered item)
13:08:58_bilgus_might leave some artifacts doing the scaling the naive way but it'd at least fill the screen properly
13:09:36speachyno... first step is to un-hardcode stuff
13:10:10speachyso statusbar_height is determined at runtime (based on the current theme's statusbar viewport)
13:12:38_bilgus_can of worms
13:16:31_bilgus_ok I went ahead and comment out both shutoff and disksafe entries so you can change them if you want but there is a warning and you need to remove the '#'
13:18:28speachyand keep un-hardcoding assumptions about font height in the codebase
13:21:24_bilgus_pervasive is the word that comes to mind
13:22:31_bilgus_luckily I think we have enough varied testers to actually catch the missing ones
13:22:52speachywell, not having common devices with sysfont>8px will severely limit that testing
13:23:24speachyIIRC cabbie only uses sysfont for the statusbar
13:24:20_bilgus_so you need a theme that uses its own sb or uses default to test it then?
13:24:50_bilgus_or do I need to try changing sysfont size and running cabbie?
13:26:56_bilgus_I wonder if bahaus 'Roman A' ever made any head way with the bad opus files I uploaded
13:27:12speachyaccording to grep nearly all sysfont uses are the debug screens
13:27:33_bilgus_well i'd be ok if those are left that way
13:27:43speachyoh that's compltely fine (and what we want)
13:27:51speachyproblem is when stuff hardcodes expected line heights
13:28:25speachythe stuff I fixed a few days ago was like that, it manually drew stuff assuming lines were 8px high. not even watching sysfont height.
13:28:38speachythe mikmod plugin was also like that
13:28:43_bilgus_yeah there are several of those
13:29:12_bilgus_I think its just because they are old
13:29:36_bilgus_wasn't a concern when written and someone just papered over the issue once it was
13:29:41speachythe usb screen is also hardcoded to sysfont; that needs to change.
13:30:08speachy(we presumably need to pre-render each of the possible strings in advance, so we can switch usb modes without needing ot possibly read from disk)
13:31:18speachylots of plugins use SYSFONT too, but they'd need to be audited to make sure they scale heights properly
13:32:46_bilgus_so you want to render them and save the resulting bmp?
13:32:57speachyyeah
13:33:28_bilgus_i think it might be better to just set aside the glyphs
13:34:01_bilgus_still render them but record the glyphs as they go by and make sure they stay
13:34:04speachywhichever is less work, but we have to ensure there is no possibility of trying to read the font (because it won't be there)
13:34:20speachybeen a problem for a long time: FS #12436
13:34:21rb-bluebothttps://www.rockbox.org/tracker/task/12436 USB keypad mode text unreadable in Russian regardless of selected font (bugs, unconfirmed)
13:34:22_bilgus_thinking less wasted space tbh
13:34:46speachydo it at runtime
13:34:58speachyso there's no wasted space because the player can't do anything else in usb mode
13:35:14_bilgus_oh you mean just before the screen
13:35:18_bilgus_GOT ya
13:35:25speachyyeah, that way it respects the current language etc
13:35:58_bilgus_that won't be too bad should just be able to index it and use bmp part
13:36:16speachyyou know more about that stuff than I do
13:36:19_bilgus_scrolling probably too much work
13:36:44speachyyeah, no point in scrolling
13:37:01_bilgus_I'll put it on my list, I need to force myself to write some plugin manuals first
13:37:30_bilgus_otherwise I never will
13:39:30_bilgus_thats what I had started to do when for that lua thing came up lol
13:40:37 Join othello7 [0] (~Thunderbi@pool-100-36-176-164.washdc.fios.verizon.net)
13:42:18speachydanbke
13:42:20speachydanke
13:43:01speachythat's one of the handful of UI non-i18n warts.
13:43:41speachyprobably won't get the voicing db entries stuff done
13:43:52speachybut we gotta leave something for 4.1. :D
14:00
14:14:18***No seen item changed, no save performed.
14:22:25 Quit jacobk (Ping timeout: 248 seconds)
14:24:10 Join jacobk [0] (~quassel@47-186-65-73.dlls.tx.frontiernet.net)
14:31:19 Join chris_s [0] (~chris_s@2a09:bac3:2d53:1b4b::2b8:4b)
14:34:36 Join lebellium [0] (~lebellium@2a01cb0405d03a004070cdf0d3f7e170.ipv6.abo.wanadoo.fr)
14:48:23_bilgus_wonder if we could just print all the usb strings and circle or invert the current mode
14:53:26_bilgus_something like [||}== keypad mode multimedia \n [presentation]\n browser\n mouse
14:55:00_bilgus_I think if it fits on the clip+ or xduoo those are the shortest screens
15:00
15:08:25 Quit Burak__ (Read error: Connection reset by peer)
15:11:07 Join sebagala [0] (~Burak@185.25.123.34)
15:40:41gnafuI'll have you know that Rockbox is responsible for me spending quite a bit of money on Bandcamp over the last few weeks >_>.
15:45:00gnafuI was strictly a "buy CDs and rip them" person until now, but for whatever reason the mental block on enjoying "digital albums" has been lifted thanks in part to having a dedicated player with Rockbox on it.
15:45:50gnafu(I will still buy CDs when albums I want are available on CD, but there are some artists I've enjoyed in passing who have only released things digitally.)
16:00
16:14:22***Saving seen data "./dancer.seen"
16:38:25 Quit drew (Quit: Quit)
16:42:02 Quit speachy (Quit: WeeChat 4.5.2)
17:00
17:18:29 Quit cstine (Quit: The Lounge - https://thelounge.chat)
17:19:02 Join cstine [0] (~cstine@150.136.136.191)
18:00
18:11:23 Quit lebellium (Quit: Leaving)
18:14:23***Saving seen data "./dancer.seen"
18:34:29 Join drew [0] (~drew@user/drew)
18:49:56zouthat's great! supporting artist (more) directly seems better than through big labels
19:00
19:01:08 Join massiveH [0] (~massiveH@2600:4040:a982:5400:49e4:7c95:2e7e:7c7d)
19:11:10 Join thanosengine [0] (~thanos@user/thanosengine)
20:00
20:14:26***No seen item changed, no save performed.
21:00
21:05:10 Join othello8 [0] (~Thunderbi@pool-100-36-176-164.washdc.fios.verizon.net)
21:19:21 Quit othello8 (Quit: othello8)
21:35:44 Quit chris_s (Quit: Client closed)
22:00
22:14:28***Saving seen data "./dancer.seen"

Previous day | Next day