Re: Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2013-01-10 Thread reviewboard


Comment #19 on issue 658 by gjun...@gmail.com: Error : The patch  
to 'filepath' didn't apply cleanly.

http://code.google.com/p/reviewboard/issues/detail?id=658

It's an existing file in CVS.
We experienced that the same file change sometime works, sometimes not.

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2013-01-09 Thread reviewboard


Comment #17 on issue 658 by gjun...@gmail.com: Error : The patch  
to 'filepath' didn't apply cleanly.

http://code.google.com/p/reviewboard/issues/detail?id=658

I attached the .rej file as below for information because I can not figure  
out the problem from the file.

Thanks!

Attachments:
tmpGdkjyO-new.rej  241 KB

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2013-01-09 Thread reviewboard


Comment #18 on issue 658 by trowb...@gmail.com: Error : The patch  
to 'filepath' didn't apply cleanly.

http://code.google.com/p/reviewboard/issues/detail?id=658

Is this a newly-added file?

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2013-01-08 Thread reviewboard


Comment #16 on issue 658 by trowb...@gmail.com: Error : The patch  
to 'filepath' didn't apply cleanly.

http://code.google.com/p/reviewboard/issues/detail?id=658

Getting rejects usually indicates some problem with how the diff was  
generated. Check the .rej file to see what failed.


--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2013-01-07 Thread reviewboard


Comment #15 on issue 658 by gjun...@gmail.com: Error : The patch  
to 'filepath' didn't apply cleanly.

http://code.google.com/p/reviewboard/issues/detail?id=658

I would strongly vote and appreciate a solution/suggestions on this issue:
Our environment is reviewboard 1.6.14 on Ubuntu 12.04 (64bits) w/ python  
2.7, SCM is CVS

We experienced quite a few this problems.
As DARSHAN pointed out the problem is intermittent. For a single file in a  
diff package sometimes works, sometimes not.


We tested on our old reviewboard (1.5.5), same error for the same diff  
package (and backend CVS repository)


Diff currently unavailable. Error: The patch to 'project_sam.xml' didn't  
apply cleanly. The temporary files have been left  
in '/tmp/reviewboard.CRgaUy' for debugging purposes. `patch` returned:  
patching file /tmp/reviewboard.CRgaUy/tmpGdkjyO Hunk #1 FAILED at 1. 1 out  
of 1 hunk FAILED -- saving rejects to file  
/tmp/reviewboard.CRgaUy/tmpGdkjyO-new.rej Details

Traceback (most recent call last):
   
File /usr/local/lib/python2.7/dist-packages/ReviewBoard-1.6.14-py2.7.egg/reviewboard/diffviewer/views.py,  
line 384, in view_diff_fragment

file = get_requested_diff_file()
   
File /usr/local/lib/python2.7/dist-packages/ReviewBoard-1.6.14-py2.7.egg/reviewboard/diffviewer/views.py,  
line 310, in get_requested_diff_file

populate_diff_chunks(files, highlighting)
   
File /usr/local/lib/python2.7/dist-packages/ReviewBoard-1.6.14-py2.7.egg/reviewboard/diffviewer/diffutils.py,  
line 1188, in populate_diff_chunks

large_data=True)
   
File /usr/local/lib/python2.7/dist-packages/Djblets-0.6.27-py2.7.egg/djblets/util/misc.py,  
line 156, in cache_memoize

data = lookup_callable()
   
File /usr/local/lib/python2.7/dist-packages/ReviewBoard-1.6.14-py2.7.egg/reviewboard/diffviewer/diffutils.py,  
line 1187, in lambda

enable_syntax_highlighting)),
   
File /usr/local/lib/python2.7/dist-packages/ReviewBoard-1.6.14-py2.7.egg/reviewboard/diffviewer/diffutils.py,  
line 595, in get_chunks

new = get_patched_file(old, filediff)
   
File /usr/local/lib/python2.7/dist-packages/ReviewBoard-1.6.14-py2.7.egg/reviewboard/diffviewer/diffutils.py,  
line 388, in get_patched_file

return patch(diff, buffer, filediff.dest_file)
   
File /usr/local/lib/python2.7/dist-packages/ReviewBoard-1.6.14-py2.7.egg/reviewboard/diffviewer/diffutils.py,  
line 247, in patch

(filename, tempdir, patch_output))
Exception: The patch to 'project_sam.xml' didn't apply cleanly. The  
temporary files have been left in '/tmp/reviewboard.CRgaUy' for debugging  
purposes.

`patch` returned: patching file /tmp/reviewboard.CRgaUy/tmpGdkjyO
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file  
/tmp/reviewboard.CRgaUy/tmpGdkjyO-new.rej


--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2010-10-12 Thread reviewboard


Comment #14 on issue 658 by darshanrn: Error : The patch to 'filepath'  
didn't apply cleanly.

http://code.google.com/p/reviewboard/issues/detail?id=658

I get the same problem Exception: The patch to 'file' didn't apply  
cleanly. This is a very intermittent issue. Earlier the diff was viewable  
but not anymore. Not sure what went wrong all of a sudden.


--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2009-09-02 Thread codesite-noreply


Comment #12 on issue 658 by sanjiv.raj: Error : The patch to 'filepath'  
didn't apply cleanly.
http://code.google.com/p/reviewboard/issues/detail?id=658

I also encountered this issue after a perforce to svn migration. Fixing the  
ident
(RCS) strings solved the issue.

The diff provided by svn had the perforce repo path in the ident string for  
the
original copy. However the local copy used by review board had SVN Keywords
substituted. As a result patch utility errored out

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~--~~~~--~~--~--~---



Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2009-06-23 Thread codesite-noreply


