Tuesday, 2018-10-16

*** frinring_ is now known as frinring02:33
tortoisedocmal : https://build.merproject.org/package/live_build_log/home:tortoisedoc:branches:home:sfietkonstantin:sailfish:rust/rust-1.27.0/sailfishos_i486/i58606:33
tortoisedocare there ways to control memory assignment to the build hosts?06:33
dcalisteHello chriadam and jpetrell!07:08
chriadamdcaliste: hi07:08
jpetrelldcaliste: hi07:08
dcalistechriadam, I wish you feel better day by day.07:09
chriadamdcaliste: slowly improving, thanks.  probably going to be a couple more weeks until the inflammation settles down entirely, but c'est la vie07:09
dcalisteYeh, have patience, good luck.07:10
dcalisteConcerning signature handling, I've seen you approved the QMF patch, thank you.07:10
dcalisteWhen accepted, I'll update https://git.merproject.org/mer-core/messagingframework/merge_requests/1807:11
chriadamI was planning to stage that one after I have a chance to chat to Matt on Thursday07:12
chriadamso Thursday or Friday with luck it will integrate upstream07:12
dcalisteThis PR is updating the spec file to package the new crypto plugin files and also is adding account settings for signatures.07:12
dcalisteThis account part cannot go upstream yet because it is based on the SSO patch we have in MER.07:13
dcalisteOk for the planning, good, good.07:13
dcalisteThen, there is a MR in nemo-qml-plugin-email that must be coordinated with the MR in ui-jolla-email because I've slightly changed the way mail are built and sent.07:14
chriadamwill that one affect also the sailfish-eas / activesync side?  e.g. will other plugins have to adapt to nemo-qml-plugin-email changes?07:15
dcalisteWhat was before:07:16
dcaliste- mail is built in nemo-qml-plugin-email;07:16
dcaliste- mail is then queue for sending;07:16
dcaliste- page is popped.07:18
dcalisteNow the following scheme is in place:07:19
dcaliste- mail is built in nemo-qml-plugin-email;07:19
dcaliste- if there is no signature, mail is queue for sending and sendEnqueued is emitted;07:20
dcaliste- if there is a signature, the passphrase is required and the sendEnqueued signal is emitted only when mail is signed and ready;07:20
dcaliste- ui-jolla-email catch the sendEnqued signal and pop the page.07:20
dcalisteIn case of error in signature, the page stay there and the user is prompt to either send it again (and enter again properly his passphrase), or uncheck the checkbox asking for signature.07:21
dcalisteSo I think this won't affect activesync IMHO, it's only an additional step between building the message and sending.07:23
dcalistechriadam & jpetrell: I need some feedback though from design, because I'm not entirely satisfied with the behaviour. When you can test sending a message, you'll see that the UX is not as smooth as it can be. Of course when resource available to look into this…07:24
chriadammakes sense, something in jolla-email / ui side only.07:25
chriadamI guess there will be some capacity available for checking UX after the current release?  jpetrell ^07:26
dcalistechriadam: yes I think so. The diff is quite clear in components/EmailComposer.qml I think to see the additional step.07:26
chriadammaybe we can merge it as-is and then iterate the UX afterward, if needed?07:26
dcalisteThat would be great.07:26
dcalisteFor normal email, it has no consequences.07:26
dcaliste(Besides side effects on plugins, but I don't see why, everything is between nemo-qml-plugin-email and ui-jolla-email).07:27
jpetrelldcaliste: not smooth how?07:28
jpetrellchriadam, dcaliste: yeah I will review UX once we get the Sailfish 3 mature07:29
dcalisteYou have some tenth of seconds spent between pulling "send" and the system window password appearing asking for the passphrase.07:29
jpetrellyeah that is not good. any idea why?07:29
dcalisteWhen password is wrong, I've tried to emphase what is wrong highlighting the signature checkbox, but I don't like it much.07:30
dcalistejpetrell: difficult to avoid: the pinentry is a separated process.07:30
jpetrellpublish notification describing the error to user? with warning icon07:30
dcalisteWill take time to be trigerred.07:30
chriadamthe IPC will take time.  jolla-email -> secrets, secrets -> gpg, gpg -> secrets_auth_plugin, secrets -> pinentry.  will need a busy spinner to show that the send operation is in progress.07:31
dcalisteMaybe it will require to add a busy something on composer during waiting time.07:31
dcalistechriadam ;)07:31
jpetrellyeah since email app knows signature checkbox is checked it should show some kind of loading state, e.g. inline banner with busy indicator and label07:31
dcalisteCurrent implementation in composer is quite crude, but since there were various options, I wanted you to try and give me your opinion before trying various options.07:32
jpetrelloverall not good to have so much latency in in-device use cases07:32
jpetrellok07:33
dcalisteFor the error, I didn't go the notification and wanted to have something in-app because the page is still present.07:33
dcalisteOnce everything is in place to easily test for you, I think we can iterate quite quickly to find a nice and smooth solution.07:34
jpetrellwe often handle such transient error states where operation fails with notifications. in-app in general would be better but notification is more consistent with other platform use cases07:34
jpetrelldcaliste: that is true. UI-wise these are easy solutions07:35
dcalisteOk for notification then. I'll give it a try maybe in a separate branch so we can test both side by side.07:35
dcalistechriadam: I know you're very busy and have more pressing topics. So when you have a moment, may you give a look to https://git.merproject.org/mer-core/nemo-qml-plugin-email/merge_requests/39 ?07:39
chriadamI will certainly take a look, hopefully Thursday07:39
dcalisteParticularly to the discussion ? It's about allowing to sync several mail folders, not only the Inbox one.07:40
chriadamsure07:40
dcalisteAll the code is already available in QMF, but Inbox is hardcoded in buteo-sync-email and in it's relation with the EmailAgent in nemo-qml-plugin-email.07:41
dcalisteThank you anyway ;)07:41
chriadamthank you very much for your work07:41
chriadamjust for clarity, what are next steps?07:41
chriadam1) qmf upstream merge07:41
chriadam2) mer-qmf updated?07:41
chriadam3) n-q-p-e reviewed + merged07:41
chriadam4) jolla-email reviewed + merged07:41
chriadam5) jolla-settings-accounts merged?07:42
chriadam6) need to merge secrets gpg plugin07:42
chriadam7) anything else?07:42
dcalisteNo 7), exactly these 1 to 6.07:42
dcaliste1-4 will provide signature checking.07:42
chriadamok.  does (2) need input from our side at this stage?07:42
dcalisteNo, I take care of 2) myself. Waiting for upstream merge to update submodule.07:43
chriadamok07:43
dcaliste4-7 will provide signature of new emails.07:43
dcalistechriadam, last question, where can I discuss with pvuorela, mvogt and yourself a design issue in QMF?07:43
dcalisteIt's about handling attachment-only emails.07:44
chriadamso then I will do (1), (3), (5), (6) next week (fingers crossed).  jpetrell hopefully will get a chance to do (4) next week.07:44
dcalisteI mean, no multipart, just the message being disposition:attchment.07:44
dcalistechriadam: great, I'll update 2) (required for 3) as sson as 1) is done.07:44
chriadamI can ask matt if he can join this channel on Thursday or Friday, I suppose07:45
chriadamnot 100% sure if he will have meetings or so forth on at this time though07:46
dcalistechriadam: yeh, thanks, because I don't see any way to handle this without major changes in QMailMessage which will require a lot of discussions to make it properly.07:46
chriadamah07:46
dcalisteSee https://together.jolla.com/question/190602/bug-email-client-unable-to-handle-content-transfer-encoding-base64/07:47
dcalisteI've checked RFC, IMHO this email is valid.07:47
dcalisteIt is treated propely by QMailMessage, but there is no attachment, because attachment is supposed to be a MailPart, while Message doen't inherit MailPart.07:48
chriadamok, I will mention this to Matt and see if he's available on Thursday or Friday for a chat. please ping me on Thursday (or Friday, whichever suits you best)07:49
dcalisteSo, if you think that the issue requires indeed discussion, that would be nice if we can have a meeting at one moment. It's not in a hurry though.07:49
chriadamhopefully pvuorela_ will be free then also07:50
dcalisteThursday or Friday morning is fine with me. I'll ping around 7am UTC. If not everyone is here, we can move it to another day/week, no problem.07:51
chriadamthanks07:51
chriadamgood that we have clear path forward, for the moment too.  Just depends on whether we have time to do the review and merge next week.  Thanks very much once again for your great contribution and communication.07:52
dcalisteGreat, thank you chriadam also for mentoring and review work.07:53
chriadamand I'll poke Matt about Thursday :-)07:54
chriadamis there anything else?  if not, have a great evening07:54
dcalisteNothing more indeed. Maybe just a word also for jpetrell about Calligra. I've corrected the performence regression issue. Latest can be tested and should be more or less to the same level as the older 2.9 we currently have.07:56
dcalisteI've upstreamed all patches making sense and am trying to propose some additional modifications upstream also to avoid one or two more.07:58
dcalisteRemaining patches are about patching build system because we don't have KF5 packages and I've tried not to compile so many of them or with full capabilities when it was not related to document viewing.07:59
chriadamthat's great news.  jpetrell, pvuorela_ what's the plan with that calligra upgrade?  3.0.1 content, or is that too risky?  or did it require something more invasive (e.g. new Qt or so)?08:00
dcalisteI've packaged LATEST (a bit more than 3.1.0). It's not requirering Qt upgrade, I've patched what was needed for 5.6 to work.08:01
dcalisteNormally, you can directly test the calligra packages from my OBS without touching anything else on device.08:01
dcalistehttps://build.merproject.org/package/show/home:dcaliste/calligra-bundle08:01
dcalisteLater modifications will require to update sailfish-office also for newer functionalities like annotation, text search or even why not edition but that's for later when calligra latest is in MER.08:03
chriadamcool08:04
chriadamok, I will poke pvuorela_ again about that one next week08:04
chriadamI guess everyone is super busy until then at the very least08:04
dcalisteSure, I understand, just to keep you informed about advances.08:07
chriadam:-)08:07
dcalisteSo see you next Thursday or Friday. Have a nice evening.08:11
chriadamthanks, you too08:11
munchausenchem|st yes sorry I meant the native sailfish pebble app, I can't remember what it's called08:42
munchausenIt's rockpool08:43
munchausenFrom https://github.com/abranson/rockpool seems it's still reasonably well supported, it has switched to the rebble app store so looks good08:44
chem|stmunchausen: just clarifying BT+android09:28
munchausenchem|st yeah cool. It's actually a shame for things like smart watches, but solutions do exist10:06
rs33smart watches should use Sharp's memory-TFT10:09
rs33sorry for OT10:09
munchausenrs33 yeah I agree, I use a pebble10:09
munchausenAlthough, the pebble panels are actually made by JDI10:10
munchausenIIRC10:10
rs33memory TFT or E-Ink?10:10
rs33unfortunately the mfgrs seem to think consumers want bright oled candy-colors on a watch10:11
munchausenmemory tft10:11
munchausenSame sort of thing as sharp10:11
rs33cool10:11
munchausenhttps://www.j-display.com/english/product/wearable.html10:11
rs33you could get a 4" n900 style screen on a wristband10:12
rs33and my idea: put a ring of thin cylindrical batteries on the watchband10:12
rs33get the batt out of the watch body10:12
rs33make it as slim as a normal watch10:12
munchausenThe pebble time/time steel were supposed to have those sort of features10:14
munchausenSmart bands I think they were called10:14
rs33nice10:14
munchausenThe batteries I mean10:14
munchausenUnfortunately pebble got bought out not long after they were released10:15
rs33too many innovative companies getting 'bought out'10:15
rs33but this is too offtopic10:15
munchausenBut they have the connectors to support power/serial comms with watch bands10:16
munchausenYeah :)10:16
munchausenThey do compliment sailfish well though, I hope :)10:16
munchausens/compliment/complement10:17
rs33sort of10:18
rs33i don't know how well the reliance on animated elements (swipes) would work on memory-tft10:19
rs33but can you sell any OS where things don't move and animate pointlessly nowadays...10:19
munchausenOh I meant as an accessory to a sailfish phone... I'm not sure about touch screens on watches at all to be honest10:19
rs33yeah that's why i say 4" screen watchband10:20
rs33these 1.8" screens are not so great10:20
munchausenAhh10:20
rs33but that 4" body would need to be slim10:20
rs33or it would look terrible10:20
rs33i'd like a nice high dpi of course, so you could read a half page of info10:21
rs33and built-in radio/sim10:22
tortoisedocyellow12:18
tortoisedocany way out of this cul-de-sac -> https://build.merproject.org/package/show/home:tortoisedoc:branches:home:sfietkonstantin:sailfish:rust/cargo?expand=112:18
tortoisedocping Nokius ^, mal ^12:22
maltortoisedoc: how exactly are you trying to build that, did you manually upload the source package there?12:26
tortoisedocmal : i tried to upload a file via a url12:27
tortoisedocand BANG12:27
malmaybe try to just manually upload the source package, an example here https://build.merproject.org/package/show/home:mal:anbox/boost12:27
tortoisedocok thanks, ill try that later12:28
*** oku_ is now known as oku15:40
*** MMori_ is now known as MMori17:43
flav0rhi im looking to install sailfish x but i have a question due to ambiguity in the instructions20:48
flav0rthis is for a sony xperia20:49
flav0ranyway the instructions say to check if i have an unlockabkle bootloader.   apparently i do20:49
flav0rbut then it talks about if the bootloader is UNLOCKED before updating android20:50
flav0rhow do i check if its actually unlocked20:50
flav0ror is this a bit misleading and its just taliing about if its unlockable20:50
flav0roh apparently its locked because it says its unlockable20:52
*** Renault_ is now known as Renault22:43

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