[mythtv] [mythtv-commits] Ticket #10793: CommFlagger improvements
John P Poet
jppoet at gmail.com
Fri Feb 1 01:46:57 UTC 2013
On Sun, Jan 13, 2013 at 12:03 PM, <noreply at mythtv.org> wrote:
> #10793: CommFlagger improvements
> -----------------------------------+-----------------------------
> Reporter: markspieth | Owner: cpinkham
> Type: Patch - Feature | Status: new
> Priority: minor | Milestone: unknown
> Component: MythTV - Mythcommflag | Version: Unspecified
> Severity: medium | Resolution:
> Keywords: | Ticket locked: 0
> -----------------------------------+-----------------------------
>
> Comment (by bryan@…):
>
> Replying to [comment:8 Wayne McDougall <waynemcdougall@…>]:
> > ... Hope you find the feedback useful ...
>
> I very much appreciate the feedback. I went back through and cleaned up
> my patches. I've reduced it to 2 (they both overlap a little in
> ClassicCommDetector): a new SceneChangeDetector and a new LogoDetector.
> My other patch had made some more changes to the overall commflagging
> method that may have been a bad idea for non-US so I decided to abandon
> those changes. These 2 (scene change and logo detection) have both worked
> really well for me over the last 2 months.
Hi Bryan,
I recommend taking a look at a tool called quilt. It makes "layering"
patches very easy. For example, to clean up your last two patches so
they could be applied cleanly, I did:
$ patch -p1 < logo-detector.diff
$ quilt new scene-change
$ quilt fold -f -p1 < scene-change-detector.diff
Then I checked the reject file for anything obviously wrong. Then I
generated a new scene-change patch will applies cleaning after the
logo patch:
$ quilt diff > scene-change-detector-v1.diff
To then clear out the quilt stuff
$ quilt delete
You can layer multiple patches with quilt, you just use "quilt new
<somename>" whenever you want to add a new "layer".
There are ways to do the same thing in git, but I am more comfortable
using quilt.
John
More information about the mythtv-dev
mailing list