threads / bug / 53377

tig: bug when binding to 'generic'?

Subject: tig: bug when binding to 'generic'?

## tl;dr

6 messages between May 2, 2020 and May 4, 2020.

replies: 5people: 5as markdown or json

Wolfram Sang· May 2, 2020, 19:04 UTC · lore
Hi,
here is a minimal, problematic .tigrc:

=== bind generic F !echo "Hello World!" ===

This works for most views, but not for 'main' where it still toggles the refs. Adding a specific 'main' binding works:

=== bind main F !echo "Hello World!" ===

It also works if I bind the 'generic' command to an unused key, e.g. '4'.
Discovered with Debian's 2.4.1, but confirmed against HEAD of today.
A bug?
Thanks,
   Wolfram
Kevin Daudt· May 2, 2020, 19:07 UTC · re: Wolfram Sang · lore

Re: tig: bug when binding to 'generic'?

On Sat, May 02, 2020 at 09:04:33PM +0200, Wolfram Sang wrote:
Show 25 quoted lines
> Hi,
> 
> here is a minimal, problematic .tigrc:
> 
> ===
> bind	generic	F	!echo "Hello World!"
> ===
> 
> This works for most views, but not for 'main' where it still toggles the
> refs. Adding a specific 'main' binding works:
> 
> ===
> bind	main	F	!echo "Hello World!"
> ===
> 
> It also works if I bind the 'generic' command to an unused key, e.g. '4'.
> 
> Discovered with Debian's 2.4.1, but confirmed against HEAD of today.
> 
> A bug?
> 
> Thanks,
> 
>    Wolfram
> 
Hello Wolfram,

Tig is a third-party interface, not maintained by git. You can report issues for tig on [github][0]

Kind regards, Kevin
[0]:https://github.com/jonas/tig/issues
Wolfram Sang· May 2, 2020, 19:31 UTC · re: Kevin Daudt · lore

Re: tig: bug when binding to 'generic'?

Hi Kevin,
> Tig is a third-party interface, not maintained by git. You can report
> issues for tig on [github][0]
I know it is third-party. Seems the README in tig needs fixing, too:

=== Bugs and feature requests can be reported using the issue tracker or by mail to either the Git mailing list or directly to the maintainer. ==

I prefer the mail option, so I wrote here and CCed the maintainers.
Kind regards,
   Wolfram
Taylor Blau· May 4, 2020, 22:05 UTC · re: Wolfram Sang · lore

Re: tig: bug when binding to 'generic'?

On Sat, May 02, 2020 at 09:31:02PM +0200, Wolfram Sang wrote:
Show 13 quoted lines
> Hi Kevin,
>
> > Tig is a third-party interface, not maintained by git. You can report
> > issues for tig on [github][0]
>
> I know it is third-party. Seems the README in tig needs fixing, too:
>
> ===
> Bugs and feature requests can be reported using the issue tracker or by
> mail to either the Git mailing list or directly to the maintainer.
> ==
>
> I prefer the mail option, so I wrote here and CCed the maintainers.

Thanks for clarifying. I was a little confused when I read your patch series, since on first blush I thought that you were submitting it to be applied in Git's integration branches, and not Tig's.

In either case, the heads-up is appreciated, but I do not think necessary in general. I am not sure about the feelings of others, but I do not think Tig-specific bugs need to be reported on Git's mailing list.

Of course, if there are Git bugs that affect Tig (or any other project), we would be interested in hearing about them.

That's all to say that I think the README there should be updated.
> Kind regards,
>
>    Wolfram
>

Thanks, Taylor

Junio C Hamano· May 4, 2020, 23:01 UTC · re: Taylor Blau · lore

Re: tig: bug when binding to 'generic'?

Taylor Blau <me@ttaylorr.com> writes:
Show 13 quoted lines
> Thanks for clarifying. I was a little confused when I read your patch
> series, since on first blush I thought that you were submitting it to be
> applied in Git's integration branches, and not Tig's.
>
> In either case, the heads-up is appreciated, but I do not think
> necessary in general. I am not sure about the feelings of others, but I
> do not think Tig-specific bugs need to be reported on Git's mailing
> list.
>
> Of course, if there are Git bugs that affect Tig (or any other project),
> we would be interested in hearing about them.
>
> That's all to say that I think the README there should be updated.

FWIW, I do not mind seeing tig related improvements discussed here (I am assuming tig does not have its own development list). That is how we've been operating with other "projects around Git", like "imerge" and "tbdiff" (no need to look for the latter these days, though).

Unless Jonas and others who are involved find the list traffic here is too heavy and messages about tig will be drowned in here, I do not see a reason to kick them out.

Taylor Blau· May 4, 2020, 23:14 UTC · re: Junio C Hamano · lore

Re: tig: bug when binding to 'generic'?

On Mon, May 04, 2020 at 04:01:40PM -0700, Junio C Hamano wrote:
Show 21 quoted lines
> Taylor Blau <me@ttaylorr.com> writes:
>
> > Thanks for clarifying. I was a little confused when I read your patch
> > series, since on first blush I thought that you were submitting it to be
> > applied in Git's integration branches, and not Tig's.
> >
> > In either case, the heads-up is appreciated, but I do not think
> > necessary in general. I am not sure about the feelings of others, but I
> > do not think Tig-specific bugs need to be reported on Git's mailing
> > list.
> >
> > Of course, if there are Git bugs that affect Tig (or any other project),
> > we would be interested in hearing about them.
> >
> > That's all to say that I think the README there should be updated.
>
> FWIW, I do not mind seeing tig related improvements discussed here
> (I am assuming tig does not have its own development list).  That is
> how we've been operating with other "projects around Git", like
> "imerge" and "tbdiff" (no need to look for the latter these days,
> though).

Ah, fair enough: I do remember seeing an imerge query on the list a couple of days (weeks?) ago.

> Unless Jonas and others who are involved find the list traffic here
> is too heavy and messages about tig will be drowned in here, I do
> not see a reason to kick them out.

OK, that sounds good (it wasn't really up to me anyway, but I'm glad that I know how you feel about the matter ;)).

Thanks, Taylor

← back to recent threads