git@vger.kernel.org list mirror (unofficial, one of many)
 help / color / Atom feed
From: Pratyush Yadav <me@yadavpratyush.com>
To: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Cc: Harish Karumuthil <harish2704@gmail.com>,
	git@vger.kernel.org, David Aguilar <davvid@gmail.com>
Subject: Re: Making GitGitGadget's list -> PR comment mirroring bidirectional, was Re: [PATCH] Feature: custom guitool commands can now have custom keyboard shortcuts
Date: Wed, 20 Nov 2019 19:52:40 +0530
Message-ID: <20191120142240.pc5kfpj4eflu7dub@yadavpratyush.com> (raw)
In-Reply-To: <nycvar.QRO.7.76.6.1911192305410.15956@tvgsbejvaqbjf.bet>

On 19/11/19 11:09PM, Johannes Schindelin wrote:
> Hi Pratyush,
> 
> On Mon, 7 Oct 2019, Pratyush Yadav wrote:
> 
> > On 06/10/19 10:27PM, Johannes Schindelin wrote:
> > > Hi Pratyush,
> > >
> > > On Mon, 7 Oct 2019, Pratyush Yadav wrote:
> > >
> > > > Anyway, GitGitGadget solves a large part of the problem. It
> > > > eliminates the need for using git-send-email, and it even shows you
> > > > the replies received on the list. I honestly think it is a great
> > > > tool, and it gives people a very good alternative to using
> > > > git-send-email.
> > >
> > > GitGitGadget is just a workaround. Not even complete. Can't be
> > > complete, really. Because problems. It has much of the same problems
> > > of `git send-email`: it's a one-way conversation. Code is not
> > > discussed in the right context (which would be a worktree with the
> > > correct commit checked out). The transfer is lossy (email is designed
> > > for human-readable messages, not for transferring machine-readable
> > > serialized objects). Matching original commits and/or branches to the
> > > ones on the other side is tedious. Any interaction requires switching
> > > between many tools. Etc
> > >
> > > > One feature that would make it complete would be the ability to
> > > > reply to review comments.
> > >
> > > And how would that work, exactly? How to determine *which* email to
> > > respond to? *Which* person to reply to? *What* to quote?
> >
> > GGG already shows replies to the patches as a comment. On GitHub you can
> > "Quote reply" a comment, which quotes the entire comment just like your
> > MUA would. The option can be found by clicking the 3 dots on the top
> > right of a comment.
> >
> > Then you can write your reply there, and the last line would be
> > '/reply', which would make GGG send that email as a reply. You would
> > need to strip the first line from the reply because GGG starts the reply
> > with something like:
> >
> >   > [On the Git mailing list](https://public-inbox.org/git/xmqq7e5l9zb1.fsf@gitster-ct.c.googlers.com), Junio C Hamano wrote ([reply to this](https://github.com/gitgitgadget/gitgitgadget/wiki/ReplyToThis)):
> >
> > GGG also adds 3 backticks before and after the reply content, so those
> > would need to be removed too.
> >
> > Does this sound like a sane solution?
> 
> Here are two real life examples where an unsuspecting GitGitGadget user
> expected GitGitGadget to mirror replies _to_ the Git mailing list:
> 
> https://github.com/gitgitgadget/git/pull/451#issuecomment-555044068 and
> https://github.com/gitgitgadget/git/pull/451#issuecomment-555077933
> 
> Neither of them include the line with the link.

Correct.

The fundamental problem we have is that GitHub's threads are 
"shallow"/"linear". You don't reply to a reply, you reply to the main 
thread, and your comment gets appended to the end of that list. In 
contrast, email based threads are "deep"/"tree-like". Here you can reply 
to a reply.

So the comment model of GitHub is less information-rich than the email 
model. The piece of information missing is "which comment does this 
comment reply to". That information has to be obtained somehow, and the 
best bet are the users themselves (by not deleting the line with the 
link in their replies). We can give the instructions in the GGG welcome 
message, and hope the users read it.

Frequent users and anyone who properly reads the instructions will 
probably manage to use this just fine. Those who don't, well they 
weren't sending replies to the list to begin with. So this will help 
people who don't want to open their email clients to send replies to the 
list, but won't help the uninformed ones. Certainly not ideal, but I 
think this might be the best we can do given the constraints.

In the meantime, I notice that GGG does not advertise the fact that 
replies don't go on the list directly very well. Yes, its mentioned in 
the welcome message, but its not instantly obvious. So maybe making it 
clearer/more noticeable will help the issue.

Another alternative might be to rely on heuristics like seeing how 
similar the quoted text in a reply is to the replies in the list. But I 
think this will cause more problems than help because users can cut 
un-necessary quoting and even edit them sometimes.

If you can think of something clever that I can't, suggestions are 
welcome :)

