<div dir="ltr">Hey,<br><br>I will do the suggested changes and also add the pending stuff and then submit the draft proposal by tomorrow.<br>Thanks for the review.<br><br>Thanks,<br>Karan Sheth<br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Apr 1, 2019 at 1:16 AM Cyril Roelandt <<a href="mailto:cyril.roelandt@aquilenet.fr">cyril.roelandt@aquilenet.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
On 2019-04-01 00:30, KARAN SHETH wrote:<br>
> I have started drafting my proposal for GSoC 2019. It would be great to<br>
> have a review on it before I submit the same.<br>
<br>
A few comments.<br>
<br>
First of all, "upt" is written in lower case, except when found at the<br>
beginning of a sentence :)<br>
<br>
To someone who is new to the subject, it is unclear what cpan2port and<br>
pypi2port do. Maybe you could write that they automatically write a<br>
Portile for a piece of software available on CPAN or PyPI, and that it<br>
is really useful for packagers.<br>
<br>
Then it is not really clear why upt would be better suited for this<br>
task. You could use a few lines to explain that, thanks to its modular<br>
design, it is easier to maintain, and will allow MacPorts to get<br>
something to work with minimal effort.<br>
<br>
I would probably have a "upt update" subcommand rather than a new option<br>
to "upt package", but it's nitpicking, and definitely not the most<br>
challenging part of this feature.<br>
<br>
About the "Adding MacPorts Backend to UPT" section: calling it very<br>
basic may not sound really nice to Mojca :) I think you could also<br>
mention that the package needs unit tests, which you will happily write!<br>
<br>
About the "Adding npm frontend" section. I do not really expect you to<br>
add support for NPM to all the backends: we should probably stay focused<br>
on MacPorts. This could be a "bonus" if you find yourself a bit bored at<br>
the end of the summer, though. You could write "default.nix" instead of<br>
"nix" in the "Package definitions" column.<br>
<br>
About the "Input / Ouptut of UPT" section. The "-o/--output" option<br>
already exists in upt. It is not always used by the backends though. The<br>
"-i/--input" option does not exist, and I would rather use the following<br>
syntax:<br>
<br>
        $ upt package -f pypi -b macports pkg1,pkg2,pkg3<br>
<br>
What do you think?<br>
<br>
About the "Support for Package Updates in UPT" section. Some questions<br>
will need to be answered first. Will all the frontends be able to<br>
support this? How hard will it be for other backends to implement this?<br>
<br>
About the schedule. You mention the support of npm/cpan/pypi for<br>
Macports in both the pre-GSoC and the Week 1. Just do it in Week 1:)<br>
<br>
Week 2: you mention the colored output, but it is not part of the<br>
"Technical details" section.<br>
<br>
Week 3.4: why would you add two methods of updating packages? Shouldn't<br>
we spend more time thinking about the right way of doing things, rather<br>
than coding 2 different solutions?<br>
<br>
Week 10,11: if you want to add the Haskell frontend to the "Technical<br>
Details", I can tell you more about why it is going to be a bit more<br>
difficult than other package managers:)<br>
<br>
About the additional questions. Linux is written with a capital "L".<br>
<br>
<br>
I hope this helps!<br>
Cyril.<br>
</blockquote></div></div>

<br>
<div><img src="http://somaiya.edu/media/images/top-layer_footer.jpg" style="font-size:1.3em"></div><div><a href="https://www.somaiya.edu" style="font-size:1.3em" target="_blank"><img src="http://somaiya.edu/media/images/svv-logo-email-footer.jpg"></a><span style="font-size:1.3em">       </span><a href="http://www.somaiya-ayurvihar.org" style="font-size:1.3em" target="_blank"><img src="http://somaiya.edu/media/images/Medical-Logo-email-footer.jpg"></a><span style="font-size:1.3em"> </span><a href="http://nareshwadi.org" style="font-size:1.3em" target="_blank"><img src="http://somaiya.edu/media/images/Nareshwadi-Logo-email-footer.jpg"></a><span style="font-size:1.3em"> </span><a href="http://somaiya.com" style="font-size:1.3em" target="_blank"><img src="http://somaiya.edu/media/images/Somaiya-Trust-Logo-email-footer.jpg"></a><span style="font-size:1.3em"> </span><a href="http://www.helpachild.in" style="font-size:1.3em" target="_blank"><img src="http://somaiya.edu/media/images/HelpAChild-Logo-email-footer.jpg"></a><span style="font-size:1.3em"> </span><a href="http://nareshwadi.org" style="font-size:1.3em" target="_blank"><img src="http://somaiya.edu/media/images/PragatiMandal-Logo-email-footer-1.jpg"></a></div>