{"thread":{"id":"53021","subject":"RFC][Feature] submodule: teach subcommand 'summary' to give summary for nested submodules","startedAt":"2020-03-18T16:32:46Z","lastAt":"2020-03-24T07:29:22Z","messageCount":6,"participants":["Shourya Shukla","Junio C Hamano","Christian Couder"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"393402","messageId":"20200318163234.21628-1-shouryashukla.oo@gmail.com","threadId":"53021","inReplyTo":null,"subject":"RFC][Feature] submodule: teach subcommand 'summary' to give summary for nested submodules","fromName":"Shourya Shukla","fromEmail":"shouryashukla.oo@gmail.com","sentAt":"2020-03-18T16:32:34Z","receivedAt":"2020-03-18T16:32:46Z","isPatch":false,"sender":{"key":"shouryashukla.oo@gmail.com","avatar":"https://avatars.githubusercontent.com/u/43680618?v=4"},"body":"Hello everyone,\n\nI was converting the subcommand 'summary'(of command submodule) from shell to C and I realised that\nthe subcommand does not have a '--recursive' option in it, meaning it cannot give out summaries of nested\nsubmodules. My opinion on this is that an option should maybe exist for the same.\n\nWhat are the community's thoughts on this? Should the aforementioned option be added? Is there any feasibility\nof this option in this case?\n\nRegards,\nShourya Shukla\n"},{"id":"393411","messageId":"xmqqh7ylfnqi.fsf@gitster.c.googlers.com","threadId":"53021","inReplyTo":"20200318163234.21628-1-shouryashukla.oo@gmail.com","subject":"Re: [RFC][Feature] submodule: teach subcommand 'summary' to give summary for nested submodules","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-03-18T19:48:37Z","receivedAt":"2020-03-18T19:48:41Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Shourya Shukla writes:\n\n> I was converting the subcommand 'summary'(of command submodule)\n> from shell to C and I realised that the subcommand does not have a\n> '--recursive' option in it, meaning it cannot give out summaries\n> of nested submodules. My opinion on this is that an option should\n> maybe exist for the same.\n>\n> What are the community's thoughts on this? Should the\n> aforementioned option be added? Is there any feasibility of this\n> option in this case?\n\nIf \"git submodule summary --recursive\" errors out with today's code,\nno sane user would be using it for any useful purpose, so I would\nthink it is OK to add such a feature.\n\nIf it makes a usable UI for large projects that you can only choose\nbetween \"include no subsubmodules\" (i.e. with --no-recursive) or\n\"include all sub(sub)*modules\" (i.e. with --recursive) is another\nmatter, though.\n"},{"id":"393478","messageId":"20200319171626.5723-1-shouryashukla.oo@gmail.com","threadId":"53021","inReplyTo":"xmqqh7ylfnqi.fsf@gitster.c.googlers.com","subject":"Re: Re: RFC][Feature] submodule","fromName":"Shourya Shukla","fromEmail":"shouryashukla.oo@gmail.com","sentAt":"2020-03-19T17:16:26Z","receivedAt":"2020-03-19T17:16:37Z","isPatch":false,"sender":{"key":"shouryashukla.oo@gmail.com","avatar":"https://avatars.githubusercontent.com/u/43680618?v=4"},"body":"Hello Junio,\n\n> If \"git submodule summary --recursive\" errors out with today's code,\n> no sane user would be using it for any useful purpose, so I would\n> think it is OK to add such a feature.\n\nSure thing! I will try to plan some makeshift implementation.\n\n> If it makes a usable UI for large projects that you can only choose\n> between \"include no subsubmodules\" (i.e. with --no-recursive) or\n> \"include all sub(sub)*modules\" (i.e. with --recursive) is another\n> matter, though.\n\nI think discussing this idea may be a good approach right now? What\nyou are saying for large projects does provoke thought though..\nAlso, what do you think may become a problem for large projects in\nhere?\n\nThank you so much for the feedback BTW :)\n\nRegards,\nShourya Shukla\n"},{"id":"393487","messageId":"xmqqh7ykdo7e.fsf@gitster.c.googlers.com","threadId":"53021","inReplyTo":"20200319171626.5723-1-shouryashukla.oo@gmail.com","subject":"Re: RFC][Feature] submodule","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-03-19T21:33:41Z","receivedAt":"2020-03-19T21:33:51Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Shourya Shukla writes:\n\n> Also, what do you think may become a problem for large projects in\n> here?\n\nPerhaps\n\n \"We want submodule A and B's sub(sub)*modules all the way down\n to the leaf submodule, but directory C houses many submodules,\n among which only one can be active, and we want C/X and its\n sub(sub)*modules all the way down to the leaf but not C/Y nor\n C/Z or any other C/.\"\n\nand similar requests along the line.\n"},{"id":"393539","messageId":"20200320160813.9242-1-shouryashukla.oo@gmail.com","threadId":"53021","inReplyTo":"xmqqh7ykdo7e.fsf@gitster.c.googlers.com","subject":"Re: Re: [RFC][Feature]","fromName":"Shourya Shukla","fromEmail":"shouryashukla.oo@gmail.com","sentAt":"2020-03-20T16:08:13Z","receivedAt":"2020-03-20T16:08:23Z","isPatch":false,"sender":{"key":"shouryashukla.oo@gmail.com","avatar":"https://avatars.githubusercontent.com/u/43680618?v=4"},"body":"> \"We want submodule A and B's sub(sub)*modules all the way down\n> to the leaf submodule, but directory C houses many submodules,\n> among which only one can be active, and we want C/X and its\n> sub(sub)*modules all the way down to the leaf but not C/Y nor\n> C/Z or any other C/.\"\n\nThat's a tough one. Would you advise developing the feature which may\nwork atleast for the \"normal\" cases? In case it does not work for the\naforementioned case and the like, we classify it as a BUG for further\ncorrection?\n\nRegards,\nShourya Shukla\n\n"},{"id":"393864","messageId":"CAP8UFD2nfqRX6+xjRk=F1g+t9T=FgxQVsaNt+b+jW8XoYzuTDQ@mail.gmail.com","threadId":"53021","inReplyTo":"20200320160813.9242-1-shouryashukla.oo@gmail.com","subject":"Re: Re: [RFC][Feature]","fromName":"Christian Couder","fromEmail":"christian.couder@gmail.com","sentAt":"2020-03-24T07:29:06Z","receivedAt":"2020-03-24T07:29:22Z","isPatch":false,"sender":{"key":"christian.couder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/208954?v=4"},"body":"On Fri, Mar 20, 2020 at 5:08 PM Shourya Shukla\n wrote:\n>\n> > \"We want submodule A and B's sub(sub)*modules all the way down\n> > to the leaf submodule, but directory C houses many submodules,\n> > among which only one can be active, and we want C/X and its\n> > sub(sub)*modules all the way down to the leaf but not C/Y nor\n> > C/Z or any other C/.\"\n>\n> That's a tough one. Would you advise developing the feature which may\n> work atleast for the \"normal\" cases? In case it does not work for the\n> aforementioned case and the like, we classify it as a BUG for further\n> correction?\n\nHow are the other git-submodule sub-commands working with --recursive?\nShouldn't --recursive work in the same way for all the sub-commands?\n"}]}