#mer log for Tuesday, 2018-06-19

*** frinring_ is now known as frinring02:25
r0kk3rzabranson: https://git.merproject.org/mer-core/libvorbis/merge_requests/208:17
r0kk3rzthat one isnt in the maintainers yaml, so someone should probably claim it08:18
abransonr0kk3rz: ok. wasn't autogen running configure afterwards?08:25
r0kk3rzit seems to have changed08:26
abransonaha good thx. so configure wasn't running after you updated?08:26
r0kk3rzi dont think so, at least the make files werent being generated08:27
r0kk3rzyeah theres a lot less in the autogen.sh now08:28
r0kk3rzabranson: https://git.merproject.org/mirror/vorbis/commit/4729a15df74cd4d6fc487a43157bf9fb65d1221808:29
abransonmakes sense tbh. it's did overcomplicate things08:30
lbtr0kk3rz: you can fork maintainers and do an MR to add packages to the right place08:31
r0kk3rzlbt: im not sure where the right place is08:32
r0kk3rzits mostly audio related so id say where the libogg package is08:32
lbtprobably alongside libogg :D :D08:32
r0kk3rzyeah08:33
r0kk3rzlbt: jusa: https://git.merproject.org/mer-core/Maintainers/merge_requests/22808:38
lbtHmmm - someone (not you) can't do alphabetical order!08:39
r0kk3rz:)08:39
lbtcould you add another seperate commit to fix that whilst you're there please08:39
r0kk3rzsure08:39
r0kk3rzlbt: is that better? can i alphabet?08:42
lbtty08:44
r0kk3rzlbt: is it correct that some packages are listed twice in the maintainers.yaml?08:57
lbtyes but it's not supposed to be common08:58
r0kk3rzi can see a bunch, but they're in the base packages group + others08:58
lbtin general that is bad08:59
lbtif they're in another group they can probably be removed from base08:59
lbtwhich?08:59
r0kk3rzlibsoup, vala, raygel, pango09:00
r0kk3rz*ragel09:00
r0kk3rzoh, not ragel, i got confused with rygel :)09:01
r0kk3rzgnome-common also09:04
lbthappy to have one or more MRs to suggest tidy ups09:06
spiiroinreview appreciated: https://git.merproject.org/mer-core/sensorfw/merge_requests/2510:15
spiiroin^ should fix compass sensor in f5121 & some other devices too10:15
r0kk3rzoo compass :)10:16
r0kk3rzspiiroin: i think your maths is a bit squiffy :P10:26
r0kk3rzlike, 1000us = 1ms, but that conversion ends up more like 1.999ms10:28
spiiroinr0kk3rz: ??? (it is integers)10:31
spiiroin(1+999) / 1000 = 1 ; (1000 + 999) = 110:31
r0kk3rzyeah ok10:32
r0kk3rzim too used to doing unit conversions with doubles :)10:32
spiiroin... especially if the latter one would divide by 1000 too ;-)10:32
r0kk3rzim sure it doesnt need to be super accurate for this purpose anyway10:34
spiiroinr0kk3rz: not being off by magnitude of 1000 & not allowing minimum in [ms] to be less than minimum in [us] = should be about enough10:54
mallbt: OBS is oddly quiet on #mer-boss, no messages from the FP2 builds which should have show at least check of patterns12:08
lbtyeah - ty12:15
lbtlooking12:15
mallbt: also webhooks seem to be failing12:18
malrainemak: hi, could you merge this https://github.com/mer-hybris/libhybris/pull/40 unless there is some testing still ongoing?13:47
r0kk3rzpvuorela: is there a particular reason cairo is setup as a submodule but with a dumped folder? - https://git.merproject.org/mer-core/cairo17:06
krnlyngr0kk3rz, that might be a git subtree17:11
pvuorelar0kk3rz: yep, for some odd reason we had these git subtrees at some point even when commonly those were just vanilla upstreams.17:12
r0kk3rzthis one looks unaltered as far as i can tell17:17
pvuorelar0kk3rz: just delete the remove directory on such cases if you do updates.18:04
r0kk3rzsure18:22

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