# [BUG] git gui blame: Show History Context [still] broken since 29e5573d 7 messages from 2010-06-24 to 2010-07-27. Participants: Matthieu Moy, Pat Thoyts, Junio C Hamano, Jonathan Nieder. Thread: https://gitlist.dev/t/24187 ## Matthieu Moy, 2010-06-24 16:28 Subject: [BUG] git gui blame: Show History Context [still] broken since 29e5573d Message-ID: URL: https://gitlist.dev/e/vpqy6e4v1h0.fsf%40bauges.imag.fr ``` Hi, I reported this bug a while ago: http://kerneltrap.org/mailarchive/git/2010/2/22/24038 In short, the "Show history context" item of the context menu of "git gui blame" is broken since february. Two patches were proposed: http://thread.gmane.org/gmane.comp.version-control.git/140688 http://thread.gmane.org/gmane.comp.version-control.git/140288 But I've just noticed that none of theme were applied. -- Matthieu Moy http://www-verimag.imag.fr/~moy/ ``` ## Pat Thoyts, 2010-07-10 23:41 Subject: Re: [BUG] git gui blame: Show History Context [still] broken since 29e5573d Message-ID: <87sk3qrjk3.fsf@fox.patthoyts.tk> URL: https://gitlist.dev/e/87sk3qrjk3.fsf%40fox.patthoyts.tk In-Reply-To: ``` Matthieu Moy writes: >Hi, > >I reported this bug a while ago: > >http://kerneltrap.org/mailarchive/git/2010/2/22/24038 > >In short, the "Show history context" item of the context menu of "git >gui blame" is broken since february. Two patches were proposed: > >http://thread.gmane.org/gmane.comp.version-control.git/140688 >http://thread.gmane.org/gmane.comp.version-control.git/140288 > >But I've just noticed that none of theme were applied. > Checking this it seems you are correct. This has been fixed for msysGit with the above patch included there but is missing from git-gui itself and hence missing from core git. -- Pat Thoyts http://www.patthoyts.tk/ PGP fingerprint 2C 6E 98 07 2C 59 C8 97 10 CE 11 E6 04 E0 B9 DD ``` ## Junio C Hamano, 2010-07-12 20:55 Subject: Re: [BUG] git gui blame: Show History Context [still] broken since 29e5573d Message-ID: <7vtyo4a096.fsf@alter.siamese.dyndns.org> URL: https://gitlist.dev/e/7vtyo4a096.fsf%40alter.siamese.dyndns.org In-Reply-To: <87sk3qrjk3.fsf@fox.patthoyts.tk> ``` Pat Thoyts writes: > Matthieu Moy writes: > >>Hi, >> >>I reported this bug a while ago: >> >>http://kerneltrap.org/mailarchive/git/2010/2/22/24038 >> >>In short, the "Show history context" item of the context menu of "git >>gui blame" is broken since february. Two patches were proposed: >> >>http://thread.gmane.org/gmane.comp.version-control.git/140688 >>http://thread.gmane.org/gmane.comp.version-control.git/140288 >> >>But I've just noticed that none of theme were applied. > > Checking this it seems you are correct. This has been fixed for > msysGit with the above patch included there but is missing from > git-gui itself and hence missing from core git. Can we resolve this before 1.7.2 final? Shawn, I see 9c898a1 (git-gui: check whether systems nice command works or disable it, 2010-02-07) on your 'maint' that is not in mine (and a merge of that commit on your 'master'), but neither of these. ``` ## Matthieu Moy, 2010-07-22 08:11 Subject: Re: [BUG] git gui blame: Show History Context [still] broken since 29e5573d Message-ID: URL: https://gitlist.dev/e/vpqzkxkorzr.fsf%40bauges.imag.fr In-Reply-To: <7vtyo4a096.fsf@alter.siamese.dyndns.org> ``` Junio C Hamano writes: > Pat Thoyts writes: > >> Matthieu Moy writes: >> >>>Hi, >>> >>>I reported this bug a while ago: >>> >>>http://kerneltrap.org/mailarchive/git/2010/2/22/24038 >>> >>>In short, the "Show history context" item of the context menu of "git >>>gui blame" is broken since february. Two patches were proposed: >>> >>>http://thread.gmane.org/gmane.comp.version-control.git/140688 >>>http://thread.gmane.org/gmane.comp.version-control.git/140288 >>> >>>But I've just noticed that none of theme were applied. >> >> Checking this it seems you are correct. This has been fixed for >> msysGit with the above patch included there but is missing from >> git-gui itself and hence missing from core git. > > Can we resolve this before 1.7.2 final? Too late for the 1.7.2, but can we finally get one of these patches applied? They're both very short and straightforward, and fix an important feature of git-gui which is basically broken as of now. Thanks, -- Matthieu Moy http://www-verimag.imag.fr/~moy/ ``` ## Jonathan Nieder, 2010-07-22 08:23 Subject: Re: [BUG] git gui blame: Show History Context [still] broken since 29e5573d Message-ID: <20100722082310.GB13386@burratino> URL: https://gitlist.dev/e/20100722082310.GB13386%40burratino In-Reply-To: ``` Matthieu Moy wrote: > Too late for the 1.7.2, but can we finally get one of these patches > applied? They're both very short and straightforward, and fix an > important feature of git-gui which is basically broken as of now. Maybe some interested person should maintain a temporary fork of git-gui that could be merged in the meantime. That is what people seem to usually do with submodules in analogous situations. ``` ## Pat Thoyts, 2010-07-25 20:19 Subject: Re: [BUG] git gui blame: Show History Context [still] broken since 29e5573d Message-ID: <87iq43wbz3.fsf@fox.patthoyts.tk> URL: https://gitlist.dev/e/87iq43wbz3.fsf%40fox.patthoyts.tk In-Reply-To: <20100722082310.GB13386@burratino> ``` Jonathan Nieder writes: >Matthieu Moy wrote: > >> Too late for the 1.7.2, but can we finally get one of these patches >> applied? They're both very short and straightforward, and fix an >> important feature of git-gui which is basically broken as of now. > >Maybe some interested person should maintain a temporary fork of >git-gui that could be merged in the meantime. That is what people >seem to usually do with submodules in analogous situations. I pushed git-gui + my patches to git://git.patthoyts.tk/git-gui (http://www.patthoyts.tk/gitweb.cgi?p=git-gui;a=summary) so if someone wants to pull from that it is Shawn's repository plus 3. Alternatively the git-gui patches could be pulled from msysGit which has a few in addition to these three. -- Pat Thoyts http://www.patthoyts.tk/ PGP fingerprint 2C 6E 98 07 2C 59 C8 97 10 CE 11 E6 04 E0 B9 DD ``` ## Jonathan Nieder, 2010-07-27 01:09 Subject: git gui tree (Re: [BUG] git gui blame: Show History Context [still] broken since 29e5573d) Message-ID: <20100727010911.GA4041@burratino> URL: https://gitlist.dev/e/20100727010911.GA4041%40burratino In-Reply-To: <87iq43wbz3.fsf@fox.patthoyts.tk> ``` Pat Thoyts wrote: > I pushed git-gui + my patches to git://git.patthoyts.tk/git-gui > (http://www.patthoyts.tk/gitweb.cgi?p=git-gui;a=summary) > so if someone wants to pull from that it is Shawn's repository plus 3. Quick thoughts: Were those patches reviewed? I assume so, but I do not see a Reviewed-by or anything so I do not know who to blame if they go awry. Here[1] is another candidate patch, though it has not received much attention, unfortunately. Thanks, Jonathan [1] http://thread.gmane.org/gmane.linux.debian.devel.bugs.general/723900/focus=150231 ```