Re: [PATCH V3] ALSA: pcm: Fix couple of typos

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 25 Mar 2021 10:56:39 +0100,
Bhaskar Chowdhury wrote:
> 
> On 10:37 Thu 25 Mar 2021, Takashi Iwai wrote:
> >On Thu, 25 Mar 2021 10:06:09 +0100,
> >Bhaskar Chowdhury wrote:
> >>
> >> s/unconditonally/unconditionally/
> >> s/succesful/successful/
> >>
> >> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@xxxxxxxxx>
> >> ---
> >>  Changes from V2:
> >>  Takashi pointed out that the patch was not applicable due to some unwanted
> >>  stuff get into it. Resending it with the new patch creation.
> >
> >Hrm, still not applicable.  Can you apply the patch from your own post
> >via git-am in your side?
> >
> Here is what I do for this specific case :
> 
> ✔ ~/git-linux/linux-next [patch L|✔]
> 15:18 $ sed -i 's/unconditonally/unconditionally/' sound/core/pcm_native.c
> ✔ ~/git-linux/linux-next [patch L|✚ 1]
> 15:19 $ sed -i 's/succesful/successful/' sound/core/pcm_native.c
> ✔ ~/git-linux/linux-next [patch L|✚ 1]
> 15:19 $ git add .
> ✔ ~/git-linux/linux-next [patch L|●1]
> 15:19 $ git ci "Fix some patch error"
> [patch 88d5af187dbb] Fix some patch error
> 1 file changed, 2 insertions(+), 2 deletions(-)
> 
> 15:21 $ git_fetch_single_file.sh sound/core/pcm_native.c
> Looks alright!✔ ~/git-linux/linux-next [patch L|●1]
> 15:21 $ git add .
> ✔ ~/git-linux/linux-next [patch L|●1]
> 15:21 $ git ci "Bring for patch"
> [patch 352e1ce8dacf] Bring for patch
> 1 file changed, 2 insertions(+), 2 deletions(-)
> ✔ ~/git-linux/linux-next [patch L|✔]
> 15:22 $ git apply --verbose 0001-Made-patche-for-this.patch
> Checking patch sound/core/pcm_native.c...
> Applied patch sound/core/pcm_native.c cleanly.

I meant to try to apply the patch from mail fetched from the ML, not
the patch you made from your git tree.


Takashi



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux