Wednesday, 2019-03-27

eugenioyup indeed the query result is a nullptr00:14
eugeniogetting late now, will return to it tomorrow00:15
eugeniothanks as always mal :)00:15
steevieStill awake03:20
r0kk3rzstill in awe of piggz-o-vision?03:23
steevieNaah, my vision is better than the piggz one03:31
T4<StillJustGerd> Hmm, i'm having trouble to apply the patch for bluetooth hci_smd06:24
T4<StillJustGerd> getting these errors when compiling
T4<StillJustGerd> is there any other way to get hci_smd working?06:28
T4<birdzhang> It seems that the Bluetooth driver does not match the  kernel  version.06:29
T4<austenite> @StillJustGerd I get the same when compiling on msm8937 the driver is for msm891606:30
T4<StillJustGerd> @birdzhang yeah, that seems to be the problem... any idea where i can find the correct one, as my kernel does not seem to contain it06:31
T4<StillJustGerd> @austenite we are both working on the same kernel apparently, what device are you porting?06:31
T4<birdzhang> what is your kernel version ?06:31
T4<austenite> @StillJustGerd yureka black kernel 3.18.3106:32
T4<StillJustGerd> 3.18.something @birdzhang06:32
T4<StillJustGerd> @austenite i'm working on the motorola moto g5s, so not the same device06:32
T4<birdzhang> wow, try this
T4<StillJustGerd> @birdzhang testing it right now06:34
T4<StillJustGerd> should i make clean06:34
T4<birdzhang> fingers crossed xD06:34
T4<StillJustGerd> just did a `make clean`, now `make -j4 hybris-hal` again, ccache should make it faster06:35
T4<StillJustGerd> if i just want to test it, i should be able to flash my latest build plus the hybris-boot.img, right?06:39
T4<StillJustGerd> so i don't have to rebuild a whole zip using mic06:39
T4<birdzhang> if your wlan is built as module, you may need symbol link /lib/modules/{kernel_nums}/06:41
T4<StillJustGerd> okay i see06:46
T4<StillJustGerd> building finished, I'm trying to flash now06:54
T4<StillJustGerd> just have to really quickly backup my lineageos /data06:57
T4<StillJustGerd> i can't even rebuild the system image yet, turns out i forgot to reinstall the sdk targets when I set up the build env again06:58
T4<austenite> @birdzhang kernel built successfully, but bluetooth is empty in settings...07:18
T4<birdzhang> @austenite does your wifi works ?07:24
T4<austenite> no @birdzhang no, havent built it as module yet07:26
T4<birdzhang> modprobe wlan07:27
T4<austenite> @birdzhang whows nothing...07:30
T4<austenite> @birdzhang wlan works after : echo sta > /sys/module/wlan/parameters/fwpath07:31
T4<birdzhang> bluetooth sitll not work ?07:33
T4<birdzhang> ^still07:33
T4<birdzhang> oh, wait07:33
T4<austenite> @birdzhang no07:34
T4<birdzhang> @austenite does this script helpful ?07:43
T4<birdzhang> it works on xiaomi mido and vince, msm895307:44
T4<austenite> @birdzhang  i will test report within few hours , i had reset the repo and cant build image @themoment07:49
T4<birdzhang> okay07:50
T4<adampigg> Mal @eugenio_g7 cool, so we can fork and build a fixed lipstick08:37
T4<adampigg> :)08:38
mal@adampigg need to figure out why that really happens, according to specification eglQueryString should always return a string11:43
malbut I also found some conflicting information, not sure what is correct11:46
malah, indeed it can return null11:47
T4<adampigg> Mal, cool,.someone submitted a pr from zoom, will have to take a look12:31
T4<adampigg> Mal, but ill hold all prs until your work is merged12:47
mal@adampigg you can merge fixes if you want, I can always rebase, I cannot say how long it takes before I'm happy with how my changes work13:02
T4<adampigg> Mal, you are focussed on fron camera and resolution models?13:05
maland dynamically updating menus in general13:10
malI have those updating fine but still some issues with other parts of camera switching13:10
T4<austenite> stuck with this error,
malwhich hybris base?18:29
T4<austenite> @mal 14.118:33
vknechtnot in pfsdk ?18:34
mal@austenite are you in platform sdk when you run that command?18:35
T4<austenite> @mal yes..18:36
malthe commandline in that paste doesn't look like it18:38
malit should be something like PlatformSDK bash-3.2$18:38
T4<austenite> @mal, the PLATFORMSDK bedore bionic disappeared after i reset repo and cleaned the .mersdk files18:38
malyou should fix the .mersdk files, makes it much easier to see everything is correct18:39
T4<austenite> @mal ok, but i can confirm its running in platformsdk , it worked with rpm/dhd/helpers/build_packages.sh18:40
steevieFerrite α18:40
T4<austenite> @mal and if i give —configs it will never build agian18:41
T4<austenite> ledeburite18:41
malwhen did you rebuild (and update) libhybris the last time?18:41
T4<austenite> @mal this is what happened, in order to fix browser crash i removed libhybris and sucessfully built image18:42
T4<austenite> and i cloned libhybris .. since then this error18:42
T4<austenite> @mal , tried building libhybris ..
malgo to libhybris sources, in libhybris/hybris/common/n/linker_gdb_support.cpp replace all instances of _hybris_enable_android_debug with _linker_enable_gdb_support18:46
malthen -b hybris/mw/libhybris18:46
mal@austenite does that help?19:01
T4<austenite> @mal it built now building image.. thanks alot19:07
steevieMeno mal!19:07
T4<austenite> @steevie Im not very good at metallurgy19:11
mal@austenite why did you think your problem would be fixed by using upstream libhybris19:13
malfor browser issues there is one usual fix19:13
mal@austenite what exact problem do you have with browser?19:15
T4<austenite> @mal full white page, then had a pink browser19:16
steevie@austenite then why that nick?19:16
mal@austenite then that file I linked should fix it19:16
T4<austenite> @steevie i like metallurgy but im not good at it :P19:16
steevieAaand you know how to dimension structures?19:17
steevieAt least that...19:17
mal@austenite for testing copy that file to your device to correct place19:17
T4<austenite> @steevie dimension structures?19:19
steevieI dont know how it's called in english, but, it's the thing you do to calculate the minimum amount of material needed to resist to the sollecitations applied to them19:22
piggz_mal: @eugenio_g7: my simple lipstick fix didnt work :/19:24
steevieIsostatic, iperstatic stuff, cinematics, esketit19:25
malpiggz_: can you show the patch you tried19:28
piggz_mal: oh, shit, hangon.....19:29
piggz_dont dare go looking for my mess up19:29
mal@austenite I made a pull request for the libhybris fix to upstream19:30
malpiggz_: :D19:30
malpiggz_: I had a look, you know there is a much simpler way?19:32
piggz_mal: likely19:32
T4<austenite> @steevie you must be talking about structural engineering, i am unaware of these in mechanical..19:33
piggz_mal: i started on something i thought was simpler, then realised the whole lot was in a struct, not  a function19:33
mallike const char *queryString = ...; supported = queryString && strstr...19:33
piggz_yeah, thats sort of what i was going for what i was going for19:33
piggz_but i didnt want to add another member to the struct19:34
malor maybe better name for that variable is extentionsString19:34
malor just extensions19:34
piggz_anyway ... the real fuckup was that i built master and not fixes on obs19:34
malanyway your version is a bit overly complicated now19:35
piggz_and doesnt actually build....19:36
piggz_strstr returns const char* not int19:36
piggz_mal: ok, i can go back to my original way and add another member ... you think thats preferable?19:38
malwhat is your original way?19:39
mal@austenite sorry, I was wrong you need these 3 lines in droid-hal spec
malthen rebuild droid-hal rpms and libhybris19:41
malpiggz_: I think a wrapper function should be used only if you need it many times19:43
steevie@austenite not only engineering19:43
steevieYou should study it in high school as well19:43
piggz_mal: well, my fix may not be the best, but oit does work :)19:45
piggz_@eugenio_g7 ^^19:45
malpiggz_: doesn't my way work, much simpler imo19:45
maljust a local variable for the egl query result and then check it and use it for strstr19:46
piggz_mal: im sure it will......19:46
malhmm, but that is a struct19:47
malmakes it more complicated19:47
piggz_yes, that was why i went my way....19:49
piggz_but im trying your way now19:49
piggz_i think it will work19:49
malit won't19:50
malwe can't define variable normally in struct19:50
malI mean to be used in other parts of the struct19:51
piggz_mal: wold have to duplicate the function call then19:53
piggz_which is also messy19:53
piggz_mal: it builds ok19:57
piggz_mal: your way works too20:03
malpiggz_: but wouldn't my way create a useless variable in the struct, and it would remain there20:20
piggz_mal: it would ... thats what i said first :D20:20
malwell I didn't focus enough20:21
piggz_mal: so, my original way?20:22
malI know another ugly way, maybe your way is better, it could be done using some ternary operator magic but it wouldn't look nice20:23
*** PragmaticAnarchi is now known as Unbegriff20:27
malpiggz_: maybe the fix could use a mer bug reference20:34
piggz_mal: i also discarded the ternary option :)20:57
piggz_as uglier20:57
piggz_so, i will go back to mine, add a bug report, and put the big in the commit20:58
piggz_and PR20:58
malpiggz_: good, thanks20:59
piggz_mal: what is the format....21:05
piggz_mer #nnnn ?21:05
piggz_in the commit coment21:05
T4<eugenio_g7> Fixes MER#nnnn21:08
piggz_also, i think i fixed the sensors21:13
malpiggz_: do you want to hear some nitpicking :P21:13
piggz_        sampling_frequency_file=$(find /sys/bus/iio/devices -type f -name in_accel_sampling_frequency -follow -maxdepth 2 | head -n1)21:13
piggz_mal: no, i figure you should rest up21:14
malpiggz_: capital letter are nice thing, also a bit long commit message as it doesn't fit well into one line21:15
T4<eugenio_g7> piggz_: nice nice21:18
piggz_mal: i dont know what you are talking about, my commit message looks perfect
piggz_@eugenio_g7 try that out for sensors and see if its better21:20
T4<eugenio_g7> Will do soon-ish21:21
eugeniopiggz, I can confirm that both the lipstick and sensors fixes work as expected :)22:14
T4UgeneR was added by: UgeneR22:18
piggzeugenio: \o/22:33

Generated by 2.17.1 by Marius Gedminas - find it at!