- User Since
- Feb 8 2018, 2:14 AM (36 w, 3 d)
Aug 9 2018
Aug 8 2018
Aug 7 2018
Aug 5 2018
Aug 4 2018
@yuja Cool, sending the option C now, will try out option D and send a patch if something comes up.
Aug 3 2018
I also expect hg grep --all-files -r0+1 foo will show matches from both rev 0 and 1.
Suppose there are ten hits in 0 and the same 10 hits in 1, do you mean we print out all the 20 results, What purpose that would serve?
Keeping all ctx objects might use too much memory.
True I will change it to files[f] = True
I don't think we should omit files that were seen in earlier revisions
If I don't skip that would mean the same file in the same state being searched across all the revisions, and getting repetitive and redundant hits.
Jul 30 2018
@yuja can you review this one
Jul 25 2018
I created a new revision instead of updating the D3976, ignore this one.
Jul 13 2018
So what's the next step should be ?
Jul 11 2018
So what is the way around this, does this mean grep can't be changed at all
Also can't we update hglib to work accordingly.
Jul 10 2018
Jul 9 2018
Jul 7 2018
Jul 5 2018
Jul 3 2018
Jul 2 2018
I am sorry I accidentally pushed here :/
Jun 28 2018
Jun 21 2018
I need some help with the tests failing here.
Jun 19 2018
Jun 17 2018
@yuja Sorry I didn't run the tests after this edit .
Jun 16 2018
Jun 15 2018
Jun 13 2018
I am updating the tests
Jun 12 2018
Jun 11 2018
@yuja I am adding an --unmodified flag to change to change the mode to grep on the unmodified files as well.
What I am trying to do is ::
def fns_generator(): if --unmodified: for f in ctx: for f in ctx: if match(f): yield f else : for f in ctx.files(): for f in ctx: if match(f): yield f
Jun 5 2018
Can you explain why you did this ?
Jun 2 2018
May 30 2018
May 26 2018
Perhaps, we'll have to catch WdirUnsupported error to fall back to the filectx API.
May 24 2018
So I was working on it, but the way I tried to correct "wdir()" is by handling it as a special case, so introducing an "if" block again similar to this.
But you were asking to eliminate the if/else block altogether, I wonder if there is way ?
Apr 17 2018
fix grep -r 'wdir()'
Apr 3 2018
Mar 27 2018
@yuja Yeah, doing it right away, my bad. I should have based it on some revision which is already in the stable.
Mar 26 2018
Since it required some explanation I have written a post on it.
Mar 25 2018
@yuja Can you please clarify this a bit more?
Mar 24 2018
Mar 23 2018
Mar 10 2018
Mar 9 2018
Since the current code just overwrites message each time with the newer. What can be done to avoid losing the earlier message values?
@durin42 Should I make any changes?
Mar 7 2018
@pulkit Yeah, this rather a not so elegant hack. How can it be parsed in hg commit?
As the first message passed is overwritten by the second one at this step only. :/
@durin42 Even I thought of this construct before as it appears more elegant. But it is failing some tests!
Specifically, the error messages are changed in this one.
Mar 4 2018
@jeffpc That means two new lines. Shall I edit this patch to effect that?