darcs

Issue 1362 patch bundle email should mention repository name

Title patch bundle email should mention repository name
Priority wishlist Status resolved
Milestone Resolved in
Superseder Nosy List WorldMaker, darcs-devel, dmitry.kurochkin, jaredj, kowey, nomeata, thorkilnaur, twb
Assigned To
Topics ProbablyEasy

Created on 2009-02-15.19:33:12 by kowey, last changed 2010-01-07.09:48:43 by nomeata.

Messages
msg7320 (view) Author: kowey Date: 2009-02-15.19:33:09
If you have multiple darcs repositories for which you receive darcs
patches, it would be nice if you could easily tell which repository
the patch is for.

Instead of 
 darcs patch: name of patch
 darcs patch: name of some other patch,

Something like
 darcs patch: [project foo] name of patch
 darcs patch: [project bar] name of some other patch

would make life a tiny bit easier.

I suppose the configuration for this would live in _darcs/prefs
(much like _darcs/prefs/email)

I'm sure there are possible refinements to this idea as well...
msg7321 (view) Author: WorldMaker Date: 2009-02-15.22:49:51
Eric Kow wrote:
> New submission from Eric Kow <kowey@darcs.net>:
> 
> If you have multiple darcs repositories for which you receive darcs
> patches, it would be nice if you could easily tell which repository
> the patch is for.
> 
> Instead of 
>  darcs patch: name of patch
>  darcs patch: name of some other patch,
> 
> Something like
>  darcs patch: [project foo] name of patch
>  darcs patch: [project bar] name of some other patch
> 
> would make life a tiny bit easier.

There is a header that ``darcs send`` includes in the patch mail already 
that tells you the repository the patch is sent to. For example:

DarcsURL: http://darcs.net/

Presumably one simple way to get what you seek is a small plugin to your 
preferred email client to get it to show that DarcsURL header somewhere 
you might better notice it.
msg7322 (view) Author: kowey Date: 2009-02-16.08:38:02
Oh yeah!  I had not considered that.  

This could still be useful if patches go to mailing lists (because then you only
have to configure one thing and everybody benefits from it, instead of everybody
configuring their mail clients).  Also, in mail clients, it is typical to get a
summary of messages, showing the subject lines of all messages at a glance.

Making Darcs-URL more prominent will be useful for when you decide to view a
message, but it doesn't help much when you're looking at the message index
(unless you're willing to do some work to make darcs-url appear in the index,
which is likely possible in mutt).

Also, this sort of option would not require adding any new flags to darcs, just
a new preference file, which is not so bad.

Anyway, I won't be too bothered if this is marked wont-fix
msg8322 (view) Author: kowey Date: 2009-08-21.17:43:57
I guess we might as well wont-fix this
msg9674 (view) Author: nomeata Date: 2009-12-24.10:57:56
The text of the mail messages that darcs send produces lists the patches (with
author, name, description). It does not mention the name or URL of the
repository the mails are sent to. This is unfortunate on mailing lists like
libraries or xmonad, where patches against multiple repos are sent in. One has
to guess by looking at the filenames _inside_ the diff to find out what repo the
patch is aimed for.

I suggest a line like "5 new patches for repository http:/....:" before the list
of patches in the mail body.
msg9676 (view) Author: kowey Date: 2009-12-24.11:07:51
I brought msg9674 in from issue1722.

Joachim, please set this back to wont-fix if you think the DarcsURL header is
really good enough after all.
msg9682 (view) Author: twb Date: 2009-12-25.10:45:56
[For completeness, mentioning an alternative approach.]

I always assumed that when this became an issue, I'd do it this way:

    http://en.wikipedia.org/wiki/Email_address#Sub-addressing

Specifically, suppose you have three projects, "foo", "bar" and "baz".  Then

    echo "nurk+foo@example.net" >foo/_darcs/prefs/email
    echo "nurk+bar@example.net" >bar/_darcs/prefs/email
    echo "nurk+baz@example.net" >baz/_darcs/prefs/email

Now a "darcs send" against foo will be sent to nurk+foo.  It'll still
end up in the "nurk" inbox, but an MTA (e.g. postfix) or MDA
(maildrop, procmail) can be intelligent about which repository it
attempts to apply it to, per "Setting up a sendable repository using
procmail" in

    http://darcs.net/manual/node3.html#SECTION00355000000000000000

I guess even through sub-addresses support is now widespread, many
users' MUAs don't know how to filter based on them... You also
mentioned an existing X-DarcsURL field, which I guess is nearly as
easy for an MTA/MDA to work with.
msg9742 (view) Author: nomeata Date: 2010-01-07.09:48:39
The following patch updated the status of issue1362 to be resolved:

* Resolve issue1362: Mention repo name in mail send body 
Ignore-this: 3e5e0bbc5415b1b3d7c72ba3e8e077c8
History
Date User Action Args
2009-02-15 19:33:12koweycreate
2009-02-15 19:33:32koweysetpriority: wishlist
nosy: + jaredj
topic: + ProbablyEasy
2009-02-15 22:49:55maxbattchersetstatus: unread -> unknown
nosy: + darcs-devel, maxbattcher
messages: + msg7321
2009-02-16 08:38:05koweysetnosy: - darcs-devel
messages: + msg7322
2009-08-21 17:43:59koweysetstatus: unknown -> wont-fix
nosy: kowey, maxbattcher, simon, thorkilnaur, jaredj, dmitry.kurochkin
messages: + msg8322
2009-08-25 17:41:01adminsetnosy: + darcs-devel, - simon
2009-08-27 14:28:54adminsetnosy: kowey, darcs-devel, maxbattcher, thorkilnaur, jaredj, dmitry.kurochkin
2009-10-23 23:59:52adminsetnosy: + WorldMaker, - maxbattcher
2009-12-24 11:03:18koweylinkissue1722 superseder
2009-12-24 11:04:37koweysetnosy: + nomeata
2009-12-24 11:05:20adminsetmessages: + msg9674
2009-12-24 11:07:53koweysetstatus: wont-fix -> needs-implementation
messages: + msg9676
title: wish: distinguishing subject line tags for patch bundles -> patch bundle email should mention repository name
2009-12-24 11:41:04nomeatalinkpatch121 issues
2009-12-25 10:45:58twbsetnosy: + twb
messages: + msg9682
2010-01-07 09:48:43nomeatasetstatus: needs-implementation -> resolved
messages: + msg9742
2010-03-09 14:07:29koweylinkissue1758 superseder