Re: [OS-BUILD PATCHv4 2/4] Update CONFIG_SPEAKUP_SYNTH_DUMMY to remove extra lines.

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

 



On Tue, Oct 13, 2020 at 9:36 AM Jiri Benc <jbenc@xxxxxxxxxx> 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 <jbenc@xxxxxxxxxx>


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 -- kernel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to kernel-leave@xxxxxxxxxxxxxxxxxxxxxxx
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/kernel@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux