Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36175 )
Change subject: util/lint: Update license header text for SPDX headers. ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/36175/3//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/36175/3//COMMIT_MSG@16 PS3, Line 16: GPL-2.0-only - Throughout coreboot source : GPL-2.0-or-later - Throughout coreboot source : GPL-3.0-only - util/amdtools : GPL-3.0-or-later - src/lib/[gcov/libgcov/gnat] not related to this patch but is this actually ok? GLP2 only is not compatible with GPL3 afaik.
https://review.coreboot.org/c/coreboot/+/36175/3/util/lint/lint-000-license-... File util/lint/lint-000-license-headers:
https://review.coreboot.org/c/coreboot/+/36175/3/util/lint/lint-000-license-... PS3, Line 106: check_for_license 'SPDX-License-Identifier: Apache-2.0' : check_for_license 'SPDX-License-Identifier: BSD-3-Clause' : check_for_license 'SPDX-License-Identifier: GPL-2.0-only' : check_for_license 'SPDX-License-Identifier: GPL-2.0-or-later' : check_for_license 'SPDX-License-Identifier: GPL-3.0-only' : check_for_license 'SPDX-License-Identifier: GPL-3.0-or-later' : check_for_license 'SPDX-License-Identifier: ISC' : check_for_license 'SPDX-License-Identifier: MIT' : check_for_license 'SPDX-License-Identifier: X11' Don't we want compatible licenses depending on the 'project'. e.g. coreboot itself needs GPL2 compatible. libpayload is X11? etc?