On Tue, Oct 13, 2020 at 9:36 AM Jiri Benc <[email protected]> wrote:
>
> On Tue, 13 Oct 2020 14:19:35 -0000, GitLab Bridge on behalf of jmflinuxtx 
> wrote:
> > diff --git a/redhat/configs/common/generic/CONFIG_SPEAKUP_SYNTH_DUMMY 
> > b/redhat/configs/common/generic/CONFIG_SPEAKUP_SYNTH_DUMMY
> > index fc91de6e3981..5df68ceb1363 100644
> > --- a/redhat/configs/common/generic/CONFIG_SPEAKUP_SYNTH_DUMMY
> > +++ b/redhat/configs/common/generic/CONFIG_SPEAKUP_SYNTH_DUMMY
> > @@ -1,2 +1 @@
> > -# CONFIG_SPEAKUP_SYNTH_DUMMY is not set
> > -
> > +# CONFIG_SPEAKUP_SYNTH_DUMMY is not set
> > \ No newline at end of file
>
> Nacked-by: Jiri Benc <[email protected]>


The extra newline was a mistake that I saw when viewing the MR on
gitlab, so I thought I would be quick and edit it using their web UI,
but it deleted the newline at the end of the file. I added that
newline back, and the V4 patch is now correct. So clearly, you are not
nacking the patch for that reason.  If you want to nack the series,
that's fine, but some context would be worthwhile there.

Justin
_______________________________________________
kernel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]

Reply via email to