{"thread":{"id":"36978","subject":"gitk with submodules does not show new commits on other branches","startedAt":"2014-06-22T14:09:22Z","lastAt":"2014-06-24T10:07:03Z","messageCount":11,"participants":["Stephen Kelly","Jens Lehmann"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"244834","messageId":"lo6o2h$vva$1@ger.gmane.org","threadId":"36978","inReplyTo":null,"subject":"gitk with submodules does not show new commits on other branches","fromName":"Stephen Kelly","fromEmail":"steveire@gmail.com","sentAt":"2014-06-22T14:09:22Z","receivedAt":"2014-06-22T14:09:22Z","isPatch":false,"sender":{"key":"steveire@gmail.com","avatar":null},"body":"\nHello,\n\nboost.git, is using submodules. \n\nIf I run gitk after a pull, there are some messages along the lines of \n\n\n    Update preprocessor from develop.\n\n  Submodule libs/preprocessor 9d2d1ff..1422fce:\n    Merge branch 'master' into develop\n\n\nThat is, it shows only the merge. \n\nIf I then run \n\n git log --oneline 9d2d1ff..1422fce \n\nI can see the commits which were made on master and then merged into \ndevelop. Please show the same information (ie all commits newly reachable \nfrom develop) in the submodule gitk output.\n\nThanks,\n\nSteve.\n"},{"id":"244836","messageId":"53A6F978.4010005@web.de","threadId":"36978","inReplyTo":"lo6o2h$vva$1@ger.gmane.org","subject":"Re: gitk with submodules does not show new commits on other branches","fromName":"Jens Lehmann","fromEmail":"jens.lehmann@web.de","sentAt":"2014-06-22T15:42:48Z","receivedAt":"2014-06-22T15:42:48Z","isPatch":false,"sender":{"key":"jens.lehmann@web.de","avatar":"https://avatars.githubusercontent.com/u/135220?v=4"},"body":"Am 22.06.2014 16:09, schrieb Stephen Kelly:\n> \n> Hello,\n> \n> boost.git, is using submodules. \n> \n> If I run gitk after a pull, there are some messages along the lines of \n> \n> \n>     Update preprocessor from develop.\n> \n>   Submodule libs/preprocessor 9d2d1ff..1422fce:\n>     Merge branch 'master' into develop\n> \n> \n> That is, it shows only the merge. \n> \n> If I then run \n> \n>  git log --oneline 9d2d1ff..1422fce \n> \n> I can see the commits which were made on master and then merged into \n> develop. Please show the same information (ie all commits newly reachable \n> from develop) in the submodule gitk output.\n\nThis should not happen by default. If you have a feature branch based\nworkflow, the merge is just what you want to see. And \"git submodule\nsummary\" is giving you the exact same information.\n\nBut I agree that this is suboptimal for your workflow. What about adding\na \"Visualize These Changes In The Submodule\" menu entry for the context\nmenu of a change in gitk just like the one git gui already has? Then the\nuser could examine the merges in more detail if he wants.\n"},{"id":"244837","messageId":"lo6tnh$p3c$1@ger.gmane.org","threadId":"36978","inReplyTo":"53A6F978.4010005@web.de","subject":"Re: gitk with submodules does not show new commits on other branches","fromName":"Stephen Kelly","fromEmail":"steveire@gmail.com","sentAt":"2014-06-22T15:45:54Z","receivedAt":"2014-06-22T15:45:54Z","isPatch":false,"sender":{"key":"steveire@gmail.com","avatar":null},"body":"Jens Lehmann wrote:\n\n> Am 22.06.2014 16:09, schrieb Stephen Kelly:\n\n>> Please show the same information (ie all commits newly reachable\n>> from develop) in the submodule gitk output.\n> \n> This should not happen by default. If you have a feature branch based\n> workflow, the merge is just what you want to see.\n\nIn my case, it is not what I want to see :).\n\n> But I agree that this is suboptimal for your workflow. What about adding\n> a \"Visualize These Changes In The Submodule\" menu entry for the context\n> menu of a change in gitk just like the one git gui already has?\n\nCan you tell me how to find and try that out in git gui?\n\nThanks,\n\nSteve.\n"},{"id":"244871","messageId":"lo99sj$e2$1@ger.gmane.org","threadId":"36978","inReplyTo":"lo6tnh$p3c$1@ger.gmane.org","subject":"Re: gitk with submodules does not show new commits on other branches","fromName":"Stephen Kelly","fromEmail":"steveire@gmail.com","sentAt":"2014-06-23T13:25:39Z","receivedAt":"2014-06-23T13:25:39Z","isPatch":false,"sender":{"key":"steveire@gmail.com","avatar":null},"body":"Stephen Kelly wrote:\n\n>> But I agree that this is suboptimal for your workflow. What about adding\n>> a \"Visualize These Changes In The Submodule\" menu entry for the context\n>> menu of a change in gitk just like the one git gui already has?\n> \n> Can you tell me how to find and try that out in git gui?\n\nIn particular, I don't see it in my git gui. Is it only in master?\n\nMy versions:\n\n git-gui version 0.18.0.14.g1b2c7\n git version 1.9.1\n\n Tcl/Tk version 8.6.1\n Aspell 0.60.7-20110707, en_US\n\nThanks,\n\nSteve.\n"},{"id":"244875","messageId":"53A85659.2040905@web.de","threadId":"36978","inReplyTo":"lo6tnh$p3c$1@ger.gmane.org","subject":"Re: gitk with submodules does not show new commits on other branches","fromName":"Jens Lehmann","fromEmail":"jens.lehmann@web.de","sentAt":"2014-06-23T16:31:21Z","receivedAt":"2014-06-23T16:31:21Z","isPatch":false,"sender":{"key":"jens.lehmann@web.de","avatar":"https://avatars.githubusercontent.com/u/135220?v=4"},"body":"Am 22.06.2014 17:45, schrieb Stephen Kelly:\n> Jens Lehmann wrote:\n> \n>> Am 22.06.2014 16:09, schrieb Stephen Kelly:\n>> But I agree that this is suboptimal for your workflow. What about adding\n>> a \"Visualize These Changes In The Submodule\" menu entry for the context\n>> menu of a change in gitk just like the one git gui already has?\n> \n> Can you tell me how to find and try that out in git gui?\n\nSure, you'll find that in the pop-up menu when right clicking the\nsubmodule summary in the diff pane (where you can choose to stage\nor unstage hunks or lines for regular files).\n"},{"id":"244877","messageId":"lo9m3f$4as$1@ger.gmane.org","threadId":"36978","inReplyTo":"53A85659.2040905@web.de","subject":"Re: gitk with submodules does not show new commits on other branches","fromName":"Stephen Kelly","fromEmail":"steveire@gmail.com","sentAt":"2014-06-23T16:54:08Z","receivedAt":"2014-06-23T16:54:08Z","isPatch":false,"sender":{"key":"steveire@gmail.com","avatar":null},"body":"Jens Lehmann wrote:\n\n> Am 22.06.2014 17:45, schrieb Stephen Kelly:\n>> Jens Lehmann wrote:\n>> \n>>> Am 22.06.2014 16:09, schrieb Stephen Kelly:\n>>> But I agree that this is suboptimal for your workflow. What about adding\n>>> a \"Visualize These Changes In The Submodule\" menu entry for the context\n>>> menu of a change in gitk just like the one git gui already has?\n>> \n>> Can you tell me how to find and try that out in git gui?\n> \n> Sure, you'll find that in the pop-up menu when right clicking the\n> submodule summary in the diff pane (where you can choose to stage\n> or unstage hunks or lines for regular files).\n\nThanks. I see it the feature now.\n"},{"id":"244878","messageId":"lo9mg1$bma$1@ger.gmane.org","threadId":"36978","inReplyTo":"53A6F978.4010005@web.de","subject":"Re: gitk with submodules does not show new commits on other branches","fromName":"Stephen Kelly","fromEmail":"steveire@gmail.com","sentAt":"2014-06-23T17:00:49Z","receivedAt":"2014-06-23T17:00:49Z","isPatch":false,"sender":{"key":"steveire@gmail.com","avatar":null},"body":"Jens Lehmann wrote:\n\n> But I agree that this is suboptimal for your workflow. What about adding\n> a \"Visualize These Changes In The Submodule\" menu entry for the context\n> menu of a change in gitk just like the one git gui already has? Then the\n> user could examine the merges in more detail if he wants.\n\nSuch a menu entry might be useful,  but it is still different workflow. It \nwould be useful in the 'that commit sounds interesting - show me' case. I \nwould need to see the commit title to know if it sounds interesting though. \n\nYour suggestion involves a different window and context, but all I want to \nsee is a list of commit titles so I can press 'up' to see the next ones, not \nhave to close a window first before I can see the next list, and then have \nto right-click to see the list of commits.\n\nIs it so difficult to list the titles of all of the newly-reachable commits? \nEven with a config option? Even an undocumented option? \n\nFailing all of that, can you show me where the code would need to be changed \nto list all of the newly-reachable commits? I can keep a commit for myself \nthen.\n\nThanks,\n\nSteve.\n"},{"id":"244884","messageId":"lo9ogu$4in$1@ger.gmane.org","threadId":"36978","inReplyTo":"lo9mg1$bma$1@ger.gmane.org","subject":"Re: gitk with submodules does not show new commits on other branches","fromName":"Stephen Kelly","fromEmail":"steveire@gmail.com","sentAt":"2014-06-23T17:35:26Z","receivedAt":"2014-06-23T17:35:26Z","isPatch":false,"sender":{"key":"steveire@gmail.com","avatar":null},"body":"Stephen Kelly wrote:\n\n> Failing all of that, can you show me where the code would need to be\n> changed to list all of the newly-reachable commits? I can keep a commit\n> for myself then.\n\nI see that gitk is showing the output of git diff --submodule, similar to \ngit submodule summary.\n\nAssuming that is not going to be changed, maybe I can hack parseblobdiffline \nlocally. I have not really tried to read of write tcl code before though, so \nI'd still prefer a 'proper' solution somehow.\n\nThanks,\n\nSteve.\n"},{"id":"244888","messageId":"lo9rcn$8tu$1@ger.gmane.org","threadId":"36978","inReplyTo":"lo9ogu$4in$1@ger.gmane.org","subject":"Re: gitk with submodules does not show new commits on other branches","fromName":"Stephen Kelly","fromEmail":"steveire@gmail.com","sentAt":"2014-06-23T18:24:24Z","receivedAt":"2014-06-23T18:24:24Z","isPatch":false,"sender":{"key":"steveire@gmail.com","avatar":null},"body":"Stephen Kelly wrote:\n\n> I see that gitk is showing the output of git diff --submodule, similar to\n> git submodule summary.\n> \n> Assuming that is not going to be changed, maybe I can hack\n> parseblobdiffline locally. I have not really tried to read of write tcl\n> code before though, so I'd still prefer a 'proper' solution somehow.\n\nI dug deeper and came up with this patch which suits my needs:\n\ndiff --git a/submodule.c b/submodule.c\nindex b80ecac..0dacd61 100644\n--- a/submodule.c\n+++ b/submodule.c\n@@ -297,7 +297,7 @@ static int prepare_submodule_summary(struct rev_info \n*rev, const char *path,\n        init_revisions(rev, NULL);\n        setup_revisions(0, NULL, rev, NULL);\n        rev->left_right = 1;\n-       rev->first_parent_only = 1;\n+  rev->max_parents = 1;\n        left->object.flags |= SYMMETRIC_LEFT;\n        add_pending_object(rev, &left->object, path);\n        add_pending_object(rev, &right->object, path);\n\n\nThe 'merge pull request' merges are only noise to me.\n\nThanks,\n\nSteve.\n"},{"id":"244892","messageId":"53A88067.5010908@web.de","threadId":"36978","inReplyTo":"lo9rcn$8tu$1@ger.gmane.org","subject":"Re: gitk with submodules does not show new commits on other branches","fromName":"Jens Lehmann","fromEmail":"jens.lehmann@web.de","sentAt":"2014-06-23T19:30:47Z","receivedAt":"2014-06-23T19:30:47Z","isPatch":false,"sender":{"key":"jens.lehmann@web.de","avatar":"https://avatars.githubusercontent.com/u/135220?v=4"},"body":"Am 23.06.2014 20:24, schrieb Stephen Kelly:\n> Stephen Kelly wrote:\n> \n>> I see that gitk is showing the output of git diff --submodule, similar to\n>> git submodule summary.\n\nRight, and for your use case --submodule would have to learn a\ndifferent value in addition to 'log' and 'short'. And the default\nis already configurable via the 'diff.submodule' config option.\n\n>> Assuming that is not going to be changed, maybe I can hack\n>> parseblobdiffline locally. I have not really tried to read of write tcl\n>> code before though, so I'd still prefer a 'proper' solution somehow.\n\nI'd prefer a proper solution too, which is exactly why I tried\nto understand your use case first before speculating about how\nit could be solved.\n\n> I dug deeper and came up with this patch which suits my needs:\n> \n> diff --git a/submodule.c b/submodule.c\n> index b80ecac..0dacd61 100644\n> --- a/submodule.c\n> +++ b/submodule.c\n> @@ -297,7 +297,7 @@ static int prepare_submodule_summary(struct rev_info \n> *rev, const char *path,\n>         init_revisions(rev, NULL);\n>         setup_revisions(0, NULL, rev, NULL);\n>         rev->left_right = 1;\n> -       rev->first_parent_only = 1;\n> +  rev->max_parents = 1;\n>         left->object.flags |= SYMMETRIC_LEFT;\n>         add_pending_object(rev, &left->object, path);\n>         add_pending_object(rev, &right->object, path);\n> \n> \n> The 'merge pull request' merges are only noise to me.\n\nYup, I think that's the core of the change necessary to help\nyour workflow.\n"},{"id":"244919","messageId":"lobik7$a7b$1@ger.gmane.org","threadId":"36978","inReplyTo":"53A88067.5010908@web.de","subject":"Re: gitk with submodules does not show new commits on other branches","fromName":"Stephen Kelly","fromEmail":"steveire@gmail.com","sentAt":"2014-06-24T10:07:03Z","receivedAt":"2014-06-24T10:07:03Z","isPatch":false,"sender":{"key":"steveire@gmail.com","avatar":null},"body":"Jens Lehmann wrote:\n\n> Am 23.06.2014 20:24, schrieb Stephen Kelly:\n>> Stephen Kelly wrote:\n>> \n>>> I see that gitk is showing the output of git diff --submodule, similar\n>>> to git submodule summary.\n> \n> Right, and for your use case --submodule would have to learn a\n> different value in addition to 'log' and 'short'. And the default\n> is already configurable via the 'diff.submodule' config option.\n\nRight.\n\n>>> Assuming that is not going to be changed, maybe I can hack\n>>> parseblobdiffline locally. I have not really tried to read of write tcl\n>>> code before though, so I'd still prefer a 'proper' solution somehow.\n> \n> I'd prefer a proper solution too, which is exactly why I tried\n> to understand your use case first before speculating about how\n> it could be solved.\n\nThanks for your help!\n\n>> I dug deeper and came up with this patch which suits my needs:\n> \n> Yup, I think that's the core of the change necessary to help\n> your workflow.\n\nOk, so I guess we can add --summary=commits ?\n\nI don't write C code generally, so I'm not certain how that needs to be \npropagated.\n\nThanks,\n\nSteve.\n"}]}