<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Nov 29, 2014 at 6:09 PM, Nitul Datt <span dir="ltr"><<a href="mailto:nitul1991@gmail.com" target="_blank">nitul1991@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><p dir="ltr"><br>
On 29 Nov 2014 05:48, "vedant agarwala" <<a href="mailto:vedant.kota@gmail.com" target="_blank">vedant.kota@gmail.com</a>> wrote:<br>
><br>
><br>
><br>
> On Sat, Nov 29, 2014 at 5:39 AM, Nitul Datt <<a href="mailto:nitul1991@gmail.com" target="_blank">nitul1991@gmail.com</a>> wrote:<br>
>><br>
>> Hey Vedant,<br>
>><br>
>> On 27 Nov 2014 05:50, "vedant agarwala" <<a href="mailto:vedant.kota@gmail.com" target="_blank">vedant.kota@gmail.com</a>> wrote:<br>
>><br>
>> ><br>
>> > Nitul,<br>
>> ><br>
>> > "undefined reference" errors are caused when the different files are not linked together properly.<br>
>> ><br>
>> > You obviously have this problem as you have added new files (CueSheet.h|cpp) but have not made any entry of it in the CMakeLists.txt. See how the files are added to CMakeLists.txt and do the same for your CueSheep.cpp<br>
>> ><br>
>><br>
>> I've looked that up.<br>
>><br>
>> > There are many more errors in your diff that we can go about later when I review your code.<br>
>> ><br>
>><br>
>> Unfortunately my laptop's hard disk has packed up. So I'll be delayed by a day or two while I get it fixed. I'll be setting up my development environment on a friend's laptop in the meanwhile. I'd like to sincerely apologise for the delay.<br>
><br>
><br>
> Well its okay. As long as you make it up later on :)<br>
><br>
> Also, I would suggest you to set up your own machine rather than a friend's. Fix yours first. Just delete the windows partition :-P</p>
</span><p dir="ltr">I'm afraid it isn't so simple. I have an ssd and a traditional hd on my laptop. The HD isn't being detected. I took it out of the laptop and tried connecting it to another system using a master slave connector. It wasn't detected there either.<br>
So I've dropped off my system at the service center and I'll be getting it back on Tuesday. </p>
<p dir="ltr">In the meanwhile I've arranged a friend's old netbook on which I've set up Git and got the source. I'll try and get the build system running on it as well. If you could review the earlier code I sent you while I do that, it would be great.</p></blockquote><div>I will review the code after it builds successfully. Right now I can only tell you to fix the whitespace errors.</div><div><br></div><div>And, setting up git is only half the story, most importantly you need all the dependencies to build amarok.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr"> </p>
<p dir="ltr">On an unrelated note, I don't have Windows on my laptop. I've been running only Linux on it for about 2 years now. :-p</p></blockquote><div>Haha. That is good to hear.</div><div><br></div><div>Regards,</div><div>Vedant.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
<p dir="ltr">> You can boot from a pen drive and back up your data and create new partitions <br>
>><br>
>> > Regards,<br>
>> > Vedant.<br>
>> ><br>
>> ><br>
>> > On Thu, Nov 27, 2014 at 5:43 AM, vedant agarwala <<a href="mailto:vedant.kota@gmail.com" target="_blank">vedant.kota@gmail.com</a>> wrote:<br>
>> >><br>
>> >><br>
>> >><br>
>> >> On Thu, Nov 27, 2014 at 12:58 AM, Nitul Datt <<a href="mailto:nitul1991@gmail.com" target="_blank">nitul1991@gmail.com</a>> wrote:<br>
>> >>><br>
>> >>> Hey Vedant,<br>
>> >>><br>
>> >>> On 11/26/14, vedant agarwala <<a href="mailto:vedant.kota@gmail.com" target="_blank">vedant.kota@gmail.com</a>> wrote:<br>
>> >>> > Hello Nitul,<br>
>> >>> ><br>
>> >>> > Also paste a link to the error using <a href="http://paste.kde.org" target="_blank">paste.kde.org</a> in the description.<br>
>> >>> ><br>
>> >>><br>
>> >>> I have done so. The link is as follows: <a href="http://goo.gl/Csp8uW" target="_blank">http://goo.gl/Csp8uW</a><br>
>> >>><br>
>> >>> > It is a very large diff to analyze. seems like you have added a lot of<br>
>> >>> > redundant lines in the form of comments. remove them. You should have the<br>
>> >>> > old code in git and not need to keep them as comments.<br>
>> >>> ><br>
>> >>><br>
>> >>> I have removed the appropriate lines and updated my diff.<br>
>> >>><br>
>> >>> > I will take some time to run the code myself. I can only look at the code<br>
>> >>> > and see if the error is obvious.<br>
>> >>> ><br>
>> >>><br>
>> >>> That would be great.<br>
>> >>><br>
>> >>> > In any case, also provide a link to the personal scratch repo commit so<br>
>> >>> > that i can directly pull from there and run the code when I can.<br>
>> >>> ><br>
>> >>><br>
>> >>> I haven't commited the code as of yet as it still isn't working properly.<br>
>> >><br>
>> >><br>
>> >><br>
>> >> I know it isn't. That is why you will upload to *your personal* repository.<br>
>> >>  <br>
>> >>><br>
>> >>><br>
>> >>> > Regards,<br>
>> >>> > Vedant.<br>
>> >>> ><br>
>> >>> > On Wed, Nov 26, 2014 at 2:01 AM, Nitul Datt <<a href="mailto:nitul1991@gmail.com" target="_blank">nitul1991@gmail.com</a>> wrote:<br>
>> >>> ><br>
>> >>> >> Hey!<br>
>> >>> >><br>
>> >>> >> I've put up an initial patch on reviewboard. It's not completely error<br>
>> >>> >> free and I need some help figuring out what's wrong.<br>
>> >>> >><br>
>> >>> >> --<br>
>> >>> >> Regards,<br>
>> >>> >> Nitul Datt<br>
>> >>> >><br>
>> >>> ><br>
>> >>><br>
>> >>> --<br>
>> >>> Regards,<br>
>> >>> Nitul Datt<br>
>> >><br>
>> >><br>
>> ><br>
>><br>
>> --<br>
>> Regards, <br>
>> Nitul Datt<br>
><br>
></p>
</div></div><p dir="ltr">--<br>
Regards,<br>
Nitul Datt<br>
</p>
</blockquote></div><br></div></div>