rbt patch "file not found"

Daniel Kuecker

unread,

Sep 22, 2016, 9:50:04 AM9/22/16

to reviewboard

Running "rbt patch <requestnum>" command results in a file not found error for every file in the request. I verified the files are synced to my local drive. Any ideas?

Command line info (path info removed for anonymity):

C:\P4>rbt patch 127325
Patch is being applied from request 127325 with diff revision 1.
Failed to execute command: [u'patch', u'-i', u'c:\\users\\*******\\appdata\\local\\temp\\tmpjq4zbu']
can't find file to patch at input line 3
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--------------------------
|--- //****/***/setupscripts/trunk/16.0/source/************/_common_impl_details.rb   //****/***/setupscripts/trunk/16.0/source/************/_common_impl_details.rb#6
|+++ //****/***/setupscripts/trunk/16.0/source/************/_common_impl_details.rb   2016-02-18 09:21:42
--------------------------
File to patch:
Skip this patch? [y]
Skipping patch.
1 out of 1 hunk ignored
missing header for unified diff at line 17 of patch
can't find file to patch at input line 17
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--------------------------
|/setupscripts/trunk/16.0/source/************/_setup_common_impl_details.rb   //****/***/setupscripts/trunk/16.0/source/************/_setup_common_impl_details.rb#2
|+++ //****/***/setupscripts/trunk/16.0/source/************/_setup_common_impl_details.rb     2016-02-18 09:21:43
--------------------------
File to patch:
Skip this patch? [y]
Skipping patch.
1 out of 1 hunk ignored
missing header for unified diff at line 27 of patch
can't find file to patch at input line 27
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--------------------------
|/***/setupscripts/trunk/16.0/source/************/_setup_impl_details.rb      //****/***/setupscripts/trunk/16.0/source/************/_setup_impl_details.rb#2
|+++ //****/***/setupscripts/trunk/16.0/source/************/_setup_impl_details.rb    2016-02-18 09:21:43
--------------------------
File to patch:
Skip this patch? [y]
Skipping patch.
1 out of 1 hunk ignored

Daniel Kuecker

unread,

Sep 22, 2016, 9:55:37 AM9/22/16

to reviewboard

Oops, forgot to mention I am running this on Windows 10.

RBTools: 0.7.6

RB: tried on versions 1.7.27 and 2.5.6.1

Christian Hammond

unread,

Sep 23, 2016, 1:06:39 AM9/23/16

to revie...@googlegroups.com

What type of repository is this, and how was the diff generated/uploaded?

Christian

--
Christian Hammond

President/CEO of Beanbag

Makers of Review Board

--
Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
---
You received this message because you are subscribed to the Google Groups "reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Daniel Kuecker

unread,

Sep 23, 2016, 7:22:27 AM9/23/16

to revie...@googlegroups.com

Perforce repo and the diff was generated with and older version of RBTools (post-review era.).

To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard...@googlegroups.com.

For more options, visit https://groups.google.com/d/optout.

--
Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
---
You received this message because you are subscribed to the Google Groups "reviewboard" group.

To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard...@googlegroups.com.

Christian Hammond

unread,

Sep 23, 2016, 9:30:25 AM9/23/16

to revie...@googlegroups.com

rbt patch hasn't had much work done for Perforce. It should work, but it doesn't map Perforce depot paths to local file paths. You may need to figure out what -p parameter you need to use (in order to strip the leading part of the depot path).

Christian

--
Christian Hammond

President/CEO of Beanbag

Makers of Review Board

To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscribe@googlegroups.com.

For more options, visit https://groups.google.com/d/optout.

--
Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
---
You received this message because you are subscribed to the Google Groups "reviewboard" group.

To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscribe@googlegroups.com.

For more options, visit https://groups.google.com/d/optout.

--
Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
---
You received this message because you are subscribed to the Google Groups "reviewboard" group.

To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscribe@googlegroups.com.

Reply all

Reply to author

Forward