{"thread":{"id":"46121","subject":"[GSoC] Update: Week 3","startedAt":"2017-06-05T20:56:35Z","lastAt":"2017-06-05T22:25:09Z","messageCount":2,"participants":["Prathamesh Chavan","Stefan Beller"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"321563","messageId":"CAME+mvXACX7DejOKDJ=-qV8Ex9ZdYT4vXYV+7AeesaKYUk1sGw@mail.gmail.com","threadId":"46121","inReplyTo":null,"subject":"[GSoC] Update: Week 3","fromName":"Prathamesh Chavan","fromEmail":"pc44800@gmail.com","sentAt":"2017-06-05T20:56:29Z","receivedAt":"2017-06-05T20:56:35Z","isPatch":false,"sender":{"key":"pc44800@gmail.com","avatar":"https://avatars.githubusercontent.com/u/17272661?v=4"},"body":"SUMMARY OF MY PROJECT:\n\nGit submodule subcommands are currently implemented by using shell script\n'git-submodule.sh'. There are several reasons why we'll prefer not to\nuse the shell script. My project intends to convert the subcommands into\nC code, thus making them builtins. This will increase Git's portability\nand hence the efficiency of working with the git-submodule commands.\nLink to the complete proposal: [1]\n\nMentors:\nStefan Beller \nChristian Couder \n\nUPDATES:\n\nAs planned for the third week, most of the time was spent on improving the\nconditions of the present patches, which deal with porting the following\nsubmodule subcommands:\n1. foreach: After a discussion over the issue of the path variable in\n windows, in this week my mentor, Stefan Beller came up with the\n appropriate solution for the problem after discussing it with Ramsay\n Jones. It is being posted on the mailing list for further discussion on\n including it.[2]\n Also, some changes are suggested to the posted version of ported foreach\n function which needs to be taken care of.[3]\n\n2. status: The subcommand's porting is completed and has been discussed\n with the mentors for the last two week after which, a new version of the\n ported function is posted on the mailing list.[4][5]\n\n3. sync: The subcommand's porting is completed and I am currently discussing\n it with my mentors for improving the ported function.\n\n4. summary: Porting of this subcommand is underway and will be trying to\n finish it in the following week.\n\nPLAN FOR WEEK-4 (6 June 2017 to 12 June 2017):\n\n1. sync: since this ported function is currently under discussion with\n the mentors, firstly I'll be looking forward to improvising it as\n suggested and do the needful changes.\n\n2. ported function on the mailing list: currently the ported functions\n foreach and status are on the mailing list, under discussion.\n I will be updating the patches on the list and improvising\n them as required for eventually getting these merged.\n\n3. summary and deinit: I will resume porting submodule subcommands\n from shell to C, firstly git-submodule summary and then git-submodule\n deinit for this week.\n\n[1]: https://docs.google.com/document/d/1krxVLooWl--75Pot3dazhfygR3wCUUWZWzTXtK1L-xU/\n[2]: https://public-inbox.org/git/20170603003710.5558-1-sbeller@google.com/\n[3]: https://public-inbox.org/git/20170602112428.11131-2-pc44800@gmail.com/\n[4]: https://public-inbox.org/git/20170605202529.22959-1-pc44800@gmail.com/\n[5]: https://public-inbox.org/git/20170605202529.22959-2-pc44800@gmail.com/\n\nThanks,\nPrathamesh Chavan\n"},{"id":"321570","messageId":"CAGZ79kasawzm19HtZmAe71FxwPq5GTuCWW+3S2CWYHymVSBTEg@mail.gmail.com","threadId":"46121","inReplyTo":"CAME+mvXACX7DejOKDJ=-qV8Ex9ZdYT4vXYV+7AeesaKYUk1sGw@mail.gmail.com","subject":"Re: [GSoC] Update: Week 3","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2017-06-05T22:25:03Z","receivedAt":"2017-06-05T22:25:09Z","isPatch":false,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"On Mon, Jun 5, 2017 at 1:56 PM, Prathamesh Chavan wrote:\n> 1. foreach: After a discussion over the issue of the path variable in\n> windows, in this week my mentor, Stefan Beller came up with the\n> appropriate solution for the problem after discussing it with Ramsay\n> Jones.\n\nThanks for having so much faith in my abilities, but it may not be\nappropriate, yet. (It does multiple things at once, which is generally\na bad sign already. )\n\nMaybe to be unblocked on the conversion of foreach, you could make\nthe patch have the original behavior, i.e.\n\n\n\nwhich makes sense in the way that it is only converting from shell to C,\nnot fixing a bug along the way. As we discovered a bug, you could just put\na NEEDSWORK comment explaining what the problem is; deferring solving\nthe issue until later.\n\nI'll review the other patches.\n\nThanks,\nStefan\n"}]}