<div dir="ltr"><div><div>Hi,<br></div>I merged latest android changes with ssl. I am having a little problem with new capabilities feature. Like I sent a file from one device to another and noticed that there was no plugin loaded on other device. I even noticed a loop of capabilities package after pairing. Can you tell me a little about this like when these capabilities list got initialized and how it is refreshed so that I can find and fix the issue.<br><br></div>Vineet<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 1, 2015 at 3:41 AM, Aleix Pol <span dir="ltr"><<a href="mailto:aleixpol@kde.org" target="_blank">aleixpol@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ah, and you tested KDE <-> Android as well, right? That's what we tested mainly.<br>
<span class="HOEnZb"><font color="#888888"><br>
Aleix<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Wed, Sep 30, 2015 at 9:25 PM, vineet garg <<a href="mailto:grgvineet@gmail.com">grgvineet@gmail.com</a>> wrote:<br>
> Because I don't have two laptops to test on them, so need to set the the<br>
> whole environment on virtual box. Well, I merged the latest master branch<br>
> with sslmaster in my local branch, will test it till weekend to be more<br>
> confident and will let you know.<br>
><br>
> Vineet<br>
><br>
> On Wed, Sep 30, 2015 at 6:52 PM, Aleix Pol <<a href="mailto:aleixpol@kde.org">aleixpol@kde.org</a>> wrote:<br>
>><br>
>> Why don't you try it on a proper non-virtualized system? How confident<br>
>> are you that this can go in master?<br>
>><br>
>> Aleix<br>
>><br>
>> On Wed, Sep 30, 2015 at 12:30 PM, vineet garg <<a href="mailto:grgvineet@gmail.com">grgvineet@gmail.com</a>> wrote:<br>
>> > Hi,<br>
>> > I once installed master branch, so runcommand plugin from that was<br>
>> > causing<br>
>> > problems. It is solved now. I installed ssl branch on my laptop and on<br>
>> > virtual box, I was not able to find out any error as Albert pointed out<br>
>> > earlier. Tried couple of pair and unpair, then unpair after disabling<br>
>> > network then pairing again. Everything was working fine.<br>
>> > Vineet<br>
>> ><br>
>> > On Wed, Sep 30, 2015 at 4:21 AM, Aleix Pol <<a href="mailto:aleixpol@kde.org">aleixpol@kde.org</a>> wrote:<br>
>> >><br>
>> >> On Sun, Sep 27, 2015 at 9:35 PM, vineet garg <<a href="mailto:grgvineet@gmail.com">grgvineet@gmail.com</a>><br>
>> >> wrote:<br>
>> >> > Hi<br>
>> >> ><br>
>> >> > Sorry for a very late reply, I had my exams. As Albert said I tried<br>
>> >> > some<br>
>> >> > manual testing, everything is working fine for me, except for one<br>
>> >> > problem<br>
>> >> > that as soon as it loads runcommand plugin, the application crashes<br>
>> >> > with<br>
>> >> > SEGFAULT. Even after disabling that plugin from cmake file or<br>
>> >> > switching<br>
>> >> > to<br>
>> >> > ssl branch where that plugin is not present, it is still loading that<br>
>> >> > plugin<br>
>> >> > which is still mystery to me.<br>
>> >> ><br>
>> >> > Vineet<br>
>> >> ><br>
>> >> > On Mon, Sep 14, 2015 at 5:31 PM, Albert Vaca <<a href="mailto:albertvaka@gmail.com">albertvaka@gmail.com</a>><br>
>> >> > wrote:<br>
>> >> >><br>
>> >> >> In my opinion the loopback pairing handler should automatically<br>
>> >> >> accept<br>
>> >> >> the<br>
>> >> >> pairing as soon as it's requested.<br>
>> >> >><br>
>> >> >> I would say if you do some manual testing pairing and unpairing<br>
>> >> >> different<br>
>> >> >> devices in different order, you will find some problems fast enough.<br>
>> >> >> Just<br>
>> >> >> try it.<br>
>> >> >><br>
>> >> >> On Mon, Sep 14, 2015 at 4:05 AM, vineet garg <<a href="mailto:grgvineet@gmail.com">grgvineet@gmail.com</a>><br>
>> >> >> wrote:<br>
>> >> >>><br>
>> >> >>> Hi<br>
>> >> >>><br>
>> >> >>> One problem what I found out that was causing tests to fail, is<br>
>> >> >>> repeated<br>
>> >> >>> pair packages. The main cause for that was I was using<br>
>> >> >>> LanPairingHandler for<br>
>> >> >>> LoopbackLink because functionality is same but this was causing<br>
>> >> >>> errors<br>
>> >> >>> since<br>
>> >> >>> the pair status is set in LanPairingHandler after sending pair<br>
>> >> >>> package<br>
>> >> >>> successfully which is correct for LanPairingHandler but in Loopback<br>
>> >> >>> link,<br>
>> >> >>> reply is received before setting pair status and it interprets as<br>
>> >> >>> pairing<br>
>> >> >>> request. This happens in accept pairing method too, where also<br>
>> >> >>> status<br>
>> >> >>> is set<br>
>> >> >>> after sending pair package successfully causing an infinite pair<br>
>> >> >>> package<br>
>> >> >>> loop in loopback link. I added a Loopback pairing handler which<br>
>> >> >>> inherits<br>
>> >> >>> from LanPairingHandler and corrected these issue. Infinite pair<br>
>> >> >>> packages<br>
>> >> >>> problem is solved but tests are still failing.  Plugin load test is<br>
>> >> >>> failing<br>
>> >> >>> due to kdeconnect_remotecontrol plugin, and file transfer test is<br>
>> >> >>> failing<br>
>> >> >>> with issue that there is no such signal fileReceived in plugin. I<br>
>> >> >>> am<br>
>> >> >>> pushing<br>
>> >> >>> changes to sslmaster with commit "Added loopback pairing handler".<br>
>> >> >>> @Aleix<br>
>> >> >>> can you again look at those tests? @Albert, you said that there are<br>
>> >> >>> some<br>
>> >> >>> problems in KDE to KDE, can tell me the specific problems?<br>
>> >> >>><br>
>> >> >>> Vineet<br>
>> >> >>><br>
>> >> >>><br>
>> >> >>> On Sat, Sep 12, 2015 at 10:19 PM, vineet garg <<a href="mailto:grgvineet@gmail.com">grgvineet@gmail.com</a>><br>
>> >> >>> wrote:<br>
>> >> >>>><br>
>> >> >>>> Hi<br>
>> >> >>>><br>
>> >> >>>> As Albert pointed out earlier that KDE to KDE has some problems,<br>
>> >> >>>> so I<br>
>> >> >>>> am<br>
>> >> >>>> setting another environment in virtual box to check it. It is<br>
>> >> >>>> taking<br>
>> >> >>>> some<br>
>> >> >>>> time. I will push it afterward.<br>
>> >> >>>><br>
>> >> >>>> Vineet<br>
>> >> >>>><br>
>> >> >>>> On Sat, Sep 12, 2015 at 8:10 PM, Aleix Pol <<a href="mailto:aleixpol@kde.org">aleixpol@kde.org</a>><br>
>> >> >>>> wrote:<br>
>> >> >>>>><br>
>> >> >>>>> On Sat, Sep 12, 2015 at 4:08 PM, vineet garg<br>
>> >> >>>>> <<a href="mailto:grgvineet@gmail.com">grgvineet@gmail.com</a>><br>
>> >> >>>>> wrote:<br>
>> >> >>>>> > Hi<br>
>> >> >>>>> ><br>
>> >> >>>>> > I saw the tests and found out the issues. In one test<br>
>> >> >>>>> > startServerEncryption<br>
>> >> >>>>> > and startClientEncryption was wrongly written, fixed that. In<br>
>> >> >>>>> > one<br>
>> >> >>>>> > plugin the<br>
>> >> >>>>> > error is due to requestPairing since I was setting pairStatus<br>
>> >> >>>>> > after<br>
>> >> >>>>> > sending<br>
>> >> >>>>> > package and loopback (which is using lan pairing handler)<br>
>> >> >>>>> > replies<br>
>> >> >>>>> > even<br>
>> >> >>>>> > before it is set causing an issue. Will add a loopback pairing<br>
>> >> >>>>> > handler.<br>
>> >> >>>>> > Plugin load test is also failing, will check that too. File<br>
>> >> >>>>> > transfer<br>
>> >> >>>>> > plugin<br>
>> >> >>>>> > does not work completely in my laptop, can you confirm that<br>
>> >> >>>>> > unit<br>
>> >> >>>>> > tests are<br>
>> >> >>>>> > passing because it is failing on my laptop and reason might be<br>
>> >> >>>>> > due<br>
>> >> >>>>> > to<br>
>> >> >>>>> > because plugin does not work correctly on my laptop.<br>
>> >> >>>>> ><br>
>> >> >>>>> > Vineet<br>
>> >> >>>>> ><br>
>> >> >>>>> > On Sat, Sep 12, 2015 at 6:00 PM, Albert Vaca<br>
>> >> >>>>> > <<a href="mailto:albertvaka@gmail.com">albertvaka@gmail.com</a>><br>
>> >> >>>>> > wrote:<br>
>> >> >>>>> >><br>
>> >> >>>>> >> Could you have a look at the tests?<br>
>> >> >>>>> >><br>
>> >> >>>>> >> On Fri, Sep 11, 2015 at 9:58 AM, vineet garg<br>
>> >> >>>>> >> <<a href="mailto:grgvineet@gmail.com">grgvineet@gmail.com</a>><br>
>> >> >>>>> >> wrote:<br>
>> >> >>>>> >>><br>
>> >> >>>>> >>> Hi Albert,<br>
>> >> >>>>> >>> I was just working with netty folks to find out the issue<br>
>> >> >>>>> >>> with<br>
>> >> >>>>> >>> Android<br>
>> >> >>>>> >>> lollipop, I will look at it and see why test cases are not<br>
>> >> >>>>> >>> passing<br>
>> >> >>>>> >>> because I<br>
>> >> >>>>> >>> committed only when all test cases passed.<br>
>> >> >>>>> >>><br>
>> >> >>>>> >>> Vineet<br>
>> >> >>>>> >>><br>
>> >> >>>>> >>> On Sep 11, 2015 10:23 PM, "Albert Vaca"<br>
>> >> >>>>> >>> <<a href="mailto:albertvaka@gmail.com">albertvaka@gmail.com</a>><br>
>> >> >>>>> >>> wrote:<br>
>> >> >>>>> >>>><br>
>> >> >>>>> >>>> Hi Vineet,<br>
>> >> >>>>> >>>><br>
>> >> >>>>> >>>> We have merged the ssl and master branches, because SSL was<br>
>> >> >>>>> >>>> forked<br>
>> >> >>>>> >>>> a<br>
>> >> >>>>> >>>> while back and hadn't had merges from master in the<br>
>> >> >>>>> >>>> meanwhile.<br>
>> >> >>>>> >>>><br>
>> >> >>>>> >>>> We got the SSL connection working from Android to Android<br>
>> >> >>>>> >>>> with<br>
>> >> >>>>> >>>> the<br>
>> >> >>>>> >>>> merged branch. It now has all the latest changes, like the<br>
>> >> >>>>> >>>> Material design,<br>
>> >> >>>>> >>>> plus the SSL link, and seems to be working fine.<br>
>> >> >>>>> >>>><br>
>> >> >>>>> >>>> KDE to KDE, though, we are having some problems. We have<br>
>> >> >>>>> >>>> seen<br>
>> >> >>>>> >>>> that<br>
>> >> >>>>> >>>> the<br>
>> >> >>>>> >>>> tests you wrote are also not passing, which makes me think<br>
>> >> >>>>> >>>> that<br>
>> >> >>>>> >>>> there is<br>
>> >> >>>>> >>>> something broken. Can you have a look?<br>
>> >> >>>>> >>>><br>
>> >> >>>>> >>>> The merged branches are called "sslmaster", to not break the<br>
>> >> >>>>> >>>> master<br>
>> >> >>>>> >>>> branch. Once we have it working there, we will merge that<br>
>> >> >>>>> >>>> back<br>
>> >> >>>>> >>>> into master.<br>
>> >> >>>>> >>>><br>
>> >> >>>>> >>>> Thanks,<br>
>> >> >>>>> >>>> Albert<br>
>> >> >>>>> >><br>
>> >> >>>>> >><br>
>> >> >>>>> ><br>
>> >> >>>>><br>
>> >> >>>>> Hi,<br>
>> >> >>>>> You didn't push anything, can you push it to sslmaster?<br>
>> >> >>>>><br>
>> >> >>>>> Aleix<br>
>> >> >>>><br>
>> >> >>>><br>
>> >> >>><br>
>> >> >><br>
>> >> ><br>
>> >><br>
>> >> What Ashish said, this shouldn't be related to the Run Command. Remove<br>
>> >> it from the file system or something to properly test the SSL branch<br>
>> >> and get the shit together.<br>
>> >> Let's not let time pass because you'll forget about this and there<br>
>> >> won't be anyone up for merging the branch!<br>
>> >><br>
>> >> Aleix<br>
>> ><br>
>> ><br>
>> _______________________________________________<br>
>> KDEConnect mailing list<br>
>> <a href="mailto:KDEConnect@kde.org">KDEConnect@kde.org</a><br>
>> <a href="https://mail.kde.org/mailman/listinfo/kdeconnect" rel="noreferrer" target="_blank">https://mail.kde.org/mailman/listinfo/kdeconnect</a><br>
><br>
><br>
</div></div></blockquote></div><br></div>