!rGlCMuXgAhgEpdvJUz:nixos.org

NixOS KDE

135 Members
44 Servers

Load older messages


SenderMessageTime
5 Sep 2024
@k900:0upti.meK900Just make it log very loudly when it's trying to cache bare QML08:07:41
@k900:0upti.meK900Along with the file path08:07:44
@k900:0upti.meK900And run that for a bit08:07:50
@k900:0upti.meK900And see when it actually gets hit, if ever08:07:57
@outfoxxed:outfoxxed.meoutfoxxeddoesn't even need to be loudly08:07:58
@outfoxxed:outfoxxed.meoutfoxxedjust needs a category08:08:01
@k900:0upti.meK900Well very loudly is helpful for finding it in the journal later08:08:09
@k900:0upti.meK900But otherwise yes08:08:14
@outfoxxed:outfoxxed.meoutfoxxedI don't run many qml applications or plasma so I can't really test that08:08:23
@outfoxxed:outfoxxed.meoutfoxxedsomeone with plasma would be the best choice08:08:29
@k900:0upti.meK900I can volunteer myself but probably not today08:09:35
@outfoxxed:outfoxxed.meoutfoxxedmy own application exclusively uses fs paths but they're all running through a url interceptor which makes them impossible to cache and test08:10:09
@outfoxxed:outfoxxed.meoutfoxxedWhat route do you think we should take the general validation storage though? I'm thinking libraryVersionHash, and I'll implement that probably tomorrow if you don't have any objections to that part08:18:32
@k900:0upti.meK900You mean hash application build-id into the timestamp field?08:19:14
@outfoxxed:outfoxxed.meoutfoxxedno, i mean the store path into libraryVersionHash08:22:02
@outfoxxed:outfoxxed.meoutfoxxedif any of this goes upstream they can just allocate fields for it08:22:15
@outfoxxed:outfoxxed.meoutfoxxedwe can't do that without increasing the maintenance burden of the patch08:22:44
@outfoxxed:outfoxxed.meoutfoxxed
In reply to @k900:0upti.me
I can volunteer myself but probably not today
and by the way if you want to log disk cache loads, ExecutableCompilationUnit::loadCacheFilePath
08:23:56
@outfoxxed:outfoxxed.meoutfoxxedRedacted or Malformed Event08:24:03
@outfoxxed:outfoxxed.meoutfoxxedyou probably want to give it a custom logging category so its easy to search for in logs08:24:21
@outfoxxed:outfoxxed.meoutfoxxedthat function will only be called for disk stores and lookups08:24:52
@k900:0upti.meK900
In reply to@outfoxxed:outfoxxed.me
no, i mean the store path into libraryVersionHash
Like, the application store path?
08:26:41
@k900:0upti.meK900That feels very cursed tbh08:26:49
@outfoxxed:outfoxxed.meoutfoxxedyes08:26:49
@k900:0upti.meK900Though I guess if we do that we can drop the Qt cmake hack08:26:59
@outfoxxed:outfoxxed.meoutfoxxedits less cursed than anything else we discussed08:27:00
@k900:0upti.meK900Which is also pretty cursed08:27:04
@outfoxxed:outfoxxed.meoutfoxxedi think we can drop the cmake hack regardless08:27:12
@outfoxxed:outfoxxed.meoutfoxxedbecause tag solves the same problem08:27:18
@k900:0upti.meK900Well yeah but you now have a patch to allow us to pass a custom tag08:27:35

Show newer messages


Back to Room ListRoom Version: 9