Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
From:   Jonathan Corbet <corbet@lwn.net>
To:     Miguel Ojeda <ojeda@kernel.org>
Cc:     Garrett LeSage <garrett@lesage.us>,
        IFo Hancroft <contact@ifohancroft.com>,
        linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
        Miguel Ojeda <ojeda@kernel.org>
Subject: Re: [PATCH v1 1/2] docs: move Linux logo into a new `images` folder
In-Reply-To: <20220510172530.29704-1-ojeda@kernel.org>
References: <20220510172530.29704-1-ojeda@kernel.org>
Date:   Thu, 12 May 2022 09:37:35 -0600
Message-ID: <87r14y4v9c.fsf@meer.lwn.net>
MIME-Version: 1.0
Content-Type: text/plain
Precedence: bulk

Miguel Ojeda <ojeda@kernel.org> writes:

> Having assets in the top-level `Documentation` directory can make
> it harder to find the documents one needs, especially if we want
> to add more of them later on.
>
> Instead, create a new `images` folder inside it that is used
> to hold assets such as logos.
>
> Link: https://lore.kernel.org/lkml/8735hicoy7.fsf@meer.lwn.net/
> Suggested-by: Jonathan Corbet <corbet@lwn.net>
> Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
> ---
>  Documentation/{ => images}/COPYING-logo |   0
>  Documentation/{ => images}/logo.gif     | Bin
>  2 files changed, 0 insertions(+), 0 deletions(-)
>  rename Documentation/{ => images}/COPYING-logo (100%)
>  rename Documentation/{ => images}/logo.gif (100%)
>
> diff --git a/Documentation/COPYING-logo b/Documentation/images/COPYING-logo
> similarity index 100%
> rename from Documentation/COPYING-logo
> rename to Documentation/images/COPYING-logo
> diff --git a/Documentation/logo.gif b/Documentation/images/logo.gif
> similarity index 100%
> rename from Documentation/logo.gif
> rename to Documentation/images/logo.gif

This will break scripts/spdxcheck-test.sh - which somebody might
actually try to run one of these years.  So this patch really needs to
update that script to follow the move.

As far as I can tell, that's the only reference to logo.gif in the
entire tree.  It makes me wonder if we need it at all.  Digging through
the history suggests it was added in 2.1.15, but never really used for
anything.  It's only role would appear to be to serve as testing
material for the SPDX checker..:)

Thanks,

jon