<div dir="ltr"><div>Because I don't have two laptops to test on them, so need to set the the whole environment on virtual box. Well, I merged the latest master branch with sslmaster in my local branch, will test it till weekend to be more confident and will let you know.<br><br></div>Vineet<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 30, 2015 at 6:52 PM, 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">Why don't you try it on a proper non-virtualized system? How confident<br>
are you that this can go in master?<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 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 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>> wrote:<br>
>> > Hi<br>
>> ><br>
>> > Sorry for a very late reply, I had my exams. As Albert said I tried 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 with<br>
>> > SEGFAULT. Even after disabling that plugin from cmake file or 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 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 errors<br>
>> >>> since<br>
>> >>> the pair status is set in LanPairingHandler after sending pair 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 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 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, so I<br>
>> >>>> am<br>
>> >>>> setting another environment in virtual box to check it. It is 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>> wrote:<br>
>> >>>>><br>
>> >>>>> On Sat, Sep 12, 2015 at 4:08 PM, vineet garg <<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 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) 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 unit<br>
>> >>>>> > tests are<br>
>> >>>>> > passing because it is failing on my laptop and reason might be 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 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" <<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 meanwhile.<br>
>> >>>>> >>>><br>
>> >>>>> >>>> We got the SSL connection working from Android to Android 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 seen<br>
>> >>>>> >>>> that<br>
>> >>>>> >>>> the<br>
>> >>>>> >>>> tests you wrote are also not passing, which makes me think 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 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>
</div></div><div class="HOEnZb"><div class="h5">_______________________________________________<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>
</div></div></blockquote></div><br></div>