linux-kernel - Re: [PATCH] docs: sphinx: add missing SPDX tags

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20250625221156.6f671e0f@foz.lan>
Date: Wed, 25 Jun 2025 22:11:56 +0200
From: Mauro Carvalho Chehab <mchehab+huawei@...nel.org>
To: Jonathan Corbet <corbet@....net>
Cc: Linux Doc Mailing List <linux-doc@...r.kernel.org>, Akira Yokosawa
 <akiyks@...il.com>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] docs: sphinx: add missing SPDX tags

Em Wed, 25 Jun 2025 12:41:01 -0600
Jonathan Corbet <corbet@....net> escreveu:

> Mauro Carvalho Chehab <mchehab+huawei@...nel.org> writes:
> 
> > Several Sphinx extensions and tools are missing SPDX tags.
> > Add them.
> >
> > Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@...nel.org>
> > ---
> >  Documentation/sphinx/cdomain.py           | 1 +
> >  Documentation/sphinx/kernel_include.py    | 1 +
> >  Documentation/sphinx/kerneldoc.py         | 1 +
> >  Documentation/sphinx/kfigure.py           | 1 +
> >  Documentation/sphinx/load_config.py       | 1 +
> >  Documentation/sphinx/min_requirements.txt | 1 +
> >  Documentation/sphinx/parse-headers.pl     | 5 ++++-
> >  Documentation/sphinx/requirements.txt     | 1 +
> >  Documentation/sphinx/rstFlatTable.py      | 1 +
> >  9 files changed, 12 insertions(+), 1 deletion(-)  
> 
> I've applied this, but the patch led me to wonder...

Thanks!

> 
> > diff --git a/Documentation/sphinx/cdomain.py b/Documentation/sphinx/cdomain.py
> > index e8ea80d4324c..3dc285dc70f5 100644
> > --- a/Documentation/sphinx/cdomain.py
> > +++ b/Documentation/sphinx/cdomain.py
> > @@ -1,4 +1,5 @@
> >  # -*- coding: utf-8; mode: python -*-  
> 
> Do we really need these lines?  UTF8 should be the default almost
> anywhere, and Emacs understands that a .py file will require Python
> mode...

Heh, I had the same filling when I saw that. I almost dropped ;-)

perhaps we could run some linters on it and so some cleanups,
inclusing the removal of those, placing instead some copyright
notes on the top of it... well, there are already copyrights there
(and at the other Sphinx extensions). In this specific case:

"""
    cdomain
    ~~~~~~~

    Replacement for the sphinx c-domain.

    :copyright:  Copyright (C) 2016  Markus Heiser
    :license:    GPL Version 2, June 1991 see Linux/COPYING for details.

So, not sure about it. What do you think?

Thanks,
Mauro