Sunday, 2022-09-11

T42<elros34> Any particular reason why there is no cpio in allowed tools patch in hybris-patches for hybris-18? TEMPORARY_DISABLE_PATH_RESTRICTIONS doesn't work anymore and that part is quite critical or this issue only affects me?13:13
mallooks like I have a patch for that in my local tree which I forgot to push to hybris-18.113:20
T42<elros34> thanks. I think this commit should also by in hybris-18 branch
malfeel free to make a PR, I can approve and merge it, please test my patch also and approve if ok13:24
T42<elros34> ok13:24
malI also should rebase 18.1 manifest13:26
malbut that can be done separately from that simple change13:26
T42<elros34> I have used hybris-18.1-update.13:32
T42<elros34> branch13:32
T42<elros34> at least it builds fine13:32
maloh, did I make one :D13:33
malnot sure why I forgot to make a PR for that13:34
malprobably better to update it again13:34
malI can try to update it again13:35
mal@elros34 I'm making a new update branch, I can include removal of that one repo in it13:41
malyou can then test it13:41
T42<elros34> ok13:41
mal@elros34 let me know if that works13:44
malhmm, I probably should also sync the patches with that13:57
malbut would be first nice to get the cpio patch in13:58
T42<elros34> anybody with failed to build droidmedia because of listAudioSessions error:
T42<elros34> I hope this will not break sailfish x14:04
T42<elros34> is CONFIG_AUTOFS4_FS really needed by anything, especially by hybris-boot script?14:14
malnot sure really14:14
T42<elros34> it's required by hybris-boot init-script, optional in mer-kernel-check and in reality documentation talks something  about network file system14:17
malbased on the description of that seems unlikely14:17
malthat option is optional for systemd but recommended, probably not really important for us14:19
mal@elros34 you can always make a PR to hybris-boot repo to remove such optional things14:23
T42<elros34> yeah just don't wont to break something14:24
malthat needs to be tested14:28
malI couldn't find any explanation why that is in the required list14:28
T42<Verevka86> Good work 👍🤝 (re @elros34: anybody with failed ...)17:38
T42<elros34> did you test it?17:44
T42<elros34> I only tested it on hybris-18, would bee good to know whether it works on other bases17:46
T42<Verevka86> Yes, hybris18.1 (re @elros34: did you test it?)17:47
mal@elros34 I can make some test builds of that droidmedia PR for official devices17:49
T42<elros34> if 'cat frameworks/av/services/audiopolicy/service/AudioPolicyService.h | grep listAudioSessions' fails then I think it shouldn't break anything17:50
T42<Verevka86> and no one solved the problem with playing the standard sailfishos 4 video in the gallery?18:17
T42<elros34> mal: I have repo init to hybris-18.1-update-new and repo sync --force-sync. Build finished without issues.18:28
T42<Verevka86> I also checked, compiled and run without problems (re @elros34: mal: I have repo ini...)18:34
T42<Verevka86> and no one solved the problem with playing the standard sailfish_OS_3.0.mp4 video in the gallery? (edited)18:35
malwhich device and android base?18:43
T42<Verevka86> Xiaomi vayu hybris-18.118:54
T42<Verevka86> Xiaomi vayu android 11.0 (edited)18:55
mal@elros34 if that branch works then I will make PR for it20:12
T42<elros34> that would be great20:13
mal@elros34 can you check how big external/chromium-webview/prebuilt/ is? wondering if we need those prebuilts20:15
T42<elros34> 223,2 MB20:19
malso not huge but quite big20:20
T42<elros34> there are only *.apk files there20:20
malok, can you check if it still build if you remove those20:20
malit could have some dependency, hopefully not20:20
malthere is that chromium patches repo which might affect it also20:21
T42<elros34> removed external/chromium-webview/prebuilt/ and seems to build fine20:30
malok, I will comment out those from my branch then20:31
ThaodanIf I read is see a use case for it, e.g. if remote file systems such as cifs or nfs are used. In any case the impact on the kernel isn't bug enough to make it worth skipping.20:40
malThaodan: the problem is that mer-kernel-check doesn't require it but init-script does20:41
Thaodanmal: but only the one from hybris-boot?20:43
maland we usually tell porters to only fix errors so that can cause issues if it's not enabled by default20:44
maleither we for it in checker or remove it from init-script20:44
mal*force it20:44
Thaodanthat's od, but I see now.20:48
ThaodanIt comes from
malyes, not sure why it is listed as required20:49
ThaodanI don't know why the check was made but it is quite fragile since kernel symbols change between versions and require checking twice.20:49
ThaodanI would vote to remove the check and solely relay on mer_verify_kernel_config.20:50

Generated by 2.17.1 by Marius Gedminas - find it at!