What version are you running?
Reviewboard v4.0.2 (official Docker container),
rbtools2.0.1 (Python 3.9.6)
What steps will reproduce the problem?
- Add a folder with files you want to exclude to your repo. Let's call it
exc. Add a file to it,
- Add the following settings to .reviewboardrc:EXCLUDE_PATTERNS = [ r'/exc/*' ]
- Commit everything, push to tracking branch (say, master).
- On a different branch, add a line to
exc/file.txt. Also, make another change that shouldn't be ignored: add a new file called
another.txtto the root.
- Start a review:
rbt post origin/master..HEAD
- Open the diff link presented.
What is the expected output? What do you see instead?
I expect the excluded files not to show up in the diff.
If you select the whole diff, first commit to last (which is the default), the excluded files are shown.
If you narrow down to single commits, none of the commits contain the excluded files (which is OK)
What operating system are you using? What browser?
Arch Linux (rolling release, updated today). Firefox 91.0.
Same thing happens on Windows, Firefox 90.0.2.
Both systems have
rbtools2.0.1, running Python 3.9.6
I figured out where the error is and posted a fix: https://reviews.reviewboard.org/r/11794/
Is this not an issue for everybody else? Should I try harder to explain the issue? Post screenshots?
I posted a simple fix, this could be reviewed and merged in 10 minutes.
We did see your patch, we've just been very busy and haven't been able to get around to it yet.
OK, I understand that. Thanks for the acknowledgement.
Fixed in release-2.0.x (97d6a9a). This will ship in RBTools 2.0.2. Thanks!
- New + Fixed