Re: Large Patch Error

2012-11-24 Thread Christian Hammond
Hi,

What version are you guys using? Some work has gone into the past few releases 
(the latest being last night's 1.6.14) to improve support for very large diffs.

There's hopefully some useful info in the reviewboard.log file that may shed 
more light on when it failed. But really, for a diff that size, you'll need 
1.6.14.

In newer versions, by default, diffs are rejected if larger than 1MB, so that 
would have to be turned off. However, I can't guarantee a 4MB diff will even 
work well.

My best advice would be to try to split it up. It'll be easier on the 
reviewers, too.

Christian


On Nov 24, 2012, at 14:52, Angus Turner  wrote:

> Over at the ASF in the wave project we're in the process of updating our 
> license headers for release and this has generated a large (~4.3mb) patch 
> that we are trying to upload to reviewboard. Everytime this is tried we get 
> this error message: http://d.pr/i/mZtl I've tried both on safari and chrome 
> for mac. 
> 
> Is there a setting infra needs to modify to let this through or is it 
> something on the reviewboard end?
> -- 
> Want to help the Review Board project? Donate today at 
> http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~--~~~~--~~--~--~---
> To unsubscribe from this group, send email to 
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at 
> http://groups.google.com/group/reviewboard?hl=en
>  
>  

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en




Large Patch Error

2012-11-24 Thread Angus Turner
Over at the ASF in the wave project we're in the process of updating our 
license headers for release and this has generated a large (~4.3mb) patch 
that we are trying to upload to reviewboard. Everytime this is tried we get 
this error message: http://d.pr/i/mZtl I've tried both on safari and chrome 
for mac. 

Is there a setting infra needs to modify to let this through or is it 
something on the reviewboard end?

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en