Ok all, I tried contacting infra with no luck, i'm not sure if my message didn't get through or what. I think it might be worth just creating a new review request so i'll try that next. Thanks Angus Turner angusisf...@gmail.com
On Mon, Nov 19, 2012 at 10:22 AM, Angus Turner <angusisf...@gmail.com>wrote: > I'll leave it for 10 minutes or so and see what happens. > Thanks > Angus Turner > angusisf...@gmail.com > > > > On Mon, Nov 19, 2012 at 10:19 AM, Ali Lown <a...@lown.me.uk> wrote: > >> Timeout related? >> >> Might it be worth mentioning to infrastructure? >> >> Ali >> >> On 18 November 2012 23:17, Angus Turner <angusisf...@gmail.com> wrote: >> > It's getting there. It seems a 4.4mb patch breaks the review system >> though >> > :) >> > Thanks >> > Angus Turner >> > angusisf...@gmail.com >> > >> > >> > >> > On Mon, Nov 19, 2012 at 10:04 AM, Angus Turner <angusisf...@gmail.com> >> > wrote: >> >> >> >> Great I'll upload it now. >> >> Thanks >> >> Angus Turner >> >> angusisf...@gmail.com >> >> >> >> >> >> >> >> On Mon, Nov 19, 2012 at 10:02 AM, Ali Lown <a...@lown.me.uk> wrote: >> >>> >> >>> > It might be easier >> >>> > to download the patch apply that and create a script that just adds >> the >> >>> > extra * needed everywhere. >> >>> >> >>> @Angus, do you want to submit the new patch onto the existing review >> >>> rather than me making a new review request for the same problem? >> >>> If so, find it attached. >> >>> Note: the perl script I wrote uses the header formatting specified in >> >>> the Apache document[0], which wraps the text in a different location >> >>> (and indents the URL consistently) from your script, so it is quite a >> >>> long patch. >> >>> I don't think it broke anything, but it is quite possible it did, feel >> >>> free to check... :) >> >>> >> >>> Ali >> >>> [0]: https://www.apache.org/legal/src-headers.html >> >> >> >> >> > >> > >
