2023-05-22 01:15:48

by Bagas Sanjaya

[permalink] [raw]
Subject: [PATCH v2 0/2] SPDX conversion from UDF

This small SPDX conversion series targets UDF file system, which is
splitted from v2 of my SPDX conversion series that is triggered by
Didi's GPL full name fixes [1]. It is done to ease review.

All boilerplates in fs/udf/ is converted, except fs/udf/ecma_167.h.
The latter file apparently looks like 2-clause BSD Source-Code
license, yet the second clause is from third clause of 3-Clause BSD.
This custom license can't be expressed satisfiably in SPDX license
identifier, hence the file doesn't get converted.

This series is based on mm-nonmm-unstable branch.

Changes since v1 [2]:
* Correct SPDX tag for LGPL (correct spdxcheck warning)

[1]: https://lore.kernel.org/linux-spdx/[email protected]/
[2]: https://lore.kernel.org/linux-mm/[email protected]/

Bagas Sanjaya (2):
fs: udf: Replace GPL 2.0 boilerplate license notice with SPDX
identifier
fs: udf: udftime: Replace LGPL boilerplate with SPDX identifier

fs/udf/balloc.c | 6 +-----
fs/udf/dir.c | 6 +-----
fs/udf/directory.c | 6 +-----
fs/udf/file.c | 6 +-----
fs/udf/ialloc.c | 6 +-----
fs/udf/inode.c | 6 +-----
fs/udf/lowlevel.c | 6 +-----
fs/udf/misc.c | 6 +-----
fs/udf/namei.c | 6 +-----
fs/udf/partition.c | 6 +-----
fs/udf/super.c | 6 +-----
fs/udf/symlink.c | 6 +-----
fs/udf/truncate.c | 6 +-----
fs/udf/udftime.c | 18 ++----------------
fs/udf/unicode.c | 6 +-----
15 files changed, 16 insertions(+), 86 deletions(-)


base-commit: 7e61b33831bc7680b24bc04af9ed9c1553dac406

Range-diff against v1:

1: 442194d17ed043 = 1: 30fb64a215be1c fs: udf: Replace GPL 2.0 boilerplate license notice with SPDX identifier
2: ccb407446ab324 ! 2: f7cfeaa5cec879 fs: udf: udftime: Replace LGPL boilerplate with SPDX identifier
@@ Commit message

## fs/udf/udftime.c ##
@@
-+// SPDX-License-Identifier: LGPL-2.0-or-later
++// SPDX-License-Identifier: LGPL-2.0+
/* Copyright (C) 1993, 1994, 1995, 1996, 1997 Free Software Foundation, Inc.
This file is part of the GNU C Library.
- Contributed by Paul Eggert ([email protected]).

--
An old man doll... just what I always wanted! - Clara



2023-05-22 10:19:53

by Jan Kara

[permalink] [raw]
Subject: Re: [PATCH v2 0/2] SPDX conversion from UDF

On Mon 22-05-23 07:54:33, Bagas Sanjaya wrote:
> This small SPDX conversion series targets UDF file system, which is
> splitted from v2 of my SPDX conversion series that is triggered by
> Didi's GPL full name fixes [1]. It is done to ease review.
>
> All boilerplates in fs/udf/ is converted, except fs/udf/ecma_167.h.
> The latter file apparently looks like 2-clause BSD Source-Code
> license, yet the second clause is from third clause of 3-Clause BSD.
> This custom license can't be expressed satisfiably in SPDX license
> identifier, hence the file doesn't get converted.
>
> This series is based on mm-nonmm-unstable branch.
>
> Changes since v1 [2]:
> * Correct SPDX tag for LGPL (correct spdxcheck warning)
>
> [1]: https://lore.kernel.org/linux-spdx/[email protected]/
> [2]: https://lore.kernel.org/linux-mm/[email protected]/
>
> Bagas Sanjaya (2):
> fs: udf: Replace GPL 2.0 boilerplate license notice with SPDX
> identifier
> fs: udf: udftime: Replace LGPL boilerplate with SPDX identifier

The patches look good to me. So unless someone objects in a few days, I'll
queue them into my tree. Thanks!

Honza

>
> fs/udf/balloc.c | 6 +-----
> fs/udf/dir.c | 6 +-----
> fs/udf/directory.c | 6 +-----
> fs/udf/file.c | 6 +-----
> fs/udf/ialloc.c | 6 +-----
> fs/udf/inode.c | 6 +-----
> fs/udf/lowlevel.c | 6 +-----
> fs/udf/misc.c | 6 +-----
> fs/udf/namei.c | 6 +-----
> fs/udf/partition.c | 6 +-----
> fs/udf/super.c | 6 +-----
> fs/udf/symlink.c | 6 +-----
> fs/udf/truncate.c | 6 +-----
> fs/udf/udftime.c | 18 ++----------------
> fs/udf/unicode.c | 6 +-----
> 15 files changed, 16 insertions(+), 86 deletions(-)
>
>
> base-commit: 7e61b33831bc7680b24bc04af9ed9c1553dac406
>
> Range-diff against v1:
>
> 1: 442194d17ed043 = 1: 30fb64a215be1c fs: udf: Replace GPL 2.0 boilerplate license notice with SPDX identifier
> 2: ccb407446ab324 ! 2: f7cfeaa5cec879 fs: udf: udftime: Replace LGPL boilerplate with SPDX identifier
> @@ Commit message
>
> ## fs/udf/udftime.c ##
> @@
> -+// SPDX-License-Identifier: LGPL-2.0-or-later
> ++// SPDX-License-Identifier: LGPL-2.0+
> /* Copyright (C) 1993, 1994, 1995, 1996, 1997 Free Software Foundation, Inc.
> This file is part of the GNU C Library.
> - Contributed by Paul Eggert ([email protected]).
>
> --
> An old man doll... just what I always wanted! - Clara
>
--
Jan Kara <[email protected]>
SUSE Labs, CR

2023-05-30 14:22:17

by Jan Kara

[permalink] [raw]
Subject: Re: [PATCH v2 0/2] SPDX conversion from UDF

On Mon 22-05-23 11:53:08, Jan Kara wrote:
> On Mon 22-05-23 07:54:33, Bagas Sanjaya wrote:
> > This small SPDX conversion series targets UDF file system, which is
> > splitted from v2 of my SPDX conversion series that is triggered by
> > Didi's GPL full name fixes [1]. It is done to ease review.
> >
> > All boilerplates in fs/udf/ is converted, except fs/udf/ecma_167.h.
> > The latter file apparently looks like 2-clause BSD Source-Code
> > license, yet the second clause is from third clause of 3-Clause BSD.
> > This custom license can't be expressed satisfiably in SPDX license
> > identifier, hence the file doesn't get converted.
> >
> > This series is based on mm-nonmm-unstable branch.
> >
> > Changes since v1 [2]:
> > * Correct SPDX tag for LGPL (correct spdxcheck warning)
> >
> > [1]: https://lore.kernel.org/linux-spdx/[email protected]/
> > [2]: https://lore.kernel.org/linux-mm/[email protected]/
> >
> > Bagas Sanjaya (2):
> > fs: udf: Replace GPL 2.0 boilerplate license notice with SPDX
> > identifier
> > fs: udf: udftime: Replace LGPL boilerplate with SPDX identifier
>
> The patches look good to me. So unless someone objects in a few days, I'll
> queue them into my tree. Thanks!

Patches queued now.

Honza