[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: patch labels in CG manual
From: |
David Kastrup |
Subject: |
Re: patch labels in CG manual |
Date: |
Mon, 06 Jun 2016 00:24:25 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1.50 (gnu/linux) |
Carl Sorensen <address@hidden> writes:
> James,
>
> On 5/20/16 10:15 AM, "James" <address@hidden> wrote:
>>
>>One thing to realise and I think people have forgotten is that the
>>Patchy-testing scripts (rather than Patchy-merge scripts) no longer work
>>as they were written to scrape the old Google tracker and don't work
>>with Allura.
>>
>>So Patch testing is 100% manually done - i.e. lots of clicking and
>>typing commands - and that includes me having to download the raw *diff
>>file from Rietveld via my browser and then running the commands, albeit
>>serially, in a CMD window completely unscripted. Then making sure I
>>manually clean my out of tree build and the tree in its current state
>>after a patch test.
>
> I certainly had forgotten this. I'm amazed that you've continued this
> long with manual patch application. You're an all-star!
>
> I will commit to helping get the Patchy-testing scripts working, so
> this manual procedure is no longer needed, or is at least simplified.
Having had quite a bit of refactoring done in a number of issues this
weekend, I am pretty sure that James would appreciate a return of the
automated testing procedures, so that he can just go through visual
checks of a whole batch when he has time.
I feel kind of bad for the amount of involved issues but they are
basically all independently testable and I would want to avoid queueing
them as one bunch when they might have individual problems that are
masked when combined.
--
David Kastrup
- Re: patch labels in CG manual,
David Kastrup <=