Status: New
Owner:
Labels: Type-Defect Priority-Medium
New issue 840 by btsebtse: post-review can't diff correctly Perforce paths
with spaces?
http://code.google.com/p/reviewboard/issues/detail?id=840
*NOTE: Do not post confidential information in this bug report.*
What's the URL of the
Comment #1 on issue 840 by btsebtse: post-review can't diff correctly
Perforce paths with spaces?
http://code.google.com/p/reviewboard/issues/detail?id=840
Ahh. After looking at the post-review script I see the comments in function:
_depot_to_local()
# XXX: This breaks on filenames w
Updates:
Status: Confirmed
Labels: Component-Scripts
Comment #2 on issue 840 by trowbrds: post-review can't diff correctly
Perforce paths with spaces?
http://code.google.com/p/reviewboard/issues/detail?id=840
(No comment was entered for this change.)
--
You received this messa
Comment #3 on issue 840 by devli...@buzzardsoft.com: post-review can't diff
correctly Perforce paths with spaces?
http://code.google.com/p/reviewboard/issues/detail?id=840
I'm attaching a patch that I believe fixes the "spaces in paths" problem
with Perforce. At a high level, it passes
the
Comment #4 on issue 840 by sharanrao: post-review can't diff correctly
Perforce paths with spaces?
http://code.google.com/p/reviewboard/issues/detail?id=840
I was facing the same issue with Perforce on windows a couple of days back.
I applied
the patch and it worked nicely.
--
You received
Updates:
Status: Fixed
Owner: chipx86
Labels: -Priority-Medium -Component-Scripts Priority-High
Component-RBTools Milestone-Release1.0
Comment #5 on issue 840 by chipx86: post-review can't diff correctly
Perforce paths with spaces?
http://code.google.com/p/reviewboard/