Re: [PATCH v3] docs: replace a duplicated word and add missing SPDX-License tag

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

 



Hi,

Many thanks for your feedback and review.
I will keep them in mind when working on upcoming patches.


On Thu, 21 Aug 2025 at 19:58, Jonathan Corbet <corbet@xxxxxxx> wrote:
>
> Bartlomiej Kubik <kubik.bartlomiej@xxxxxxxxx> writes:
>
> > Fix documentation issues by removing a duplicated word and adding the
> > missing SPDX-License identifier.
> >
> > Signed-off-by: Bartlomiej Kubik <kubik.bartlomiej@xxxxxxxxx>
> > ---
> >
> > Changelog:
> >
> > Changes since v2:
> > - Replace “that” with “the.”
> >
> > Changes since v1:
> > - Add missing SPDX-License tag
> > - Remove duplicated word "that"
> >
> >  Documentation/admin-guide/kernel-parameters.rst | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
>
> A couple of minor notes:
>
> - It is good to indicate *which* file you are modifying; I've rewritten
>   the subject line to that end.
>
> - Adding SPDX headers is a bit fraught unless you are truly sure of the
>   license under which the file was contributed.  Given the nature of
>   this file and all of the people who have touched it, though, GPLv2 is
>   the only option that makes any sense.
>
> Applied, thanks.
>
> jon



-- 
Best regards
Bartłomiej Kubik





[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux