darcs

Issue 1404 darcs replace --verbose should indicate the number of occurrences it replaced

Title darcs replace --verbose should indicate the number of occurrences it replaced
Priority wishlist Status given-up
Milestone Resolved in
Superseder in patch/hunk selection, $ is for see in external tool, option lists overhaul: --foo={yes,no} instead of --{no-}foo
View: 1603, 1557
Nosy List darcs-devel, dmitry.kurochkin, galbolle, kowey, thorkilnaur
Assigned To
Topics Provisional

Created on 2009-03-23.12:19:59 by galbolle, last changed 2017-07-30.23:17:10 by gh.

Messages
msg7498 (view) Author: galbolle Date: 2009-03-23.12:19:55
darcs replace -v -v should also print the affected lines.

Maybe hitting "v" in interactive selection when selecting a patch containing a
replace should do that too.
msg7649 (view) Author: kowey Date: 2009-04-09.15:59:15
We used to track multiple uses of the --verbose flag, but now we use --debug instead
msg7665 (view) Author: galbolle Date: 2009-04-10.08:25:40
I don't think this would belong to --debug: it's not internal output, it's for
the user's comfort.
msg8702 (view) Author: kowey Date: 2009-09-05.06:37:53
I don't think this particularly needs to be -v -v (--verbose will do).  

I am slightly nervous about this because I think it may mislead people into
forming an inaccurate mental model of what darcs replace does (the number of
occurrences replaced depends on the context), but I guess it's fine if we only
display this count in --verbose.

I think the best would be for somebody to submit a patch first and then argue
for it later.
msg8703 (view) Author: kowey Date: 2009-09-05.06:40:23
Oh, I misread this: Florent was saying
* -v   print count
* -v v print affected lines

I'd say the second part of this request depends on issue1557, which would give
us the ability to recognise -v 2
msg8771 (view) Author: galbolle Date: 2009-09-09.15:01:55
The 'v' during interactive selection is a particular case of issue 1603.
History
Date User Action Args
2009-03-23 12:19:59galbollecreate
2009-04-09 15:58:45koweysetstatus: unread -> needs-reproduction
nosy: kowey, simon, thorkilnaur, dmitry.kurochkin, galbolle
2009-04-09 15:59:18koweysetnosy: kowey, simon, thorkilnaur, dmitry.kurochkin, galbolle
messages: + msg7649
2009-04-10 08:25:42galbollesetnosy: kowey, simon, thorkilnaur, dmitry.kurochkin, galbolle
messages: + msg7665
2009-08-25 17:42:32adminsetnosy: + darcs-devel, - simon
2009-08-27 14:16:54adminsetnosy: kowey, darcs-devel, thorkilnaur, dmitry.kurochkin, galbolle
2009-09-05 06:37:57koweysetstatus: needs-reproduction -> needs-implementation
priority: feature -> wishlist
title: darcs replace --verbose should indicate the number of occurences it replaced -> darcs replace --verbose should indicate the number of occurrences it replaced
nosy: kowey, darcs-devel, thorkilnaur, dmitry.kurochkin, galbolle
messages: + msg8702
topic: + Provisional
2009-09-05 06:40:25koweysetnosy: kowey, darcs-devel, thorkilnaur, dmitry.kurochkin, galbolle
superseder: + option lists overhaul: --foo={yes,no} instead of --{no-}foo
messages: + msg8703
2009-09-09 15:01:57galbollesetnosy: kowey, darcs-devel, thorkilnaur, dmitry.kurochkin, galbolle
superseder: + in patch/hunk selection, $ is for see in external tool
messages: + msg8771
2017-07-30 23:17:10ghsetstatus: needs-implementation -> given-up