# Resizing panels in the gitk window 14 messages from 2022-05-05 to 2022-05-11. Participants: Christian Grün, stefan.naewe@atlas-elektronik.com, Matthias Aßhauer, Junio C Hamano, Halil SEN, Paul Mackerras. Thread: https://gitlist.dev/t/57847 ## Christian Grün, 2022-05-05 07:06 Subject: Resizing panels in the gitk window Message-ID: URL: https://gitlist.dev/e/CAP94bnMHggYR%3DFOh8462tncFAxOUJ-db%3Dyj7YPoTJRS-VwjEgw%40mail.gmail.com In-Reply-To: ``` Resizing panels in the gitk window leads to a error message that’s presented multiple times in a dialog: expected integer but got "" expected integer but got "" while executing "$win sash place 0 $sash0 [lindex $s0 1]" (procedure "resizeclistpanes" line 38) invoked from within "resizeclistpanes .tf.histframe.pwclist 1834" (command bound to event) Is this a known bug? I’ll be happy to give more details. Best, Christian ``` ## Christian Grün, 2022-05-06 07:46 Subject: Re: Resizing panels in the gitk window Message-ID: URL: https://gitlist.dev/e/CAP94bnNbXaZA9pdDspb2ajWk%3D%3DrSAP0xEAQ4LNfwkzsN7B7Exg%40mail.gmail.com In-Reply-To: ``` Maybe there’s a better place to address gitk? Anyone? Thanks in advance, Christian On Thu, May 5, 2022 at 9:01 AM Christian Grün wrote: > > Resizing panels in the gitk window leads to a error message that’s presented multiple times in a dialog: > > expected integer but got "" > expected integer but got "" > while executing > "$win sash place 0 $sash0 [lindex $s0 1]" > (procedure "resizeclistpanes" line 38) > invoked from within > "resizeclistpanes .tf.histframe.pwclist 1834" > (command bound to event) > > Is this a known bug? > > I’ll be happy to give more details. > Best, Christian > ``` ## stefan.naewe@atlas-elektronik.com, 2022-05-06 08:32 Subject: AW: Resizing panels in the gitk window Message-ID: URL: https://gitlist.dev/e/ff856dc509c7424cb7137d9cac318714%40atlas-elektronik.com In-Reply-To: ``` > -----Ursprüngliche Nachricht----- > Von: Christian Grün > Gesendet: Freitag, 6. Mai 2022 09:46 > An: git@vger.kernel.org > Betreff: Re: Resizing panels in the gitk window > > Maybe there’s a better place to address gitk? Anyone? > Thanks in advance, Christian I think it would be a good idea to at least say on which OS you're using gitk and which version of git is installed. Regards, Stefan > > On Thu, May 5, 2022 at 9:01 AM Christian Grün > wrote: > > > > Resizing panels in the gitk window leads to a error message that’s presented > multiple times in a dialog: > > > > expected integer but got "" > > expected integer but got "" > > while executing > > "$win sash place 0 $sash0 [lindex $s0 1]" > > (procedure "resizeclistpanes" line 38) > > invoked from within > > "resizeclistpanes .tf.histframe.pwclist 1834" > > (command bound to event) > > > > Is this a known bug? > > > > I’ll be happy to give more details. > > Best, Christian > > ``` ## Christian Grün, 2022-05-06 09:01 Subject: Re: Resizing panels in the gitk window Message-ID: URL: https://gitlist.dev/e/CAP94bnNw7rjiYQtv5k5vznwOi8tgQxzmJ3Wc_6Fvu9aXWqcPZQ%40mail.gmail.com In-Reply-To: ``` Thanks, Stefan. I use Windows 10 Pro (latest version, 10.0.19044 Build 19044) and git version 2.36.0.windows.1. On Fri, May 6, 2022 at 10:32 AM wrote: > > > > > -----Ursprüngliche Nachricht----- > > Von: Christian Grün > > Gesendet: Freitag, 6. Mai 2022 09:46 > > An: git@vger.kernel.org > > Betreff: Re: Resizing panels in the gitk window > > > > Maybe there’s a better place to address gitk? Anyone? > > Thanks in advance, Christian > > I think it would be a good idea to at least say on which OS you're using gitk > and which version of git is installed. > > Regards, > Stefan > > > > > On Thu, May 5, 2022 at 9:01 AM Christian Grün > > wrote: > > > > > > Resizing panels in the gitk window leads to a error message that’s presented > > multiple times in a dialog: > > > > > > expected integer but got "" > > > expected integer but got "" > > > while executing > > > "$win sash place 0 $sash0 [lindex $s0 1]" > > > (procedure "resizeclistpanes" line 38) > > > invoked from within > > > "resizeclistpanes .tf.histframe.pwclist 1834" > > > (command bound to event) > > > > > > Is this a known bug? > > > > > > I’ll be happy to give more details. > > > Best, Christian > > > ``` ## Matthias Aßhauer, 2022-05-06 12:00 Subject: Re: Resizing panels in the gitk window Message-ID: URL: https://gitlist.dev/e/AM0PR04MB6019000A6C54CF97447E7B6EA5C59%40AM0PR04MB6019.eurprd04.prod.outlook.com In-Reply-To: ``` On Thu, 5 May 2022, Christian Grün wrote: > Resizing panels in the gitk window leads to a error message that’s > presented multiple times in a dialog: > > expected integer but got "" > expected integer but got "" > while executing > "$win sash place 0 $sash0 [lindex $s0 1]" > (procedure "resizeclistpanes" line 38) > invoked from within > "resizeclistpanes .tf.histframe.pwclist 1834" > (command bound to event) > > Is this a known bug? It looks like the issue at [1]. It sadly seems like nobody ever reviewed or applied thos suggested patches, but there is some more off-list discussion at [2]. > I’ll be happy to give more details. > Best, Christian > Best regards Matthias [1] https://lore.kernel.org/git/pull.1219.git.git.1645386457.gitgitgadget@gmail.com/#t [2] https://github.com/git/git/pull/1219#issuecomment-1113122632 ``` ## Christian Grün, 2022-05-06 12:12 Subject: Re: Resizing panels in the gitk window Message-ID: URL: https://gitlist.dev/e/CAP94bnO9myHnSv7V15XdJDZY6WkvA85r5m_1%3DkyFi5%3DGCYMmeQ%40mail.gmail.com In-Reply-To: ``` Thanks, Matthias, for the clarification. I’ll follow the discussion on GitHub. On Fri, May 6, 2022 at 2:00 PM Matthias Aßhauer wrote: > > > On Thu, 5 May 2022, Christian Grün wrote: > > > Resizing panels in the gitk window leads to a error message that’s > > presented multiple times in a dialog: > > > > expected integer but got "" > > expected integer but got "" > > while executing > > "$win sash place 0 $sash0 [lindex $s0 1]" > > (procedure "resizeclistpanes" line 38) > > invoked from within > > "resizeclistpanes .tf.histframe.pwclist 1834" > > (command bound to event) > > > > Is this a known bug? > > It looks like the issue at [1]. It sadly seems like nobody ever reviewed > or applied thos suggested patches, but there is some more off-list > discussion at [2]. > > > I’ll be happy to give more details. > > Best, Christian > > > > Best regards > > Matthias > > [1] https://lore.kernel.org/git/pull.1219.git.git.1645386457.gitgitgadget@gmail.com/#t > [2] https://github.com/git/git/pull/1219#issuecomment-1113122632 ``` ## Junio C Hamano, 2022-05-06 16:23 Subject: Re: Resizing panels in the gitk window Message-ID: URL: https://gitlist.dev/e/xmqqzgjuocjk.fsf%40gitster.g In-Reply-To: ``` Matthias Aßhauer writes: > It looks like the issue at [1]. It sadly seems like nobody ever > reviewed or applied thos suggested patches, but there is some more > off-list discussion at [2]. > > [1] https://lore.kernel.org/git/pull.1219.git.git.1645386457.gitgitgadget@gmail.com/#t > [2] https://github.com/git/git/pull/1219#issuecomment-1113122632 Thanks for a pointer. I do recall seeing [1] but I do not see anything happened on the thread since then. I didn't look at it back then primarily because the patches were made against a wrong history (no, you do not want to make a gitk patch against my tree nor throw a pull request at my repository), and there was no way for me to pick it up. As Documentation/SubmittingPatches says: Some parts of the system have dedicated maintainers with their own repositories. ... Patches to these parts should be based on their trees. Having said that, gitk has been quiescent for quite a while, and I wonder if Paul's been too busy for (or no longer interested in) maintaining it and perhaps he can use a replacement maintainer? ``` ## Halil SEN, 2022-05-09 12:46 Subject: Re: Resizing panels in the gitk window Message-ID: URL: https://gitlist.dev/e/CAOeUTuufDgsyp7FgJj4T%2Bh%3DJOMaWYVBkMx8S6fNRQ3cdVo6tOw%40mail.gmail.com In-Reply-To: ``` Junio C Hamano, wrote: > I didn't look at it back then primarily because > the patches were made against a wrong history I remember looking for Paul's repository but I failed to find it so created the patch against the main git repo. Maybe I should have mentioned this in my message since I knew the documentation included that note. I am happy to rebase my patch if someone could point me in the right direction. ``` ## Junio C Hamano, 2022-05-09 17:11 Subject: Re: Resizing panels in the gitk window Message-ID: URL: https://gitlist.dev/e/xmqqtu9yd42d.fsf%40gitster.g In-Reply-To: ``` Halil SEN writes: > Junio C Hamano, wrote: >> I didn't look at it back then primarily because >> the patches were made against a wrong history > > I remember looking for Paul's repository but I > failed to find it so created the patch against > the main git repo. Maybe I should have > mentioned this in my message since I knew > the documentation included that note. > > I am happy to rebase my patch if someone > could point me in the right direction. Thanks for giving me a chance to double-check what we have in Documentation/SubmittingPatches is not stale: - `gitk-git/` comes from Paul Mackerras's gitk project: git://ozlabs.org/~paulus/gitk I just made a fresh clone of it just to make sure. The latest commit in the repository is 6cd80496 (gitk: Resize panes correctly when reducing window size, 2020-10-03). It seems to be related to the topic you are working on? Again, I wonder if Paul's been too busy for (or no longer interested in) maintaining it and perhaps he can use a replacement maintainer? Thanks. ``` ## Paul Mackerras, 2022-05-11 08:17 Subject: Re: Resizing panels in the gitk window Message-ID: URL: https://gitlist.dev/e/YntxL%2FfTplFm8lr6%40cleo In-Reply-To: ``` On Fri, May 06, 2022 at 09:23:27AM -0700, Junio C Hamano wrote: > Matthias Aßhauer writes: > > > It looks like the issue at [1]. It sadly seems like nobody ever > > reviewed or applied thos suggested patches, but there is some more > > off-list discussion at [2]. > > > > [1] https://lore.kernel.org/git/pull.1219.git.git.1645386457.gitgitgadget@gmail.com/#t > > [2] https://github.com/git/git/pull/1219#issuecomment-1113122632 > > Thanks for a pointer. > > I do recall seeing [1] but I do not see anything happened on the > thread since then. I didn't look at it back then primarily because > the patches were made against a wrong history (no, you do not want > to make a gitk patch against my tree nor throw a pull request at my > repository), and there was no way for me to pick it up. I have just picked up Halil Sen's two patches and applied them, and pushed the result out to git.ozlabs.org. > As Documentation/SubmittingPatches says: > > Some parts of the system have dedicated maintainers with their own > repositories. > ... > Patches to these parts should be based on their trees. > > Having said that, gitk has been quiescent for quite a while, and I > wonder if Paul's been too busy for (or no longer interested in) > maintaining it and perhaps he can use a replacement maintainer? I am indeed very busy and would welcome a replacement maintainer for gitk, if somebody was willing to take it up. Thanks, Paul. ``` ## Paul Mackerras, 2022-05-11 08:20 Subject: Re: Resizing panels in the gitk window Message-ID: URL: https://gitlist.dev/e/Yntx33Q9pNLJ%2BQqS%40cleo In-Reply-To: ``` On Mon, May 09, 2022 at 10:11:06AM -0700, Junio C Hamano wrote: > Halil SEN writes: > > > Junio C Hamano, wrote: > >> I didn't look at it back then primarily because > >> the patches were made against a wrong history > > > > I remember looking for Paul's repository but I > > failed to find it so created the patch against > > the main git repo. Maybe I should have > > mentioned this in my message since I knew > > the documentation included that note. > > > > I am happy to rebase my patch if someone > > could point me in the right direction. > > Thanks for giving me a chance to double-check what we have in > Documentation/SubmittingPatches is not stale: > > - `gitk-git/` comes from Paul Mackerras's gitk project: > > git://ozlabs.org/~paulus/gitk Could you change that to git://git.ozlabs.org/~paulus/gitk please? The ozlabs.org infrastructure changes every so often, and while the plain ozlabs.org works now and is not expected to stop working, git.ozlabs.org would be a little more future-proof. Thanks, Paul. ``` ## Paul Mackerras, 2022-05-11 08:21 Subject: Re: Resizing panels in the gitk window Message-ID: URL: https://gitlist.dev/e/YntyF%2BdHp2WEpI8v%40cleo In-Reply-To: ``` On Mon, May 09, 2022 at 10:19:21PM +0200, Halil SEN wrote: > Junio C Hamano writes: > > > It seems to be related to the topic you are working on? > > It indeed is related. Sorry for troubling you with the repo, I don't > remember exactly how I failed before, thanks for sharing it. > > > Again, I wonder if Paul's been too busy for (or no longer interested > > in) maintaining it and perhaps he can use a replacement maintainer? > > I guess I should rebase my patch and resubmit it nevertheless. > I'll do it within the next few days. Now that I thought about it, > I think I had trouble creating and e-mailing the patch to the mail > list and the gitgitgadget was available only for the main repo. > I shall ask for help if I fail to do so again. No need to rebase, I have manually applied your patches. Junio, could you do a pull from my repository to get them? Thanks, Paul. ``` ## Junio C Hamano, 2022-05-11 15:08 Subject: Re: Resizing panels in the gitk window Message-ID: URL: https://gitlist.dev/e/xmqqpmkkm7if.fsf%40gitster.g In-Reply-To: ``` Paul Mackerras writes: > The ozlabs.org infrastructure changes every so often, and while the > plain ozlabs.org works now and is not expected to stop working, > git.ozlabs.org would be a little more future-proof. Will do. Thanks. ``` ## Junio C Hamano, 2022-05-11 15:25 Subject: Re: Resizing panels in the gitk window Message-ID: URL: https://gitlist.dev/e/xmqqlev8m6q4.fsf%40gitster.g In-Reply-To: ``` Paul Mackerras writes: > Junio, could you do a pull from my repository to get them? Thanks, done. ```