darcs

Issue 643 wish: "darcs send -o" should output the remote repo e-mail address.

Title wish: "darcs send -o" should output the remote repo e-mail address.
Priority wishlist Status resolved
Milestone 2.5.0 Resolved in 2.5.0
Superseder Nosy List btcoburn, darcs-devel, dmitry.kurochkin, gh, jaredj, kowey, markstos, thorkilnaur, tommy
Assigned To
Topics ProbablyEasy, UI

Created on 2008-02-04.00:23:53 by btcoburn, last changed 2010-06-15.22:14:16 by admin.

Messages
msg3086 (view) Author: btcoburn Date: 2008-02-04.00:23:52
Darcs "get" does not download the "_darcs/prefs/email" preference. Maybe I'm
misunderstanding the manual here, but this behavior seems wrong. IMHO, the
default email for darcs "send" should follow the repository around like the
"defaultrepo" preference.

This happens with both darcs 2.* and darcs 1.0.9.
msg3092 (view) Author: droundy Date: 2008-02-04.15:23:57
On Mon, Feb 04, 2008 at 12:23:53AM -0000, Ben Coburn wrote:
> Darcs "get" does not download the "_darcs/prefs/email" preference. Maybe I'm
> misunderstanding the manual here, but this behavior seems wrong. IMHO, the
> default email for darcs "send" should follow the repository around like the
> "defaultrepo" preference.

I disagree.  Since the 'email' pref is retrieved from the remote repo when
you say "I want to send to this remote repo", having the email pref follow
you around would result in things like this:

echo "foo@bar.com" > trunk/_darcs/prefs/email
darcs get trunk local
darcs get local work
cd work
<do stuff>
darcs send
<darcs looks at local's email pref, and sends the patch to trunk's e-mail
address>

Which means that the patch ends up in trunk, and not in local, which isn't
what's expected.  What's more, repeated sends will say "ooh, I need to send
this patch" when it'll just get ignored by the actual repo that the patch is
being applied to because it's already been applied.

The current behaviour ultimately makes a lot more sense, IMO.  This is even
despite that fact that I use multi-level repos like you're proposing, to
allow me to do a darcs send while offline (or make it quicker when I'm
online) -- I have a local 'trunk' that is a replica of the main upstream
repo, with the upstream's e-mail address in the 'email' pref.  Despite it's
utility, I don't think it should be the default behaviour, because it
requires a lot more thought on my part to track where patches are and need
to be moved to.  Making it the default would just confuse people.

- Matt
_______________________________________________
darcs-devel mailing list
darcs-devel@darcs.net
http://lists.osuosl.org/mailman/listinfo/darcs-devel
msg3096 (view) Author: droundy Date: 2008-02-04.15:27:59
That last post here wasn't actually from me, but from Matt, I just bounced it to
the bug tracker.  Anyhow, I agree with him.  email is like motd, it's a
repository-specific setting.  If I get your repository, there's no reason why
someone sending patches to my repository should email them to you.

David
msg3117 (view) Author: btcoburn Date: 2008-02-04.22:49:08
On Feb 4, 2008, at 07:28, David Roundy wrote:

> That last post here wasn't actually from me, but from Matt, I just  
> bounced it to the bug tracker.  Anyhow, I agree with him.  email 
> is like motd, it's a repository-specific setting.  If I get your 
> repository, there's no reason why someone sending patches to my 
> repository should email them to you.
>


Actually, I agree with everyone that the current behavior is correct  
(but I have a small suggestion[1]).... Turns out I just did not  
understand how darcs was looking up the correct email address when  
doing a "darcs send".

To make sure I've go this right.... When doing a "darcs send", darcs  
retrieves the email address stored in the default repository (most  
recently pushed or pulled from, without --no-set-default) and then  
uses that address for the "darcs send".


[1] I think what prompted me to misunderstand this to begin with, was  
that "darcs send -o" does not display the email address that "darcs  
send" would have used. This forces users who can only "darcs send -o"  
to manually look up the email address, possibly from a non-standard  
location in project documentation, which can get out of sync with  
darcs. IMHO, before "darcs send -o" finishes it should print the email  
address to which the user is expected to send the patch.


Regards, Ben Coburn
msg3119 (view) Author: btcoburn Date: 2008-02-05.12:27:24
On Feb 4, 2008, at 07:28, David Roundy wrote:

>
>
> That last post here wasn't actually from me, but from Matt, I just  
> bounced it to
> the bug tracker.  Anyhow, I agree with him.  email is like motd,  
> it's a
> repository-specific setting.  If I get your repository, there's no  
> reason why
> someone sending patches to my repository should email them to you.
>

