| eugenio | yup indeed the query result is a nullptr | 00:14 |
|---|---|---|
| eugenio | getting late now, will return to it tomorrow | 00:15 |
| eugenio | thanks as always mal :) | 00:15 |
| steevie | Still awake | 03:20 |
| r0kk3rz | still in awe of piggz-o-vision? | 03:23 |
| steevie | Naah, my vision is better than the piggz one | 03:31 |
| T4 | <StillJustGerd> Hmm, i'm having trouble to apply the patch for bluetooth hci_smd | 06:24 |
| T4 | <StillJustGerd> getting these errors when compiling https://ghostbin.com/paste/9bb26 | 06:24 |
| 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 msm8916 | 06: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 it | 06: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.31 | 06:32 |
| T4 | <StillJustGerd> 3.18.something @birdzhang | 06:32 |
| T4 | <StillJustGerd> @austenite i'm working on the motorola moto g5s, so not the same device | 06:32 |
| T4 | <birdzhang> wow, try this https://github.com/Sailfish-On-Vince/Xiaomi_Kernel_OpenSource/commit/f633f265486229130b3024937365e605eb2be33f | 06:32 |
| T4 | <StillJustGerd> @birdzhang testing it right now | 06:34 |
| T4 | <StillJustGerd> should i make clean | 06:34 |
| T4 | <birdzhang> fingers crossed xD | 06:34 |
| T4 | <StillJustGerd> just did a `make clean`, now `make -j4 hybris-hal` again, ccache should make it faster | 06: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 mic | 06: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 see | 06:46 |
| T4 | <StillJustGerd> building finished, I'm trying to flash now | 06:54 |
| T4 | <StillJustGerd> just have to really quickly backup my lineageos /data | 06: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 again | 06: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 yet | 07:26 |
| T4 | <birdzhang> modprobe wlan | 07:27 |
| T4 | <austenite> @birdzhang whows nothing... | 07:30 |
| T4 | <austenite> @birdzhang wlan works after : echo sta > /sys/module/wlan/parameters/fwpath | 07:31 |
| T4 | <birdzhang> bluetooth sitll not work ? | 07:33 |
| T4 | <birdzhang> ^still | 07:33 |
| T4 | <birdzhang> oh, wait | 07:33 |
| T4 | <austenite> @birdzhang no | 07:34 |
| T4 | <birdzhang> https://github.com/piggz/droid-config-mido/blob/master/sparse/usr/bin/droid/droid-hcismd-up.sh | 07:34 |
| T4 | <birdzhang> @austenite does this script helpful ? | 07:43 |
| T4 | <birdzhang> it works on xiaomi mido and vince, msm8953 | 07:44 |
| T4 | <austenite> @birdzhang i will test report within few hours , i had reset the repo and cant build image @themoment | 07:49 |
| T4 | <birdzhang> okay | 07:50 |
| T4 | <adampigg> Mal @eugenio_g7 cool, so we can fork and build a fixed lipstick | 08:37 |
| T4 | <adampigg> https://openrepos.net/comment/28537#comment-28537 :) | 08:38 |
| dr_gogeta86 | hu | 09:43 |
| mal | @adampigg need to figure out why that really happens, according to specification eglQueryString should always return a string | 11:43 |
| mal | but I also found some conflicting information, not sure what is correct | 11:46 |
| mal | ah, indeed it can return null | 11:47 |
| T4 | <adampigg> Mal, cool,.someone submitted a pr from zoom, will have to take a look | 12:31 |
| T4 | <adampigg> Mal, but ill hold all prs until your work is merged | 12: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 work | 13:02 |
| T4 | <adampigg> Mal, you are focussed on fron camera and resolution models? | 13:05 |
| mal | yes | 13:10 |
| mal | and dynamically updating menus in general | 13:10 |
| mal | I have those updating fine but still some issues with other parts of camera switching | 13:10 |
| steevie | Steeve-o-vision | 13:14 |
| T4 | <austenite> stuck with this error, https://del.dog/sfos-garlic8.coffeescript | 18:11 |
| mal | which hybris base? | 18:29 |
| T4 | <austenite> @mal 14.1 | 18:33 |
| vknecht | not in pfsdk ? | 18:34 |
| mal | @austenite are you in platform sdk when you run that command? | 18:35 |
| T4 | <austenite> @mal yes.. | 18:36 |
| mal | the commandline in that paste doesn't look like it | 18:38 |
| mal | it 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 files | 18:38 |
| mal | you should fix the .mersdk files, makes it much easier to see everything is correct | 18:39 |
| T4 | <austenite> @mal ok, but i can confirm its running in platformsdk , it worked with rpm/dhd/helpers/build_packages.sh | 18:40 |
| steevie | Ferrite α | 18:40 |
| T4 | <austenite> @mal and if i give —configs it will never build agian | 18:41 |
| T4 | <austenite> ledeburite | 18:41 |
| mal | when 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 image | 18:42 |
| T4 | <austenite> and i cloned libhybris .. since then this error | 18:42 |
| T4 | <austenite> @mal , tried building libhybris .. https://del.dog/sfos-garlic9.coffeescript | 18:46 |
| mal | go to libhybris sources, in libhybris/hybris/common/n/linker_gdb_support.cpp replace all instances of _hybris_enable_android_debug with _linker_enable_gdb_support | 18:46 |
| mal | then build_packages.sh -b hybris/mw/libhybris | 18:46 |
| mal | @austenite does that help? | 19:01 |
| steevie | Bainite | 19:05 |
| T4 | <austenite> @mal it built now building image.. thanks alot | 19:07 |
| steevie | Meno mal! | 19:07 |
| T4 | <austenite> @steevie Im not very good at metallurgy | 19:11 |
| mal | @austenite why did you think your problem would be fixed by using upstream libhybris | 19:13 |
| mal | for browser issues there is one usual fix | 19:13 |
| mal | https://github.com/mer-hybris/droid-config-sony-nile/blob/master/sparse/var/lib/environment/nemo/99-qtscenegraph.conf | 19:14 |
| mal | @austenite what exact problem do you have with browser? | 19:15 |
| T4 | <austenite> @mal full white page, then had a pink browser | 19:16 |
| steevie | @austenite then why that nick? | 19:16 |
| mal | @austenite then that file I linked should fix it | 19:16 |
| T4 | <austenite> @steevie i like metallurgy but im not good at it :P | 19:16 |
| steevie | Aaand you know how to dimension structures? | 19:17 |
| steevie | At least that... | 19:17 |
| mal | @austenite for testing copy that file to your device to correct place | 19:17 |
| T4 | <austenite> @steevie dimension structures? | 19:19 |
| steevie | I 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 them | 19:22 |
| piggz_ | mal: @eugenio_g7: my simple lipstick fix didnt work :/ | 19:24 |
| steevie | Isostatic, iperstatic stuff, cinematics, esketit | 19:25 |
| mal | piggz_: can you show the patch you tried | 19:28 |
| piggz_ | mal: oh, shit, hangon..... | 19:29 |
| piggz_ | dont dare go looking for my mess up | 19:29 |
| piggz_ | :D | 19:29 |
| mal | @austenite I made a pull request for the libhybris fix to upstream | 19:30 |
| mal | piggz_: :D | 19:30 |
| mal | piggz_: I had a look, you know there is a much simpler way? | 19:32 |
| piggz_ | mal: likely | 19: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 function | 19:33 |
| mal | like const char *queryString = ...; supported = queryString && strstr... | 19:33 |
| piggz_ | yeah, thats sort of what i was going for what i was going for | 19:33 |
| piggz_ | but i didnt want to add another member to the struct | 19:34 |
| mal | or maybe better name for that variable is extentionsString | 19:34 |
| mal | or just extensions | 19:34 |
| piggz_ | anyway ... the real fuckup was that i built master and not fixes on obs | 19:34 |
| mal | ah | 19:35 |
| mal | anyway your version is a bit overly complicated now | 19:35 |
| piggz_ | and doesnt actually build.... | 19:36 |
| piggz_ | strstr returns const char* not int | 19:36 |
| piggz_ | mal: ok, i can go back to my original way and add another member ... you think thats preferable? | 19:38 |
| mal | what is your original way? | 19:39 |
| mal | @austenite sorry, I was wrong you need these 3 lines in droid-hal spec https://github.com/mer-hybris/droid-hal-f5121/blob/master/droid-hal-f5121.spec#L26 | 19:40 |
| mal | then rebuild droid-hal rpms and libhybris | 19:41 |
| mal | piggz_: I think a wrapper function should be used only if you need it many times | 19:43 |
| steevie | @austenite not only engineering | 19:43 |
| steevie | You should study it in high school as well | 19:43 |
| piggz_ | mal: well, my fix may not be the best, but oit does work :) | 19:45 |
| piggz_ | @eugenio_g7 ^^ | 19:45 |
| mal | piggz_: doesn't my way work, much simpler imo | 19:45 |
| mal | just a local variable for the egl query result and then check it and use it for strstr | 19:46 |
| piggz_ | mal: im sure it will...... | 19:46 |
| mal | hmm, but that is a struct | 19:47 |
| mal | makes it more complicated | 19:47 |
| piggz_ | yes, that was why i went my way.... | 19:49 |
| piggz_ | but im trying your way now | 19:49 |
| piggz_ | i think it will work | 19:49 |
| mal | it won't | 19:50 |
| mal | we can't define variable normally in struct | 19:50 |
| mal | I mean to be used in other parts of the struct | 19:51 |
| piggz_ | ah | 19:52 |
| piggz_ | mal: wold have to duplicate the function call then | 19:53 |
| piggz_ | which is also messy | 19:53 |
| piggz_ | mal: it builds ok | 19:57 |
| piggz_ | mal: your way works too | 20:03 |
| mal | piggz_: but wouldn't my way create a useless variable in the struct, and it would remain there | 20:20 |
| piggz_ | mal: it would ... thats what i said first :D | 20:20 |
| mal | well I didn't focus enough | 20:21 |
| piggz_ | mal: so, my original way? | 20:22 |
| mal | I know another ugly way, maybe your way is better, it could be done using some ternary operator magic but it wouldn't look nice | 20:23 |
| *** PragmaticAnarchi is now known as Unbegriff | 20:27 | |
| mal | piggz_: maybe the fix could use a mer bug reference | 20:34 |
| piggz_ | mal: i also discarded the ternary option :) | 20:57 |
| piggz_ | as uglier | 20:57 |
| piggz_ | so, i will go back to mine, add a bug report, and put the big in the commit | 20:58 |
| piggz_ | and PR | 20:58 |
| mal | piggz_: good, thanks | 20:59 |
| piggz_ | mal: what is the format.... | 21:05 |
| piggz_ | mer #nnnn ? | 21:05 |
| piggz_ | in the commit coment | 21:05 |
| T4 | <eugenio_g7> Fixes MER#nnnn | 21:08 |
| piggz_ | @eugenio_g7 https://git.merproject.org/mer-core/lipstick/merge_requests/110 | 21:13 |
| piggz_ | also, i think i fixed the sensors | 21:13 |
| mal | piggz_: do you want to hear some nitpicking :P | 21: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 up | 21:14 |
| mal | piggz_: capital letter are nice thing, also a bit long commit message as it doesn't fit well into one line | 21:15 |
| T4 | <eugenio_g7> piggz_: nice nice | 21:18 |
| piggz_ | mal: i dont know what you are talking about, my commit message looks perfect https://git.merproject.org/piggz/lipstick/commit/e86827214826d6ebc2f6ece0070bb0b562bd2802 | 21:19 |
| piggz_ | :D | 21:19 |
| piggz_ | @eugenio_g7 try that out for sensors and see if its better | 21:20 |
| T4 | <eugenio_g7> Will do soon-ish | 21:21 |
| eugenio | piggz, I can confirm that both the lipstick and sensors fixes work as expected :) | 22:14 |
| T4 | UgeneR was added by: UgeneR | 22:18 |
| piggz | eugenio: \o/ | 22:33 |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!