On Sun, 10 Dec 2017 16:48:26 -0500, Augie Fackler <r...@durin42.com> wrote:


On Dec 10, 2017, at 3:47 PM, Matt Harbison <mharbiso...@gmail.com> wrote:

# HG changeset patch
# User Matt Harbison <matt_harbi...@yahoo.com>
# Date 1512882971 18000
#      Sun Dec 10 00:16:11 2017 -0500
# Node ID ebc7a05d9389989fc2e81d30fa1c10dd01be9a76
# Parent  e1119050f15e2169ac766e6a69eeab70729e9e40
run-tests: accept '\' vs '/' path differences without '(glob)’

series queued, many thanks

[…]

It's probably unreasonable to submit a single patch that wipes out all of the (glob) instances that were only used to hide path differences, given the churn from other contributors. Since their presence isn't harming the tests, these
can be removed through attrition.

You could do it if you wanted with a skip-blame annotation as was done in d92dc725223b. Then we can pass the right filter to `hg annotate` and not see the noop changes. Not a requirement though, don’t feel obligated to do so.

I hadn't thought of blame- I was more worried about import conflicts. Does skip-blame somehow help the import tools?

Probably a rough approximation:

$ grep '(glob)' *.t | grep -v '[*?]' | wc -l
1125

Without the negative filter, it's 3216.

(If I knew the equivalent regex, it should be trivial to pipe it through sed, and see what blows up.)
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to