Actually, I agree with everyone that the current behavior is correct  
(but I have a small suggestion[1]).... Turns out I just did not  
understand how darcs was looking up the correct email address when  
doing a "darcs send".

To make sure I've go this right.... When doing a "darcs send", darcs  
retrieves the email address stored in the default repository (most  
recently pushed or pulled from, without --no-set-default) and then  
uses that address for the "darcs send".

[1] I think what prompted me to misunderstand this to begin with, was  
that "darcs send -o" does not display the email address that "darcs  
send" would have used. This forces users who can only "darcs send -o"  
to manually look up the email address, possibly from a non-standard  
location in project documentation, which can get out of sync with  
darcs. IMHO, before "darcs send -o" finishes it should print the email  
address to which the user is expected to send the patch.

Regards, Ben Coburn
msg3176 (view) Author: markstos Date: 2008-02-07.05:03:30
I agree with the revised "wish" that "darcs send -o" also provide the e-mail of
the remote repo, realizing the language would need to be different...it would
just be factually  reporting the address, insteading of claiming that an e-mail
has been sent there.
msg10144 (view) Author: kowey Date: 2010-03-09.20:16:09
Bumping to 2.5
msg10376 (view) Author: gh Date: 2010-03-21.14:17:54
The following patch updated the status of issue643 to be resolved:

* Resolve issue643 darcs send -o outputs remote repo email address 
Ignore-this: ba9b2380b2e327f45a2ae038fad2d3e3
msg11391 (view) Author: gh Date: 2010-06-13.17:16:46
The following patch updated the status of issue643 to be resolved-in-stable:

* Resolve issue643 darcs send -o outputs remote repo email address 
Ignore-this: ba9b2380b2e327f45a2ae038fad2d3e3
History
Date User Action Args
2008-02-04 00:23:53btcoburncreate
2008-02-04 15:23:59droundysetnosy: + darcs-devel
messages: + msg3092
2008-02-04 15:28:00droundysetpriority: bug -> wishlist
status: unknown -> wont-fix
messages: + msg3096
nosy: droundy, tommy, beschmi, kowey, darcs-devel, btcoburn
2008-02-04 22:49:09btcoburnsetstatus: wont-fix -> unknown
nosy: droundy, tommy, beschmi, kowey, darcs-devel, btcoburn
messages: + msg3117
title: darcs get does not get email pref -> "darcs send -o" should announce destination email
2008-02-05 12:27:27btcoburnsetnosy: droundy, tommy, beschmi, kowey, darcs-devel, btcoburn
messages: + msg3119
title: "darcs send -o" should announce destination email -> darcs get does not get email pref
2008-02-07 05:03:31markstossetstatus: unknown -> deferred
nosy: + markstos
messages: + msg3176
title: darcs get does not get email pref -> wish: "darcs send -o" should output the repo repo e-mail address.
2009-08-06 17:53:14adminsetnosy: + jast, Serware, dmitry.kurochkin, zooko, dagit, mornfall, simon, thorkilnaur, - droundy, btcoburn
2009-08-06 20:56:30adminsetnosy: - beschmi
2009-08-10 22:13:16adminsetnosy: + btcoburn, - zooko, jast, dagit, Serware, mornfall
2009-08-25 18:04:39adminsetnosy: - simon
2009-08-27 13:54:18adminsetnosy: tommy, kowey, markstos, darcs-devel, thorkilnaur, btcoburn, dmitry.kurochkin
2009-08-27 14:45:40koweysetstatus: deferred -> needs-implementation
nosy: + jaredj
topic: + ProbablyEasy, UI
title: wish: "darcs send -o" should output the repo repo e-mail address. -> wish: "darcs send -o" should output the remote repo e-mail address.
2009-11-15 14:03:43davidmsettopic: + Target-2.4
2009-11-15 15:37:00koweylinkissue1686 superseder
2010-03-09 20:16:15koweysettopic: + Target-2.5, - Target-2.4
messages: + msg10144
2010-03-20 15:22:32ganeshlinkpatch190 issues
2010-03-21 14:18:00ghsetstatus: needs-implementation -> resolved
nosy: + gh
messages: + msg10376
2010-06-13 17:16:47ghsetstatus: resolved -> resolved-in-stable
messages: + msg11391
2010-06-15 20:52:09adminsetmilestone: 2.5.0
2010-06-15 20:59:47adminsettopic: - Target-2.5
2010-06-15 22:14:16adminsetstatus: resolved-in-stable -> resolved
2010-06-15 22:14:16adminsetresolvedin: 2.5.0