Anyway, I probably won't have much time to work on this feature for at 
least a couple more weeks. Maybe we'll learn more after the feature goes 
live.

> Just to throw a bit of real life into the discussion...
> 
> Ciao,
> Dscho

-- 
Regards,
Pratyush Yadav

  reply index

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-29 11:38 Harish K
2016-03-31 16:41 ` David Aguilar
2016-04-01  6:32   ` harish k
2019-10-03 14:48     ` harish k
2019-10-03 21:44       ` Pratyush Yadav
2019-10-04  8:49         ` Johannes Schindelin
2019-10-04 12:01           ` Pratyush Yadav
2019-10-05 20:16             ` Harish Karumuthil
2019-10-05 21:01               ` Pratyush Yadav
2019-10-06  9:49                 ` Johannes Schindelin
2019-10-06 18:39                   ` Pratyush Yadav
2019-10-06 19:37                     ` Philip Oakley
2019-10-06 20:27                     ` Johannes Schindelin
2019-10-06 21:06                       ` Pratyush Yadav
2019-10-07  9:20                         ` GitGUIGadget, was " Johannes Schindelin
2019-10-07 10:43                           ` Birger Skogeng Pedersen
2019-10-07 19:16                             ` Alban Gruin
2019-11-19 22:09                         ` Making GitGitGadget's list -> PR comment mirroring bidirectional, " Johannes Schindelin
2019-11-20 14:22                           ` Pratyush Yadav [this message]
2019-10-06 22:40                       ` Philip Oakley
2019-10-07  6:22                   ` Harish Karumuthil
2019-10-07 10:01                     ` Johannes Schindelin
2019-10-08 19:31                       ` Harish Karumuthil
2019-10-09 20:42                         ` Johannes Schindelin
2019-10-13 20:09                         ` Pratyush Yadav
2019-10-07  6:13                 ` Harish Karumuthil
2019-10-13 19:17                   ` Pratyush Yadav

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20191120142240.pc5kfpj4eflu7dub@yadavpratyush.com \
    --to=me@yadavpratyush.com \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=davvid@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=harish2704@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

git@vger.kernel.org list mirror (unofficial, one of many)

Archives are clonable:
	git clone --mirror https://public-inbox.org/git
	git clone --mirror http://ou63pmih66umazou.onion/git
	git clone --mirror http://czquwvybam4bgbro.onion/git
	git clone --mirror http://hjrcffqmbrq6wope.onion/git

Example config snippet for mirrors

Newsgroups are available over NNTP:
	nntp://news.public-inbox.org/inbox.comp.version-control.git
	nntp://ou63pmih66umazou.onion/inbox.comp.version-control.git
	nntp://czquwvybam4bgbro.onion/inbox.comp.version-control.git
	nntp://hjrcffqmbrq6wope.onion/inbox.comp.version-control.git
	nntp://news.gmane.io/gmane.comp.version-control.git

 note: .onion URLs require Tor: https://www.torproject.org/

AGPL code for this site: git clone https://public-inbox.org/public-inbox.git