Add exclude files support to the Perforce SCMClient.
Review Request #6376 — Created Sept. 26, 2014 and submitted
The Perforce SCMClient now supports the -X flag for diff and post
operations. The -X parameter specifies a pattern to match against.
To specify a pattern against a depot, use-X //depot/...
and to
specify a relative (to the current working directory) pattern, use
-X pattern
. A pattern that beings with a single slash is
interpreted as an absolute path.
Generated diffs via
rbt diff -X
that excluded files. This was tested
using numbered changesets (both submitted and pending) and the default
changeset (i.e., open files)
- Change Summary:
-
Update documentation for rbtools
- Commit:
-
251063dab968bfb92fd1517b5698277e43a2cfe591118fb7312e9be3dd8242da08c89ce5100c2197
![reviewbot](https://reviews.reviewboard.org/static/ext/reviewbotext.extension.ReviewBotExtension/images/reviewbot.png)
-
Tool: Pyflakes Processed Files: rbtools/clients/perforce.py Ignored Files: docs/rbtools/rbt/commands/post.rst docs/rbtools/rbt/commands/diff.rst Tool: PEP8 Style Checker Processed Files: rbtools/clients/perforce.py Ignored Files: docs/rbtools/rbt/commands/post.rst docs/rbtools/rbt/commands/diff.rst
-
Would it be possible to have it so that if an exclude pattern starts with "//", treat it as a depot path, and if it doesn't, treat it as a local (relative) path?
- Change Summary:
-
Allow depot patterns and relative patterns.
- Description:
-
The Perforce SCMClient now supports the -X flag for diff and post
~ operations. The -X parameter specifies patterns that match against ~ the depot path, excluding the leading '//'. ~ operations. The -X parameter specifies a pattern to match against. ~ To specify a pattern against a depot, use -X //depot/...
and to+ specify a relative (to the current working directory) pattern, use + -X pattern
. A pattern that beings with a single slash is+ interpreted as an absolute path. - Commit:
-
91118fb7312e9be3dd8242da08c89ce5100c219728b25935ae65e921e361412e5f8b330aa9800a44
![reviewbot](https://reviews.reviewboard.org/static/ext/reviewbotext.extension.ReviewBotExtension/images/reviewbot.png)
-
Tool: Pyflakes Processed Files: rbtools/clients/perforce.py Ignored Files: docs/rbtools/rbt/commands/post.rst docs/rbtools/rbt/commands/diff.rst Tool: PEP8 Style Checker Processed Files: rbtools/clients/perforce.py Ignored Files: docs/rbtools/rbt/commands/post.rst docs/rbtools/rbt/commands/diff.rst
-
-
- Change Summary:
-
PEP8
- Commit:
-
28b25935ae65e921e361412e5f8b330aa9800a44a9c2e1b9dc28f94d70bfd1a65c835693ae39b17e
![reviewbot](https://reviews.reviewboard.org/static/ext/reviewbotext.extension.ReviewBotExtension/images/reviewbot.png)
-
Tool: PEP8 Style Checker Processed Files: rbtools/clients/perforce.py Ignored Files: docs/rbtools/rbt/commands/post.rst docs/rbtools/rbt/commands/diff.rst Tool: Pyflakes Processed Files: rbtools/clients/perforce.py Ignored Files: docs/rbtools/rbt/commands/post.rst docs/rbtools/rbt/commands/diff.rst
![reviewbot](https://reviews.reviewboard.org/static/ext/reviewbotext.extension.ReviewBotExtension/images/reviewbot.png)
-
Tool: Pyflakes Processed Files: rbtools/clients/perforce.py Ignored Files: docs/rbtools/rbt/commands/post.rst docs/rbtools/rbt/commands/diff.rst Tool: PEP8 Style Checker Processed Files: rbtools/clients/perforce.py Ignored Files: docs/rbtools/rbt/commands/post.rst docs/rbtools/rbt/commands/diff.rst
![reviewbot](https://reviews.reviewboard.org/static/ext/reviewbotext.extension.ReviewBotExtension/images/reviewbot.png)
-
Tool: Pyflakes Processed Files: rbtools/clients/perforce.py Ignored Files: docs/rbtools/rbt/commands/post.rst docs/rbtools/rbt/commands/diff.rst Tool: PEP8 Style Checker Processed Files: rbtools/clients/perforce.py Ignored Files: docs/rbtools/rbt/commands/post.rst docs/rbtools/rbt/commands/diff.rst
- Change Summary:
-
Use os.path.abspath instead of building paths myself.
- Commit:
-
86b118c9be9db6562c0aac163c76bc3959b1616248f54414b2cfd2c46ab6059cb8daee20305d7174