darcs

Issue 1758 Make darcs send more informative about what repo a patch is for

Title Make darcs send more informative about what repo a patch is for
Priority wishlist Status duplicate
Milestone Resolved in
Superseder patch bundle email should mention repository name
View: 1362
Nosy List darcs-devel, dmitry.kurochkin, gwern, kowey
Assigned To
Topics

Created on 2010-03-09.14:01:13 by gwern, last changed 2010-03-09.14:07:29 by kowey.

Messages
msg10135 (view) Author: gwern Date: 2010-03-09.14:01:08
I recently sent Malcolm Wallace a patch for the nobench repo, but he
couldn't figure out what repo it was for since he has so many. There is
a 'DarcsURL' field buried in the email headers, but very few people know
about it or look at headers.

Perhaps we could include the repo URL in the body of the email? (since
the subject seems to be overloaded as it is).

It'd look, I think, something like:


hunk ./src/Darcs/Email.hs 211
             text "Content-Type: text/plain"
          $$ text "Content-Transfer-Encoding: quoted-printable"
          $$ text ""
+         $$ text "A patch for repo:" <+> text repodir
          $$ packedString (qpencode (renderPS contents))
          $$ text ""
          $$ text "--=_"

(I can't compile right now because I'm not sure where to get icuuc, but
I think this is right.)
msg10136 (view) Author: kowey Date: 2010-03-09.14:07:27
I think this was fixed in Darcs-2.4, but thanks!
History
Date User Action Args
2010-03-09 14:01:13gwerncreate
2010-03-09 14:07:30koweysetpriority: wishlist
status: unknown -> duplicate
superseder: + patch bundle email should mention repository name
messages: + msg10136
nosy: + kowey