Comment #5 on issue 658 by chipx86: Error : The patch to 'filepath'  
didn't apply cleanly.
http://code.google.com/p/reviewboard/issues/detail?id=658

This usually indicates that the revision is wrong. How was the diff  
generated, and
what repository type?

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~--~~~~--~~--~--~---



Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2009-06-23 Thread codesite-noreply


Comment #6 on issue 658 by manjunath.shiva: Error : The patch  
to 'filepath' didn't apply cleanly.
http://code.google.com/p/reviewboard/issues/detail?id=658

diff is generated on UNIX and copied from screen to windows and that was the
problem. Now he tried did an ftp and it worked. With the Review Baord 1.0  
facing
some problem for new request in two places. I am opening a new ticket  
shortly.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~--~~~~--~~--~--~---



Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2009-06-23 Thread codesite-noreply


Comment #7 on issue 658 by chipx86: Error : The patch to 'filepath'  
didn't apply cleanly.
http://code.google.com/p/reviewboard/issues/detail?id=658

But how was the diff generated? What tool, parameters?

Copying from screen to windows will break things. I assume you mean  
copy/paste. That
won't necessarily get the right control characters (tabs, same type of  
newlines,
etc). You will need to transfer the file directly.

You also should look into using post-review, which will make life a lot  
easier.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~--~~~~--~~--~--~---



Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2009-06-23 Thread codesite-noreply


Comment #8 on issue 658 by manjunath.shiva: Error : The patch  
to 'filepath' didn't apply cleanly.
http://code.google.com/p/reviewboard/issues/detail?id=658

diff was generated using cvs diff -u amuoci.c  amuoci.diff on UNIX. Here  
use made a
mistake of copy/paste from UNIX shell to Windows. Now it worked after ftp.
Sure, I will take a look into post-review and propose to our team hear.  
Thanks for
the tip!

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~--~~~~--~~--~--~---



Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2009-06-23 Thread codesite-noreply


Comment #9 on issue 658 by manjunath.shiva: Error : The patch  
to 'filepath' didn't apply cleanly.
http://code.google.com/p/reviewboard/issues/detail?id=658

I tried with latest post-review version 0.8. The problem is we do have post-
review.exe so we need to use UWIN / MKS / Cigwin to run this script on  
Windows. It
would have been good if we have a post-review.exe stand alone application  
so user
can use it here. I have no problem of using without exe but others are not
comfortable.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~--~~~~--~~--~--~---



Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2009-06-23 Thread codesite-noreply


Comment #10 on issue 658 by chipx86: Error : The patch to 'filepath'  
didn't apply cleanly.
http://code.google.com/p/reviewboard/issues/detail?id=658

Ah yeah, that's a potential problem for now. However, I do plan to get a
post-review.exe into the next release.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~--~~~~--~~--~--~---



Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2009-03-27 Thread codesite-noreply


Comment #2 on issue 658 by miguel.rgonzalez: Error : The patch  
to 'filepath' didn't apply cleanly.
http://code.google.com/p/reviewboard/issues/detail?id=658

same issue here.
I get the same error when posting reviews for newly created file (nothing  
to diff
against)

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~--~~~~--~~--~--~---



Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2008-09-18 Thread codesite-noreply

Issue 658: Error : The patch to 'filepath' didn't apply cleanly.
http://code.google.com/p/reviewboard/issues/detail?id=658

New issue report by sudarsan:
*NOTE: Do not post confidential information in this bug report.*

What's the URL of the page containing the problem?

http://reviewboard.eng.vmware.com/r/43966/diff/#index_header

What steps will reproduce the problem?
1. I have a change that moves files from one directory to another (i.e.
integrate/delete)
2. generated diffs using p5 diff -u -c ...
3. Upload the diffs and try to view the diffs.

What is the expected output? What do you see instead?

Normal diffs - what I see is
http://reviewboard.eng.vmware.com/r/43966/diff/#index_header (paraphrased
below)

What operating system are you using? What browser?
Windows XP / Firefox 3.0.1


Please provide any additional information below.

The output is like the following:


The patch to 'filepath' didn't apply cleanly. The temporary files have
been left in '/tmp/reviewboard.U-4jPA' for debugging purposes. `patch`
returned: patching file /tmp/reviewboard.U-4jPA/tmpI632Lt patch: 
malformed patch at line 5:

Traceback (most recent call last):
   File /root/reviewboard/diffviewer/views.py, line 82, in view_diff
 files = get_diff_files(diffset, None, interdiffset, highlighting)
   File /root/reviewboard/diffviewer/diffutils.py, line 577, in  
get_diff_files
 enable_syntax_highlighting)
   File /root/reviewboard/diffviewer/diffutils.py, line 502, in  
generate_files
 lambda: get_chunks(filediff.diffset,
   File /root/reviewboard/djblets/util/misc.py, line 47, in cache_memoize
 data = lookup_callable()
   File /root/reviewboard/diffviewer/diffutils.py, line 505, in lambda
 enable_syntax_highlighting))
   File /root/reviewboard/diffviewer/diffutils.py, line 301, in get_chunks
 new = get_patched_file(old, filediff)
   File /root/reviewboard/diffviewer/diffutils.py, line 214, in
get_patched_file
 return patch(filediff.diff, buffer, filediff.dest_file)
   File /root/reviewboard/diffviewer/diffutils.py, line 112, in patch
 (filename, tempdir, patch_output))
Exception: The patch to 'filepath' didn't apply cleanly. The temporary
files have been left in '/tmp/reviewboard.U-4jPA' for debugging purposes.
`patch` returned: patching file /tmp/reviewboard.U-4jPA/tmpI632Lt
patch:  malformed patch at line 5:




Issue attributes:
Status: New
Owner: 
Labels: Type-Defect Priority-Medium

-- 
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~--~~~~--~~--~--~---