<div dir="ltr"><div><div>Hi,<br></div>I once installed master branch, so runcommand plugin from that was causing problems. It is solved now. I installed ssl branch on my laptop and on virtual box, I was not able to find out any error as Albert pointed out earlier. Tried couple of pair and unpair, then unpair after disabling network then pairing again. Everything was working fine.<br></div>Vineet<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 30, 2015 at 4:21 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"><div class="HOEnZb"><div class="h5">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 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 to<br>
> ssl branch where that plugin is not present, it is still loading that 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>> wrote:<br>
>><br>
>> In my opinion the loopback pairing handler should automatically accept the<br>
>> pairing as soon as it's requested.<br>
>><br>
>> I would say if you do some manual testing pairing and unpairing different<br>
>> devices in different order, you will find some problems fast enough. 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>> wrote:<br>
>>><br>
>>> Hi<br>
>>><br>
>>> One problem what I found out that was causing tests to fail, is repeated<br>
>>> pair packages. The main cause for that was I was using LanPairingHandler for<br>
>>> LoopbackLink because functionality is same but this was causing errors since<br>
>>> the pair status is set in LanPairingHandler after sending pair package<br>
>>> successfully which is correct for LanPairingHandler but in Loopback link,<br>
>>> reply is received before setting pair status and it interprets as pairing<br>
>>> request. This happens in accept pairing method too, where also status is set<br>
>>> after sending pair package successfully causing an infinite pair package<br>
>>> loop in loopback link. I added a Loopback pairing handler which inherits<br>
>>> from LanPairingHandler and corrected these issue. Infinite pair packages<br>
>>> problem is solved but tests are still failing.  Plugin load test is failing<br>
>>> due to kdeconnect_remotecontrol plugin, and file transfer test is failing<br>
>>> with issue that there is no such signal fileReceived in plugin. I am pushing<br>
>>> changes to sslmaster with commit "Added loopback pairing handler". @Aleix<br>
>>> can you again look at those tests? @Albert, you said that there are 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 am<br>
>>>> setting another environment in virtual box to check it. It is taking 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 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 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 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 <<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 <<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 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 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 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 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>
</div></div>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>
<span class="HOEnZb"><font color="#888888"><br>
Aleix<br>
</font></span></blockquote></div><br></div>