[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: cherry-picking something to -stable which might require other change
From: |
Daniel P . Berrangé |
Subject: |
Re: cherry-picking something to -stable which might require other changes |
Date: |
Tue, 12 Sep 2023 16:23:28 +0100 |
User-agent: |
Mutt/2.2.9 (2022-11-12) |
On Tue, Sep 12, 2023 at 10:00:46AM -0400, Stefan Hajnoczi wrote:
> When I backport patches into RHEL, the general process I follow is:
> 1. For context conflicts, just adjust the patch to resolve them.
> 2. For real dependencies, backport the dependencies, if possible.
> 3. If backporting the dependencies is not possible, think of a
> downstream-only solution. This should be rare.
>
> People make different backporting decisions (just like structuring
> patch series). It can be a matter of taste.
I tend to try to cherry-pick the dependancies in case (1) too
unless they are functionally invasive. Any time you manually
adjust a patch, you increase the likelihood that later cherry
picks will also require manual work. So I always favour a clean
cherry-pick until the point the functional risk becomes
unacceptable in the context of testing the change I'm pulling
back.
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|