<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><br></div></div></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 27, 2019 at 10:22 PM Mojca Miklavec <<a href="mailto:mojca@macports.org" target="_blank">mojca@macports.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Dear Mihir,<br>
<br>
On Wed, 27 Mar 2019 at 17:25, Mihir Luthra wrote:<br>
><br>
> Hi,<br>
><br>
> I have shared my draft application from the GSoC dashboard.<br>
> Please provide me with feedbacks. :)<br>
<br>
Please note again that I'm not familiar with the contents at all, so<br>
I'm providing just some general feedback.<br>
<br>
What I miss a bit is some clear definition of deliverables, what<br>
pieces of code would be suitable enough for merging them into base and<br>
when.<br>
<br>
Background: One of the problems with many earlier projects with<br>
macports base was that a student was working all summer in his own<br>
branch, waiting for the code to be "complete and well-tested", and the<br>
code was subsequently never merged into the trunk / master, or maybe<br>
it's still considered useful, just not 100% polished, and some members<br>
still plan to clean up and merge the code as old as 10 years. On the<br>
other hand, if the code ends up in the master branch early and some<br>
issues are discovered, they would still be fixed. If the code lies in<br>
its own branch, it doesn't get nearly as much testing and might get<br>
forgotten.<br>
<br>
Of course you cannot always merge immediately, as some pieces might<br>
need a bigger pile of code at once to produce something useful without<br>
breaking stuff. But it would be very helpful if some code could be<br>
merged into master at least once per week. If you could split it into<br>
smaller reasonable chunks, it would be even better to do this on an<br>
almost daily basis when possible (it's still ok to have two weeks of<br>
some bigger feature rework in the meantime).<br>
<br></blockquote><div><br></div><div><br></div><div>Thanks for the feedback Mojca. ^_^.<div><br></div><div>That definitely makes sense. </div><div>Clemens told me to work on trie, ctrie or related data structures in his last mail. </div><div>Most likely till weekend I will understand how to relate the main code with these data structs and plan where to place the desired functions.</div><div>Will also try to break in the smallest chunks possible. ^_^</div></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
And please try to find a way to try to contribute some patches, docs,<br>
bugfixes ... to the MacPorts base in the near future. Maybe you could<br>
write some unit tests for the base related to the trace mode? Talk to<br>
Marcus or Clemens about some challenges if you fail to find some<br>
yourself.<br>
<br></blockquote><div><br></div><div><br></div><div><div>I made a contribution, not a big one though, day before yesterday I guess .Here is the link:</div><div> <a href="https://github.com/macports/macports-base/pull/117">https://github.com/macports/macports-base/pull/117</a></div><div><br></div><div>And I am working on docs already, I will share them too till the weekend. Hope those docs proof helpful :)</div></div><div><br></div><div>I will give a research on unit test task and see the best I can do right now. ^_^ </div><div><br></div><div>Till now what I saw the code in each file works in conjunction with many others file, so while trying to fix bugs or adding patches right now I may cause more bugs, but I will try to contribute patches as soon as I can. :)</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
> Also, should I share the link to document here as well?<br>
<br>
Yes, that would definitely make sense.<br></blockquote><div><br></div><div>Here is the link to my draft proposal.<br></div><div><br></div><div><a href="https://docs.google.com/document/d/1qH5VMtrQ3tvd5gFPf51lmJtd6dYfUuEmO1AvXmU_4qM/edit#heading=h.tal46x1pbsaj">https://docs.google.com/document/d/1qH5VMtrQ3tvd5gFPf51lmJtd6dYfUuEmO1AvXmU_4qM/edit#heading=h.tal46x1pbsaj</a><br></div><div><br></div><div>Please provide with any more feedbacks where ever possible. ^_^</div><div><br></div><div>Regards,</div><div><br></div><div>Mihir</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<br>
Mojca<br>
</blockquote></div>
</div></div></div></div></div>