libera/#eudev/ Sunday, 2023-10-01

bb|hcbvivien: The man page talks about rule files from /run/.../hwdb.d but I see nothing like that in the code...00:15
bb|hcbSo the question is shall we fix the code or the man page ;)00:15
vivienbb|hcb, maybe this is a bad copy-paste from the "rules" side of eudev? Or maybe the behavior was present but removed? I think noone ever complained about that in the issues...08:18
vivienThe text comes from 9d06e6560, but it is rather unhelpful as it does not give much information about the provenance of the text, and /run is not in the path either at that time. Anthony G. Basile would be the one to ask, but it was in 2014.08:26
vivienFrom what I understand right now, there is no justification for having files in /run08:30
vivienAlso, if you think about it, it would be a disaster to have hwdb.bin depend on volatile files, but not be volatile itself, don’t you think so?08:32
bb|hcbvivien: Yes, I agree. Are you willing to remove the texts from the man pages?19:37
bb|hcbAbout PR #253 - no one cares to make a review, also there are no objections. I am planning to merge that without review in the next couple of days19:42
bb|hcbAbout PR #258 - I did some research and a similar rule is not present in systemd's udev, there are converns that it will decrease security and I plan to reject and close it19:43
vivienbb|hcb, I don’t know if #253 is sufficient, upower still crashes on the GNOME update in Guix. If and when I manage to get a VM booting again, I will comment whether the dummy functions are sufficient!19:56
vivienIt is definitely sufficient to have unit tests passed in upower though20:00
bb|hcbThe implementation is not dummy - it returns all tags regardless if they are current or not current20:04
vivienSorry the word was wrong20:05
bb|hcbThe full implementation would be to add a current field in tag db, then add enumerators that return only the current tags and so on...20:05

Generated by irclog2html.py 2.17.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!