darcs

Patch 382 darcs show tags --repo (instead of --rep... (and 1 more)

Title darcs show tags --repo (instead of --rep... (and 1 more)
Superseder Nosy List ganesh, kowey, mornfall
Related Issues
Status accepted Assigned To
Milestone

Created on 2010-09-02.10:57:09 by kowey, last changed 2011-05-10.20:36:38 by darcswatch. Tracked on DarcsWatch.

Files
File name Status Uploaded Type Edit Remove
darcs-show-tags-__repo-_instead-of-__repodir__.dpatch kowey, 2010-09-02.10:57:09 text/x-darcs-patch
darcs-show-tags-__repo-_instead-of-__repodir__.dpatch kowey, 2010-10-19.21:55:05 text/x-darcs-patch
unnamed kowey, 2010-09-02.10:57:09
unnamed kowey, 2010-10-19.21:55:05
See mailing list archives for discussion on individual patches.
Messages
msg12407 (view) Author: kowey Date: 2010-09-02.10:57:09
2 patches for repository http://darcs.net:

Thu Sep  2 11:46:17 BST 2010  Eric Kow <kowey@darcs.net>
  * darcs show tags --repo (instead of --repodir).

Thu Sep  2 11:58:08 BST 2010  Eric Kow <kowey@darcs.net>
  * Faster setup for buildbot-try.
  We start from a tag that both repositories have in common.


___________________________________________________________
This email has been scanned by MessageLabs' Email Security
System on behalf of the University of Brighton.
For more information see http://www.brighton.ac.uk/is/spam/
___________________________________________________________
Attachments
msg12409 (view) Author: kowey Date: 2010-09-02.11:04:30
What do you think, Petr?
msg12411 (view) Author: kowey Date: 2010-09-02.12:13:51
I'll just mention one flaw with this plan off the top of my head: it 
assumes that if you have a tag with a given name in both repositories, 
that the two actually are the same tag.  I wonder if there is a more 
reliable way to pick up the tags that would give us the patch info too, 
maybe some trick with changes --xml
msg12662 (view) Author: ganesh Date: 2010-10-05.20:56:09
Note that the buildbot-try patch is not wanted any more, just the --repo 
patch.
msg12675 (view) Author: ganesh Date: 2010-10-09.16:11:05
What does this patch actually do?

Before the patch, I could do

darcs show tags --repodir=foo

after the path, I can't do

darcs show tags --repo=foo

Also, the help text implies I should also be able to do

darcs show tags --repo=http://darcs.net

but I can't.
msg12758 (view) Author: kowey Date: 2010-10-17.15:23:41
On further testing, it looks like I had managed to fool myself into
thinking this worked because I was running darcs show tags --repo
http://darcs.net and (I think) I must have coincidentally happened to be
in a darcs darcs repository at the time...

Sigh!

I'll try to figure out what went wrong

(evidence that patches should really come with proper tests...)
msg12770 (view) Author: kowey Date: 2010-10-19.21:55:05
2 patches for repository http://darcs.net:

Correcting my blunder in patch382 (self-screened).

Thu Sep  2 11:46:17 BST 2010  Eric Kow <kowey@darcs.net>
  * darcs show tags --repo (instead of --repodir).

Mon Oct 18 08:00:06 BST 2010  Eric Kow <kowey@darcs.net>
  * Fix patch382: darcs show tags --repo.
  I had fooled myself by using http://darcs.net for informal
  testing when I also happened to darcs darcs repository.
Attachments
msg12774 (view) Author: kowey Date: 2010-10-19.22:50:46
For some reason, my followup has not been auto-applied to screened
(despite my signing it).

Anyway, it needs review.  I'll probably just push it directly
msg12842 (view) Author: darcswatch Date: 2010-10-27.17:50:29
This patch bundle (with 2 patches) was just applied to the repository http://darcs.net/.
This message was brought to you by DarcsWatch
http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-a6e07b49410ac8ba00600c8b30c45b6e57535bcc
msg14091 (view) Author: darcswatch Date: 2011-05-10.18:06:04
This patch bundle (with 2 patches) was just applied to the repository http://darcs.net/reviewed.
This message was brought to you by DarcsWatch
http://darcswatch.nomeata.de/repo_http:__darcs.net_reviewed.html#bundle-a6e07b49410ac8ba00600c8b30c45b6e57535bcc
History
Date User Action Args
2010-09-02 10:57:09koweycreate
2010-09-02 10:58:38darcswatchsetdarcswatchurl: http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-ece25ec3f254c1fcb190572b0df697642fc7d73d
2010-09-02 11:04:30koweysetassignedto: mornfall
messages: + msg12409
nosy: + mornfall
2010-09-02 12:13:51koweysetmessages: + msg12411
2010-09-15 13:59:24koweysetstatus: needs-review -> needs-screening
2010-10-05 20:56:09ganeshsetmessages: + msg12662
2010-10-05 20:56:13ganeshsetstatus: needs-screening -> needs-review
2010-10-09 16:11:05ganeshsetstatus: needs-review -> in-discussion
messages: + msg12675
2010-10-09 16:11:14ganeshsetnosy: + ganesh
assignedto: mornfall -> ganesh
2010-10-17 15:23:41koweysetstatus: in-discussion -> followup-in-progress
assignedto: ganesh -> kowey
messages: + msg12758
2010-10-19 21:55:06koweysetfiles: + darcs-show-tags-__repo-_instead-of-__repodir__.dpatch, unnamed
messages: + msg12770
2010-10-19 21:55:41darcswatchsetdarcswatchurl: http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-ece25ec3f254c1fcb190572b0df697642fc7d73d -> http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-a6e07b49410ac8ba00600c8b30c45b6e57535bcc
2010-10-19 22:24:58koweysetstatus: followup-in-progress -> needs-review
2010-10-19 22:50:46koweysetmessages: + msg12774
2010-10-20 17:45:21koweysetassignedto: kowey ->
2010-10-27 17:50:29darcswatchsetstatus: needs-review -> accepted
messages: + msg12842
2011-05-10 18:06:04darcswatchsetmessages: + msg14091
2011-05-10 20:36:38darcswatchsetdarcswatchurl: http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-a6e07b49410ac8ba00600c8b30c45b6e57535bcc -> http://darcswatch.nomeata.de/repo_http:__darcs.net_reviewed.html#bundle-ece25ec3f254c1fcb190572b0df697642fc7d73d