{"thread":{"id":"57847","subject":"Resizing panels in the gitk window","startedAt":"2022-05-05T07:06:36Z","lastAt":"2022-05-11T15:25:51Z","messageCount":14,"participants":["Christian Grün","stefan.naewe@atlas-elektronik.com","Matthias Aßhauer","Junio C Hamano","Halil SEN","Paul Mackerras"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"454847","messageId":"CAP94bnMHggYR=FOh8462tncFAxOUJ-db=yj7YPoTJRS-VwjEgw@mail.gmail.com","threadId":"57847","inReplyTo":"CAP94bnOyrx6hCmWSjmejVfopUydk10ga42wJzm1M4nk_OXn9Mg@mail.gmail.com","subject":"Resizing panels in the gitk window","fromName":"Christian Grün","fromEmail":"christian.gruen@gmail.com","sentAt":"2022-05-05T07:06:19Z","receivedAt":"2022-05-05T07:06:36Z","isPatch":false,"sender":{"key":"christian.gruen@gmail.com","avatar":"https://gravatar.com/avatar/33d8988b32c2bb96e322fd8b13a635ef159917abd7a41e1883e6bd59ea967f3f?d=mp&s=160"},"body":"Resizing panels in the gitk window leads to a error message that’s\npresented multiple times in a dialog:\n\nexpected integer but got \"\"\nexpected integer but got \"\"\n while executing\n\"$win sash place 0 $sash0 [lindex $s0 1]\"\n (procedure \"resizeclistpanes\" line 38)\n invoked from within\n\"resizeclistpanes .tf.histframe.pwclist 1834\"\n (command bound to event)\n\nIs this a known bug?\n\nI’ll be happy to give more details.\nBest, Christian\n"},{"id":"454900","messageId":"CAP94bnNbXaZA9pdDspb2ajWk==rSAP0xEAQ4LNfwkzsN7B7Exg@mail.gmail.com","threadId":"57847","inReplyTo":"CAP94bnOyrx6hCmWSjmejVfopUydk10ga42wJzm1M4nk_OXn9Mg@mail.gmail.com","subject":"Re: Resizing panels in the gitk window","fromName":"Christian Grün","fromEmail":"christian.gruen@gmail.com","sentAt":"2022-05-06T07:46:03Z","receivedAt":"2022-05-06T07:46:20Z","isPatch":false,"sender":{"key":"christian.gruen@gmail.com","avatar":"https://gravatar.com/avatar/33d8988b32c2bb96e322fd8b13a635ef159917abd7a41e1883e6bd59ea967f3f?d=mp&s=160"},"body":"Maybe there’s a better place to address gitk? Anyone?\nThanks in advance, Christian\n\n\n\nOn Thu, May 5, 2022 at 9:01 AM Christian Grün wrote:\n>\n> Resizing panels in the gitk window leads to a error message that’s presented multiple times in a dialog:\n>\n> expected integer but got \"\"\n> expected integer but got \"\"\n> while executing\n> \"$win sash place 0 $sash0 [lindex $s0 1]\"\n> (procedure \"resizeclistpanes\" line 38)\n> invoked from within\n> \"resizeclistpanes .tf.histframe.pwclist 1834\"\n> (command bound to event)\n>\n> Is this a known bug?\n>\n> I’ll be happy to give more details.\n> Best, Christian\n>\n"},{"id":"454903","messageId":"ff856dc509c7424cb7137d9cac318714@atlas-elektronik.com","threadId":"57847","inReplyTo":"CAP94bnNbXaZA9pdDspb2ajWk==rSAP0xEAQ4LNfwkzsN7B7Exg@mail.gmail.com","subject":"AW: Resizing panels in the gitk window","fromName":"","fromEmail":"stefan.naewe@atlas-elektronik.com","sentAt":"2022-05-06T08:32:49Z","receivedAt":"2022-05-06T08:34:08Z","isPatch":false,"sender":{"key":"stefan.naewe@gmail.com","avatar":"https://avatars.githubusercontent.com/u/4468?v=4"},"body":"\n\n> -----Ursprüngliche Nachricht-----\n> Von: Christian Grün \n> Gesendet: Freitag, 6. Mai 2022 09:46\n> An: git@vger.kernel.org\n> Betreff: Re: Resizing panels in the gitk window\n> \n> Maybe there’s a better place to address gitk? Anyone?\n> Thanks in advance, Christian\n\nI think it would be a good idea to at least say on which OS you're using gitk\nand which version of git is installed.\n \nRegards,\n Stefan\n \n> \n> On Thu, May 5, 2022 at 9:01 AM Christian Grün \n> wrote:\n> >\n> > Resizing panels in the gitk window leads to a error message that’s presented\n> multiple times in a dialog:\n> >\n> > expected integer but got \"\"\n> > expected integer but got \"\"\n> > while executing\n> > \"$win sash place 0 $sash0 [lindex $s0 1]\"\n> > (procedure \"resizeclistpanes\" line 38)\n> > invoked from within\n> > \"resizeclistpanes .tf.histframe.pwclist 1834\"\n> > (command bound to event)\n> >\n> > Is this a known bug?\n> >\n> > I’ll be happy to give more details.\n> > Best, Christian\n> >\n"},{"id":"454904","messageId":"CAP94bnNw7rjiYQtv5k5vznwOi8tgQxzmJ3Wc_6Fvu9aXWqcPZQ@mail.gmail.com","threadId":"57847","inReplyTo":"ff856dc509c7424cb7137d9cac318714@atlas-elektronik.com","subject":"Re: Resizing panels in the gitk window","fromName":"Christian Grün","fromEmail":"christian.gruen@gmail.com","sentAt":"2022-05-06T09:01:55Z","receivedAt":"2022-05-06T09:02:09Z","isPatch":false,"sender":{"key":"christian.gruen@gmail.com","avatar":"https://gravatar.com/avatar/33d8988b32c2bb96e322fd8b13a635ef159917abd7a41e1883e6bd59ea967f3f?d=mp&s=160"},"body":"Thanks, Stefan. I use Windows 10 Pro (latest version, 10.0.19044 Build\n19044) and git version 2.36.0.windows.1.\n\n\n\nOn Fri, May 6, 2022 at 10:32 AM wrote:\n>\n>\n>\n> > -----Ursprüngliche Nachricht-----\n> > Von: Christian Grün \n> > Gesendet: Freitag, 6. Mai 2022 09:46\n> > An: git@vger.kernel.org\n> > Betreff: Re: Resizing panels in the gitk window\n> >\n> > Maybe there’s a better place to address gitk? Anyone?\n> > Thanks in advance, Christian\n>\n> I think it would be a good idea to at least say on which OS you're using gitk\n> and which version of git is installed.\n>\n> Regards,\n> Stefan\n>\n> >\n> > On Thu, May 5, 2022 at 9:01 AM Christian Grün \n> > wrote:\n> > >\n> > > Resizing panels in the gitk window leads to a error message that’s presented\n> > multiple times in a dialog:\n> > >\n> > > expected integer but got \"\"\n> > > expected integer but got \"\"\n> > > while executing\n> > > \"$win sash place 0 $sash0 [lindex $s0 1]\"\n> > > (procedure \"resizeclistpanes\" line 38)\n> > > invoked from within\n> > > \"resizeclistpanes .tf.histframe.pwclist 1834\"\n> > > (command bound to event)\n> > >\n> > > Is this a known bug?\n> > >\n> > > I’ll be happy to give more details.\n> > > Best, Christian\n> > >\n"},{"id":"454913","messageId":"AM0PR04MB6019000A6C54CF97447E7B6EA5C59@AM0PR04MB6019.eurprd04.prod.outlook.com","threadId":"57847","inReplyTo":"CAP94bnMHggYR=FOh8462tncFAxOUJ-db=yj7YPoTJRS-VwjEgw@mail.gmail.com","subject":"Re: Resizing panels in the gitk window","fromName":"Matthias Aßhauer","fromEmail":"mha1993@live.de","sentAt":"2022-05-06T12:00:08Z","receivedAt":"2022-05-06T12:00:34Z","isPatch":false,"sender":{"key":"mha1993@live.de","avatar":"https://avatars.githubusercontent.com/u/6178234?v=4"},"body":"\nOn Thu, 5 May 2022, Christian Grün wrote:\n\n> Resizing panels in the gitk window leads to a error message that’s\n> presented multiple times in a dialog:\n>\n> expected integer but got \"\"\n> expected integer but got \"\"\n> while executing\n> \"$win sash place 0 $sash0 [lindex $s0 1]\"\n> (procedure \"resizeclistpanes\" line 38)\n> invoked from within\n> \"resizeclistpanes .tf.histframe.pwclist 1834\"\n> (command bound to event)\n>\n> Is this a known bug?\n\nIt looks like the issue at [1]. It sadly seems like nobody ever reviewed \nor applied thos suggested patches, but there is some more off-list \ndiscussion at [2].\n\n> I’ll be happy to give more details.\n> Best, Christian\n>\n\nBest regards\n\nMatthias\n\n[1] https://lore.kernel.org/git/pull.1219.git.git.1645386457.gitgitgadget@gmail.com/#t\n[2] https://github.com/git/git/pull/1219#issuecomment-1113122632\n"},{"id":"454914","messageId":"CAP94bnO9myHnSv7V15XdJDZY6WkvA85r5m_1=kyFi5=GCYMmeQ@mail.gmail.com","threadId":"57847","inReplyTo":"AM0PR04MB6019000A6C54CF97447E7B6EA5C59@AM0PR04MB6019.eurprd04.prod.outlook.com","subject":"Re: Resizing panels in the gitk window","fromName":"Christian Grün","fromEmail":"christian.gruen@gmail.com","sentAt":"2022-05-06T12:12:22Z","receivedAt":"2022-05-06T12:12:36Z","isPatch":false,"sender":{"key":"christian.gruen@gmail.com","avatar":"https://gravatar.com/avatar/33d8988b32c2bb96e322fd8b13a635ef159917abd7a41e1883e6bd59ea967f3f?d=mp&s=160"},"body":"Thanks, Matthias, for the clarification. I’ll follow the discussion on GitHub.\n\n\nOn Fri, May 6, 2022 at 2:00 PM Matthias Aßhauer wrote:\n>\n>\n> On Thu, 5 May 2022, Christian Grün wrote:\n>\n> > Resizing panels in the gitk window leads to a error message that’s\n> > presented multiple times in a dialog:\n> >\n> > expected integer but got \"\"\n> > expected integer but got \"\"\n> > while executing\n> > \"$win sash place 0 $sash0 [lindex $s0 1]\"\n> > (procedure \"resizeclistpanes\" line 38)\n> > invoked from within\n> > \"resizeclistpanes .tf.histframe.pwclist 1834\"\n> > (command bound to event)\n> >\n> > Is this a known bug?\n>\n> It looks like the issue at [1]. It sadly seems like nobody ever reviewed\n> or applied thos suggested patches, but there is some more off-list\n> discussion at [2].\n>\n> > I’ll be happy to give more details.\n> > Best, Christian\n> >\n>\n> Best regards\n>\n> Matthias\n>\n> [1] https://lore.kernel.org/git/pull.1219.git.git.1645386457.gitgitgadget@gmail.com/#t\n> [2] https://github.com/git/git/pull/1219#issuecomment-1113122632\n"},{"id":"454917","messageId":"xmqqzgjuocjk.fsf@gitster.g","threadId":"57847","inReplyTo":"AM0PR04MB6019000A6C54CF97447E7B6EA5C59@AM0PR04MB6019.eurprd04.prod.outlook.com","subject":"Re: Resizing panels in the gitk window","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-05-06T16:23:27Z","receivedAt":"2022-05-06T16:23:37Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Matthias Aßhauer writes:\n\n> It looks like the issue at [1]. It sadly seems like nobody ever\n> reviewed or applied thos suggested patches, but there is some more\n> off-list discussion at [2].\n>\n> [1] https://lore.kernel.org/git/pull.1219.git.git.1645386457.gitgitgadget@gmail.com/#t\n> [2] https://github.com/git/git/pull/1219#issuecomment-1113122632\n\nThanks for a pointer.\n\nI do recall seeing [1] but I do not see anything happened on the\nthread since then. I didn't look at it back then primarily because\nthe patches were made against a wrong history (no, you do not want\nto make a gitk patch against my tree nor throw a pull request at my\nrepository), and there was no way for me to pick it up.\n\nAs Documentation/SubmittingPatches says:\n\n Some parts of the system have dedicated maintainers with their own\n repositories.\n ... \n Patches to these parts should be based on their trees.\n\nHaving said that, gitk has been quiescent for quite a while, and I\nwonder if Paul's been too busy for (or no longer interested in)\nmaintaining it and perhaps he can use a replacement maintainer?\n\n"},{"id":"454999","messageId":"CAOeUTuufDgsyp7FgJj4T+h=JOMaWYVBkMx8S6fNRQ3cdVo6tOw@mail.gmail.com","threadId":"57847","inReplyTo":"xmqqzgjuocjk.fsf@gitster.g","subject":"Re: Resizing panels in the gitk window","fromName":"Halil SEN","fromEmail":"halilsen@gmail.com","sentAt":"2022-05-09T12:46:45Z","receivedAt":"2022-05-09T12:47:05Z","isPatch":false,"sender":{"key":"halilsen@gmail.com","avatar":null},"body":"Junio C Hamano, wrote:\n> I didn't look at it back then primarily because\n> the patches were made against a wrong history\n\nI remember looking for Paul's repository but I\nfailed to find it so created the patch against\nthe main git repo. Maybe I should have\nmentioned this in my message since I knew\nthe documentation included that note.\n\nI am happy to rebase my patch if someone\ncould point me in the right direction.\n"},{"id":"455026","messageId":"xmqqtu9yd42d.fsf@gitster.g","threadId":"57847","inReplyTo":"CAOeUTuufDgsyp7FgJj4T+h=JOMaWYVBkMx8S6fNRQ3cdVo6tOw@mail.gmail.com","subject":"Re: Resizing panels in the gitk window","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-05-09T17:11:06Z","receivedAt":"2022-05-09T17:11:16Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Halil SEN writes:\n\n> Junio C Hamano, wrote:\n>> I didn't look at it back then primarily because\n>> the patches were made against a wrong history\n>\n> I remember looking for Paul's repository but I\n> failed to find it so created the patch against\n> the main git repo. Maybe I should have\n> mentioned this in my message since I knew\n> the documentation included that note.\n>\n> I am happy to rebase my patch if someone\n> could point me in the right direction.\n\nThanks for giving me a chance to double-check what we have in\nDocumentation/SubmittingPatches is not stale:\n\n- `gitk-git/` comes from Paul Mackerras's gitk project:\n\n\tgit://ozlabs.org/~paulus/gitk\n\nI just made a fresh clone of it just to make sure. The latest\ncommit in the repository is 6cd80496 (gitk: Resize panes correctly\nwhen reducing window size, 2020-10-03). It seems to be related to\nthe topic you are working on?\n\nAgain, I wonder if Paul's been too busy for (or no longer interested\nin) maintaining it and perhaps he can use a replacement maintainer?\n\nThanks.\n"},{"id":"455154","messageId":"YntxL/fTplFm8lr6@cleo","threadId":"57847","inReplyTo":"xmqqzgjuocjk.fsf@gitster.g","subject":"Re: Resizing panels in the gitk window","fromName":"Paul Mackerras","fromEmail":"paulus@ozlabs.org","sentAt":"2022-05-11T08:17:51Z","receivedAt":"2022-05-11T08:21:55Z","isPatch":false,"sender":{"key":"paulus@ozlabs.org","avatar":"https://avatars.githubusercontent.com/u/1606439?v=4"},"body":"On Fri, May 06, 2022 at 09:23:27AM -0700, Junio C Hamano wrote:\n> Matthias Aßhauer writes:\n> \n> > It looks like the issue at [1]. It sadly seems like nobody ever\n> > reviewed or applied thos suggested patches, but there is some more\n> > off-list discussion at [2].\n> >\n> > [1] https://lore.kernel.org/git/pull.1219.git.git.1645386457.gitgitgadget@gmail.com/#t\n> > [2] https://github.com/git/git/pull/1219#issuecomment-1113122632\n> \n> Thanks for a pointer.\n> \n> I do recall seeing [1] but I do not see anything happened on the\n> thread since then. I didn't look at it back then primarily because\n> the patches were made against a wrong history (no, you do not want\n> to make a gitk patch against my tree nor throw a pull request at my\n> repository), and there was no way for me to pick it up.\n\nI have just picked up Halil Sen's two patches and applied them, and\npushed the result out to git.ozlabs.org.\n\n> As Documentation/SubmittingPatches says:\n> \n> Some parts of the system have dedicated maintainers with their own\n> repositories.\n> ... \n> Patches to these parts should be based on their trees.\n> \n> Having said that, gitk has been quiescent for quite a while, and I\n> wonder if Paul's been too busy for (or no longer interested in)\n> maintaining it and perhaps he can use a replacement maintainer?\n\nI am indeed very busy and would welcome a replacement maintainer for\ngitk, if somebody was willing to take it up.\n\nThanks,\nPaul.\n\n"},{"id":"455155","messageId":"Yntx33Q9pNLJ+QqS@cleo","threadId":"57847","inReplyTo":"xmqqtu9yd42d.fsf@gitster.g","subject":"Re: Resizing panels in the gitk window","fromName":"Paul Mackerras","fromEmail":"paulus@ozlabs.org","sentAt":"2022-05-11T08:20:47Z","receivedAt":"2022-05-11T08:21:56Z","isPatch":false,"sender":{"key":"paulus@ozlabs.org","avatar":"https://avatars.githubusercontent.com/u/1606439?v=4"},"body":"On Mon, May 09, 2022 at 10:11:06AM -0700, Junio C Hamano wrote:\n> Halil SEN writes:\n> \n> > Junio C Hamano, wrote:\n> >> I didn't look at it back then primarily because\n> >> the patches were made against a wrong history\n> >\n> > I remember looking for Paul's repository but I\n> > failed to find it so created the patch against\n> > the main git repo. Maybe I should have\n> > mentioned this in my message since I knew\n> > the documentation included that note.\n> >\n> > I am happy to rebase my patch if someone\n> > could point me in the right direction.\n> \n> Thanks for giving me a chance to double-check what we have in\n> Documentation/SubmittingPatches is not stale:\n> \n> - `gitk-git/` comes from Paul Mackerras's gitk project:\n> \n> \tgit://ozlabs.org/~paulus/gitk\n\nCould you change that to\n\n\tgit://git.ozlabs.org/~paulus/gitk\n\nplease?\n\nThe ozlabs.org infrastructure changes every so often, and while the\nplain ozlabs.org works now and is not expected to stop working,\ngit.ozlabs.org would be a little more future-proof.\n\nThanks,\nPaul.\n"},{"id":"455156","messageId":"YntyF+dHp2WEpI8v@cleo","threadId":"57847","inReplyTo":"CAOeUTusOW1=cDKNhrBCzdzfbVLmY_qb7yg114ukRfy904S34uA@mail.gmail.com","subject":"Re: Resizing panels in the gitk window","fromName":"Paul Mackerras","fromEmail":"paulus@ozlabs.org","sentAt":"2022-05-11T08:21:43Z","receivedAt":"2022-05-11T08:22:04Z","isPatch":false,"sender":{"key":"paulus@ozlabs.org","avatar":"https://avatars.githubusercontent.com/u/1606439?v=4"},"body":"On Mon, May 09, 2022 at 10:19:21PM +0200, Halil SEN wrote:\n> Junio C Hamano writes:\n> \n> > It seems to be related to the topic you are working on?\n> \n> It indeed is related. Sorry for troubling you with the repo, I don't\n> remember exactly how I failed before, thanks for sharing it.\n> \n> > Again, I wonder if Paul's been too busy for (or no longer interested\n> > in) maintaining it and perhaps he can use a replacement maintainer?\n> \n> I guess I should rebase my patch and resubmit it nevertheless.\n> I'll do it within the next few days. Now that I thought about it,\n> I think I had trouble creating and e-mailing the patch to the mail\n> list and the gitgitgadget was available only for the main repo.\n> I shall ask for help if I fail to do so again.\n\nNo need to rebase, I have manually applied your patches.\n\nJunio, could you do a pull from my repository to get them?\n\nThanks,\nPaul.\n"},{"id":"455162","messageId":"xmqqpmkkm7if.fsf@gitster.g","threadId":"57847","inReplyTo":"Yntx33Q9pNLJ+QqS@cleo","subject":"Re: Resizing panels in the gitk window","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-05-11T15:08:40Z","receivedAt":"2022-05-11T15:08:49Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Paul Mackerras writes:\n\n> The ozlabs.org infrastructure changes every so often, and while the\n> plain ozlabs.org works now and is not expected to stop working,\n> git.ozlabs.org would be a little more future-proof.\n\nWill do. Thanks.\n"},{"id":"455163","messageId":"xmqqlev8m6q4.fsf@gitster.g","threadId":"57847","inReplyTo":"YntyF+dHp2WEpI8v@cleo","subject":"Re: Resizing panels in the gitk window","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-05-11T15:25:39Z","receivedAt":"2022-05-11T15:25:51Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Paul Mackerras writes:\n\n> Junio, could you do a pull from my repository to get them?\n\nThanks, done.\n"}]}