unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Drew Adams <drew.adams@oracle.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: "stefan@marxist.se" <stefan@marxist.se>,
	"17567@debbugs.gnu.org" <17567@debbugs.gnu.org>
Subject: bug#17567: [External] : Re: bug#17567: 24.4.50; doc string of `define-derived-mode'
Date: Sat, 25 Sep 2021 21:08:22 +0000	[thread overview]
Message-ID: <SJ0PR10MB54888F45EF95239E40B973DDF3A59@SJ0PR10MB5488.namprd10.prod.outlook.com> (raw)
In-Reply-To: <835yuoph36.fsf@gnu.org>

> > If you'd like me to check the fix (Stefan's request)
> > then please consider putting the new text in the bug
> > thread.  Otherwise, you'll just have to do without
> > my review and feedback, I guess.  Not a biggee (the
> > bug was anyway closed, without such a review).
> 
> This response makes no sense to me.  You don't mind writing the
> longest email messages ever, but refuse to type a 50-character URL
> into your browser?  Why?

This response makes no sense to me.  You don't mind
dragging out this thread, but pasting the updated
doc string into the thread, after requesting its
review, is too much trouble?  Why?

Reviewing text starts with the text to be reviewed.
And it's often collaborative - back & forth.  Doing
that in one place is more productive than sending a
reviewer off to some diffs somewhere.  A typical
doc review involves, well, doc, not just diffs.

> But whatever.

Indeed.

> > > If you refuse to do that, you are not helping
> > > the project as much as you could have.
> >
> > If you refuse to put the updated doc string in a bug
> > thread that asks only for a doc-string fix, then
> > you're not helping users, or the project, as much as
> > you could.  Copy...paste...send.  Simple.
> 
> No, it's not simple.  There's no place to copy/paste from, because
> installing a change usually doesn't involve looking at the diffs, or
> even producing them.  We edit the sources and commit them, that's it.

It's apparently too hard for someone who's fixed a
doc string to look at the edited result (sources)
in an Emacs buffer and copy that.  OK, got it.

It's likewise too hard for me to dig out the diffs
and reconstruct the resulting doc string from them,
in order to respond to a review request.

Not the end of the world, but the end of the review,
I guess.  Again, I didn't ask to review the fix.

(And the bug is already closed.  I didn't say a
word about that, except "Thanks".  I just responded
to the review request ("please verify"), with a
request for the text to review.)

> > (I'd even consider that common courtesy, but you're
> > by no means obliged to see courtesy the same way,
> > of course.)
> 
> Yes of course: everyone has to be courteous to you,
> but you don't feel you need to be courteous to others.

I do feel that need.  And I try to be courteous,
and I want to, even when some, such as yourself,
are needlessly aggressive, such as now.

And it's not about being courteous to me.  Showing
the text to be reviewed is, in my world, a normal,
courteous part of asking for a review - regardless
of who the review is requested from or who's doing
the requesting.

Asking for a review is asking for help.  Unnecessary
hurdles make it harder to help.  Filing bug reports,
just like fixing bugs, is helping, or trying to.
We're all volunteers here, though you seem to keep
opposing volunteers who fix problems to volunteers
who report them.  I'm thankful for both the fixers
and the reporters.





  reply	other threads:[~2021-09-25 21:08 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-23 19:15 bug#17567: 24.4.50; doc string of `define-derived-mode' Drew Adams
2014-05-23 19:35 ` Drew Adams
2021-09-25 17:15   ` Stefan Kangas
2021-09-25 17:30     ` Eli Zaretskii
2021-09-25 18:01       ` Stefan Kangas
2021-09-25 18:46         ` Eli Zaretskii
2021-09-25 19:07           ` Stefan Kangas
2021-09-25 19:11             ` Eli Zaretskii
2021-09-25 19:38               ` Stefan Kangas
2021-09-25 17:36     ` bug#17567: [External] : " Drew Adams
2021-09-25 18:42       ` Eli Zaretskii
2021-09-25 18:59         ` Drew Adams
2021-09-25 19:05           ` Eli Zaretskii
2021-09-25 19:44             ` Drew Adams
2021-09-25 19:53               ` Eli Zaretskii
2021-09-25 21:08                 ` Drew Adams [this message]
2021-09-27 22:36                   ` Richard Stallman

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: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=SJ0PR10MB54888F45EF95239E40B973DDF3A59@SJ0PR10MB5488.namprd10.prod.outlook.com \
    --to=drew.adams@oracle.com \
    --cc=17567@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    --cc=stefan@marxist.se \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).