[pulseaudio-discuss] [PATCH] backend-native: rebase of patch to support HSP HeadeSet profile

Tanu Kaskinen tanuk at iki.fi
Tue Feb 14 11:08:50 UTC 2017


On Sat, 2017-02-04 at 20:47 +0100, Georg Chini wrote:
> Subject: [PATCH] backend-native: rebase of patch to support HSP HeadeSet profile

Usually it's not interesting that a patch was rebased. In this case
it's good to mention it in the commit message, since the one who did
the rebasing is different from the original author of the patch, but
the patch heading is not the right place for that information.

You seem to have replaced Wim with yourself as the main patch author.
Was the rebasing effort large enough to justify that?

Also, there's a typo: "HeadeSet"

> This is a rebase of Wim Taymans patch to support the HSP headset role that has
> somehow been forgotten. Original patch can be found at
> https://lists.freedesktop.org/archives/pulseaudio-discuss/2015-February/023242.html

This is how it happened: Arun reviewed the patch, but provided the
feedback in IRC, so there was no permanent record of the review. Since
the patch was already reviewed, nobody else reviewed it later, but no
new version was submitted either.

I can review this new submission, but first I want to be able to apply
the patch locally. It doesn't seem to apply cleanly on master. Can you
clarify which branch the patch is expected to be applied on?

-- 
Tanu

https://www.patreon.com/tanuk


More information about the pulseaudio-discuss mailing list