commitpkg: introduce reuse lint and download

Signed-off-by: Morten Linderud <foxboron@archlinux.org>

Makefile

@@ -1,6 +1,6 @@
 SHELL=/bin/bash -o pipefail
 
-V=1.4.0
+V=1.3.2
 BUILDTOOLVER ?= $(V)
 
 PREFIX = /usr/local
@@ -19,7 +19,6 @@ PACMAN_CONFIGS=$(wildcard config/pacman/*)
 GIT_CONFIGS = $(wildcard config/git/*)
 SETARCH_ALIASES = $(wildcard config/setarch-aliases.d/*)
 MANS = $(addprefix $(BUILDDIR)/,$(patsubst %.asciidoc,%,$(wildcard doc/man/*.asciidoc)))
-DATA_FILES = $(wildcard data/*)
 
 COMMITPKG_LINKS = \
 	core-testingpkg \
@@ -60,7 +59,7 @@ BATS_ARGS ?= --jobs $(JOBS) $(BATS_EXTRA_ARGS) --verbose-run
 COVERAGE_DIR ?= $(BUILDDIR)/coverage
 
 
-all: binprogs library conf completion man data
+all: binprogs library conf completion man
 binprogs: $(BINPROGS)
 library: $(LIBRARY)
 completion: $(COMPLETIONS)
@@ -113,10 +112,6 @@ conf:
 	@install -d $(BUILDDIR)/git.conf.d
 	@cp -a $(GIT_CONFIGS) $(BUILDDIR)/git.conf.d
 
-data:
-	@install -d $(BUILDDIR)/data
-	@cp -ra $(DATA_FILES) $(BUILDDIR)/data
-
 clean:
 	rm -rf $(BUILDDIR)
 
@@ -127,11 +122,9 @@ install: all
 	install -dm0755 $(DESTDIR)$(DATADIR)/pacman.conf.d
 	install -m0755 ${BINPROGS} $(DESTDIR)$(PREFIX)/bin
 	install -dm0755 $(DESTDIR)$(DATADIR)/lib
-	install -dm0755 $(DESTDIR)$(DATADIR)/data
 	cp -ra $(BUILDDIR)/lib/* $(DESTDIR)$(DATADIR)/lib
 	cp -a $(BUILDDIR)/git.conf.d -t $(DESTDIR)$(DATADIR)
 	cp -ra $(BUILDDIR)/makepkg.conf.d -t $(DESTDIR)$(DATADIR)
-	cp -ra $(BUILDDIR)/data -t $(DESTDIR)$(DATADIR)
 	for conf in $(notdir $(PACMAN_CONFIGS)); do install -Dm0644 $(BUILDDIR)/pacman.conf.d/$$conf $(DESTDIR)$(DATADIR)/pacman.conf.d/$${conf##*/}; done
 	for a in ${SETARCH_ALIASES}; do install -m0644 $$a -t $(DESTDIR)$(DATADIR)/setarch-aliases.d; done
 	for l in ${COMMITPKG_LINKS}; do ln -sf commitpkg $(DESTDIR)$(PREFIX)/bin/$$l; done
@@ -150,7 +143,6 @@ uninstall:
 	rm -rf $(DESTDIR)$(DATADIR)/lib
 	rm -rf $(DESTDIR)$(DATADIR)/git.conf.d
 	rm -rf $(DESTDIR)$(DATADIR)/makepkg.conf.d
-	rm -rf $(DESTDIR)$(DATADIR)/data
 	for conf in $(notdir $(PACMAN_CONFIGS)); do rm -f $(DESTDIR)$(DATADIR)/pacman.conf.d/$${conf##*/}; done
 	for f in $(notdir $(SETARCH_ALIASES)); do rm -f $(DESTDIR)$(DATADIR)/setarch-aliases.d/$$f; done
 	for l in ${COMMITPKG_LINKS}; do rm -f $(DESTDIR)$(PREFIX)/bin/$$l; done
@@ -194,5 +186,5 @@ coverage: binprogs library conf completion man
 check: $(BINPROGS_SRC) $(LIBRARY_SRC) contrib/completion/bash/devtools.in config/makepkg/x86_64.conf contrib/makepkg/PKGBUILD.proto
 	shellcheck $^
 
-.PHONY: all binprogs library completion conf man data clean install uninstall tag dist upload test coverage check
+.PHONY: all binprogs library completion conf man clean install uninstall tag dist upload test coverage check
 .DELETE_ON_ERROR:

README.md

@@ -94,7 +94,6 @@ Component: pkgctl db remove
 
 - bat (pretty printing)
 - nvchecker (version checking)
-- reuse (license compliance)
 
 ### Development Dependencies
 

config/git/template/info/exclude

@@ -2,7 +2,6 @@
 /src
 /*/
 !/keys/
-!/LICENSES/
 
 /*.log
 /*.tar.*

config/makepkg/conf.d/fortran.conf

@@ -1,6 +1,4 @@
 #!/hint/bash
-# shellcheck disable=2034
-
 #
 # /etc/makepkg.conf.d/fortran.conf
 #
@@ -11,12 +9,10 @@
 
 # Flags used for the Fortran compiler, similar in spirit to CFLAGS. Read
 # linkman:gfortran[1] for more details on the available flags.
-FFLAGS="-march=x86-64 -mtune=generic -O2 -pipe -fno-plt \
+#FFLAGS="-O2 -pipe"
-        -Wp,-D_FORTIFY_SOURCE=3 -fstack-clash-protection -fcf-protection \
+#FCFLAGS="$FFLAGS"
-        -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer"
-FCFLAGS="$FFLAGS"
 
 # Additional compiler flags appended to `FFLAGS` and `FCFLAGS` for use in debugging. Usually
 # this would include: ``-g''. Read linkman:gfortran[1] for more details on the wide
 # variety of compiler flags available.
-DEBUG_FFLAGS="-g"
+#DEBUG_FFLAGS="-g"

contrib/completion/bash/devtools.in

@@ -150,7 +150,6 @@ _pkgctl_cmds=(
 	db
 	diff
 	issue
-	license
 	release
 	repo
 	search
@@ -368,25 +367,6 @@ _pkgctl_repo_switch_opts() {
 	fi
 }
 
-_pkgctl_license_cmds=(
-	check
-	setup
-)
-
-_pkgctl_license_check_args=(
-	-h --help
-)
-
-_pkgctl_license_check_opts() { _filedir -d; }
-
-_pkgctl_license_setup_args=(
-	--no-check
-	-f --force
-	-h --help
-)
-
-_pkgctl_license_setup_opts() { _filedir -d; }
-
 _pkgctl_version_cmds=(
 	check
 	setup

contrib/completion/zsh/_devtools.in

@@ -399,7 +399,6 @@ _pkgctl_cmds=(
 	"db[Pacman database modification for package update, move etc]"
 	"diff[Compare package files using different modes]"
 	"issue[Work with GitLab packaging issues]"
-	"license[Check and manage package license compliance]"
 	"release[Release step to commit, tag and upload build artifacts]"
 	"repo[Manage Git packaging repositories and their configuration]"
 	"search[Search for an expression across the GitLab packaging group]"
@@ -411,24 +410,6 @@ _pkgctl_args=(
 	'(-h --help)'{-h,--help}'[Display usage]'
 )
 
-_pkgctl_license_cmds=(
-	"pkgctl license command"
-	"check[Checks package licensing compliance using REUSE]"
-	"setup[Automatically detect and setup a basic REUSE config]"
-)
-
-_pkgctl_license_check_args=(
-	'(-h --help)'{-h,--help}'[Display usage]'
-	'*:git_dir:_files -/'
-)
-
-_pkgctl_license_setup_args=(
-	'(-f --force)'{-f,--force}'[Overwrite existing REUSE config]'
-	'--no-check[Do not run license check after setup]'
-	'(-h --help)'{-h,--help}'[Display usage]'
-	'*:git_dir:_files -/'
-)
-
 _pkgctl_version_cmds=(
 	"pkgctl version command"
 	"check[Compares local package versions against upstream versions]"

data/LICENSE

@@ -1,12 +0,0 @@
-Copyright Arch Linux Contributors
-
-Permission to use, copy, modify, and/or distribute this software for
-any purpose with or without fee is hereby granted.
-
-THE SOFTWARE IS PROVIDED “AS IS” AND THE AUTHOR DISCLAIMS ALL
-WARRANTIES WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES
-OF MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE
-FOR ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY
-DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN
-AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT
-OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.

doc/man/pkgctl-license-check.1.asciidoc

@@ -1,54 +0,0 @@
-pkgctl-license-check(1)
-=======================
-
-Name
-----
-pkgctl-license-check - Checks package licensing compliance using REUSE
-
-Synopsis
---------
-pkgctl license check [OPTIONS] [PKGBASE...]
-
-Description
------------
-
-Checks package licensing compliance using REUSE and also verifies whether
-a LICENSE file with the expected Arch Linux-specific 0BSD license text exists.
-
-Configuration
--------------
-
-Uses reuse(1) and a `REUSE.toml` file located alongside the PKGBUILD(5). Refer
-to the configuration section in pkgctl-license(1).
-
-If no `PKGBASE` is specified, the command defaults to using the current working
-directory.
-
-Options
--------
-
-*-h, --help*::
-	Show a help text
-
-Exit Codes
-----------
-
-On exit, return one of the following codes:
-
-*0*::
-	Normal exit condition, all checked packages are compliant
-
-*1*::
-	Unknown cause of failure
-
-*2*::
-	Normal exit condition, but some packages are not compliant
-
-See Also
---------
-
-pkgctl-license(1)
-reuse(1)
-PKGBUILD(5)
-
-include::include/footer.asciidoc[]

doc/man/pkgctl-license-setup.1.asciidoc

@@ -1,55 +0,0 @@
-pkgctl-license-setup(1)
-=======================
-
-Name
-----
-pkgctl-license-setup - Automatically detect and setup a basic REUSE
-configuration
-
-Synopsis
---------
-pkgctl license setup [OPTIONS] [PKGBASE...]
-
-Description
------------
-
-This subcommand automates the creation of the Arch Linux 0BSD package license
-file as well as a basic reuse(1) configuration by applying simple heuristics.
-It comes in especially handy when initially setting up licensing for a package
-without the need to manually write a `REUSE.toml` file.
-
-If any `.patch` files are detected and the PKGBUILD(5) has only a single entry
-in the `license=()` array, this subcommand assumes the patches are licensed
-under that license and generates annotations for them.
-
-In case there are no patches, no additional annotations are generated.
-
-Manual annotations are necessary in case the subcommand can't generate a
-configuration that accounts for all files. In this case, `reuse lint` will fail
-with a descriptive error of which files are missing an annotation.
-
-If no `PKGBASE` is specified, the command defaults to using the current working
-directory.
-
-Options
--------
-
-*-f, --force*::
-	Overwrite existing reuse(1) configuration
-
-*--no-check*::
-	Do not run pkgctl-license-check(1) after setup
-
-*-h, --help*::
-	Show a help text
-
-
-See Also
---------
-
-pkgctl-license(1)
-pkgctl-license-check(1)
-reuse(1)
-PKGBUILD(5)
-
-include::include/footer.asciidoc[]

doc/man/pkgctl-license.1.asciidoc

@@ -1,54 +0,0 @@
-pkgctl-license(1)
-=================
-
-Name
-----
-pkgctl-license - Check and manage package license compliance
-
-Synopsis
---------
-pkgctl license [OPTIONS] [SUBCOMMAND]
-
-Description
------------
-
-Commands related to package licenses, including checks for compliance.
-
-Uses reuse(1) and a `REUSE.toml` file located alongside the PKGBUILD(5).
-
-Configuration
--------------
-
-The `REUSE.toml` file must contain annotations for all regular files expected
-to be present in an Arch Linux package repository.
-
-Use pkgctl-license-setup(1) to automatically detect and setup a basic REUSE
-config file based on the files in the package repository.
-
-For detailed information on the various configuration options available for the
-`REUSE.toml` file, refer to the REUSE Specification (https://reuse.software/spec).
-
-Options
--------
-
-*-h, --help*::
-	Show a help text
-
-Subcommands
------------
-
-pkgctl license check::
-	Checks package licensing compliance using REUSE
-
-pkgctl license setup::
-	Automatically detect and setup a basic REUSE config
-
-See Also
---------
-
-pkgctl-license-check(1)
-pkgctl-license-setup(1)
-reuse(1)
-PKGBUILD(5)
-
-include::include/footer.asciidoc[]

src/arch-nspawn.in

@@ -116,9 +116,6 @@ copy_hostconf () {
 		cp "$makepkg_conf" "$working_dir/etc/makepkg.conf"
 		if [[ -d "${makepkg_conf}.d" ]] && is_globfile "${makepkg_conf}.d"/*.conf; then
 			mkdir --parents "$working_dir/etc/makepkg.conf.d/"
-			if is_globfile "$working_dir/etc/makepkg.conf.d"/*.conf; then
-				rm --force "$working_dir/etc/makepkg.conf.d"/*.conf
-			fi
 			cp "${makepkg_conf}.d/"*.conf "$working_dir/etc/makepkg.conf.d/"
 		fi
 	fi

src/commitpkg.in

@@ -120,20 +120,16 @@ if (( ${#validpgpkeys[@]} != 0 )); then
 	git add --force -- keys/pgp/*
 fi
 
-needsversioning=()
+# Allow us to disable REUSE
-
+_RUN_REUSE=${_RUN_REUSE:-1}
-if [[ ! -e REUSE.toml || ! -e LICENSE || ! -d LICENSES ]]; then
+if ((_RUN_REUSE)); then
-	# TODO: Make this a hard failure in the future after packagers have had
+	reuse download
-	# some time to add licenses to all packages.
+	reuse lint
-	warning "package doesn't have proper licensing information, set it up using:"
+	git add --force -- LICENSES/*
-	msg2 'pkgctl license setup'
-else
-	pkgctl license check
-	needsversioning+=(REUSE.toml LICENSE LICENSES/*)
 fi
 
 # find files which should be under source control
-needsversioning+=(PKGBUILD)
+needsversioning=(PKGBUILD)
 for s in "${source[@]}"; do
 	[[ $s != *://* ]] && needsversioning+=("$s")
 done
@@ -155,7 +151,7 @@ if (( ${#needsversioning[*]} )); then
 		if [[ ! -f "${file}" ]]; then
 			continue
 		fi
-		if ! git ls-files --error-unmatch "$file" >/dev/null; then
+		if ! git ls-files --error-unmatch "$file"; then
 			die "%s is not under version control" "$file"
 		fi
 	done

src/lib/license.sh

@@ -1,66 +0,0 @@
-#!/hint/bash
-#
-# SPDX-License-Identifier: GPL-3.0-or-later
-
-[[ -z ${DEVTOOLS_INCLUDE_LICENSE_SH:-} ]] || return 0
-DEVTOOLS_INCLUDE_LICENSE_SH=1
-
-_DEVTOOLS_LIBRARY_DIR=${_DEVTOOLS_LIBRARY_DIR:-@pkgdatadir@}
-
-set -e
-
-
-pkgctl_license_usage() {
-	local -r COMMAND=${_DEVTOOLS_COMMAND:-${BASH_SOURCE[0]##*/}}
-    cat <<- _EOF_
-		Usage: ${COMMAND} [COMMAND] [OPTIONS]
-
-		Check and manage package license compliance.
-
-		COMMANDS
-		    check      Check package license compliance
-		    setup      Automatically detect and setup a basic REUSE config
-
-		OPTIONS
-		    -h, --help    Show this help text
-
-		EXAMPLES
-		    $ ${COMMAND} check libfoo linux libbar
-		    $ ${COMMAND} setup libfoo
-_EOF_
-}
-
-pkgctl_license() {
-	if (( $# < 1 )); then
-		pkgctl_license_usage
-		exit 0
-	fi
-
-	while (( $# )); do
-		case $1 in
-			-h|--help)
-				pkgctl_license_usage
-				exit 0
-				;;
-			check)
-				_DEVTOOLS_COMMAND+=" $1"
-				shift
-				# shellcheck source=src/lib/license/check.sh
-				source "${_DEVTOOLS_LIBRARY_DIR}"/lib/license/check.sh
-				pkgctl_license_check "$@"
-				exit $?
-				;;
-			setup)
-				_DEVTOOLS_COMMAND+=" $1"
-				shift
-				# shellcheck source=src/lib/license/setup.sh
-				source "${_DEVTOOLS_LIBRARY_DIR}"/lib/license/setup.sh
-				pkgctl_license_setup "$@"
-				exit 0
-				;;
-			*)
-				die "invalid argument: %s" "$1"
-				;;
-		esac
-	done
-}

src/lib/license/check.sh

@@ -1,147 +0,0 @@
-#!/bin/bash
-#
-# SPDX-License-Identifier: GPL-3.0-or-later
-#
-[[ -z ${DEVTOOLS_INCLUDE_LICENSE_CHECK_SH:-} ]] || return 0
-DEVTOOLS_INCLUDE_LICENSE_CHECK_SH=1
-
-_DEVTOOLS_LIBRARY_DIR=${_DEVTOOLS_LIBRARY_DIR:-@pkgdatadir@}
-# shellcheck source=src/lib/common.sh
-source "${_DEVTOOLS_LIBRARY_DIR}"/lib/common.sh
-
-source /usr/share/makepkg/util/message.sh
-
-set -eo pipefail
-
-readonly PKGCTL_LICENSE_CHECK_EXIT_COMPLIANT=0
-export PKGCTL_LICENSE_CHECK_EXIT_COMPLIANT
-readonly PKGCTL_LICENSE_CHECK_EXIT_FAILURE=2
-export PKGCTL_LICENSE_CHECK_EXIT_FAILURE
-
-
-pkgctl_license_check_usage() {
-	local -r COMMAND=${_DEVTOOLS_COMMAND:-${BASH_SOURCE[0]##*/}}
-	cat <<- _EOF_
-		Usage: ${COMMAND} [OPTIONS] [PKGBASE]...
-
-		Checks package licensing compliance using REUSE and also verifies
-		whether a LICENSE file with the expected Arch Linux-specific 0BSD
-		license text exists.
-
-		Upon execution, it runs 'reuse lint'.
-
-		OPTIONS
-		    -h, --help   Show this help text
-
-		EXAMPLES
-		    $ ${COMMAND} neovim vim
-_EOF_
-}
-
-pkgctl_license_check() {
-	local pkgbases=()
-	local verbose=0
-
-	local license_text
-	license_text=$(< "${_DEVTOOLS_LIBRARY_DIR}"/data/LICENSE)
-
-	local exit_code=${PKGCTL_LICENSE_CHECK_EXIT_COMPLIANT}
-
-	while (( $# )); do
-		case $1 in
-			-h|--help)
-				pkgctl_license_check_usage
-				exit 0
-				;;
-			--)
-				shift
-				break
-				;;
-			-*)
-				die "invalid argument: %s" "$1"
-				;;
-			*)
-				pkgbases=("$@")
-				break
-				;;
-		esac
-	done
-
-	if ! command -v reuse &>/dev/null; then
-		die "The \"$_DEVTOOLS_COMMAND\" command requires the 'reuse' CLI tool"
-	fi
-
-	# Check if used without pkgbases in a packaging directory
-	if (( ${#pkgbases[@]} == 0 )); then
-		if [[ -f PKGBUILD ]]; then
-			pkgbases=(".")
-		else
-			pkgctl_license_check_usage
-			exit 1
-		fi
-	fi
-
-	# enable verbose mode when we only have a single item to check
-	if (( ${#pkgbases[@]} == 1 )); then
-		verbose=1
-	fi
-
-	for path in "${pkgbases[@]}"; do
-		# skip paths that are not directories
-		if [[ ! -d "${path}" ]]; then
-			continue
-		fi
-		pushd "${path}" >/dev/null
-
-		if [[ ! -f PKGBUILD ]]; then
-			msg_error "${BOLD}${pkgbase}:${ALL_OFF} no PKGBUILD found"
-			return 1
-		fi
-
-		# reset common PKGBUILD variables
-		unset pkgbase
-
-		# shellcheck source=contrib/makepkg/PKGBUILD.proto
-		if ! . ./PKGBUILD; then
-			msg_error "${BOLD}${pkgbase}:${ALL_OFF} failed to source PKGBUILD"
-			return 1
-		fi
-		pkgbase=${pkgbase:-$pkgname}
-
-		if [[ ! -e LICENSE ]]; then
-			msg_error "${BOLD}${pkgbase}:${ALL_OFF} is missing the LICENSE file"
-			return "${PKGCTL_LICENSE_CHECK_EXIT_FAILURE}"
-		fi
-
-		if [[ ! -L LICENSES/0BSD.txt ]]; then
-			msg_error "${BOLD}${pkgbase}:${ALL_OFF} LICENSES/0BSD should be a symlink to LICENSE but it isn't"
-			return "${PKGCTL_LICENSE_CHECK_EXIT_FAILURE}"
-		fi
-
-		# Check if the local LICENSE file mismatches our expectations
-		if [[ $license_text != $(< LICENSE) ]]; then
-			msg_error "${BOLD}${pkgbase}:${ALL_OFF} LICENSE file doesn't have the expected Arch Linux-specific license text"
-			return "${PKGCTL_LICENSE_CHECK_EXIT_FAILURE}"
-		fi
-
-		# Check for REUSE compliance
-		if ! reuse lint --json | jq --exit-status '.summary.compliant' &>/dev/null; then
-			msg_error "${BOLD}${pkgbase}:${ALL_OFF} repository is not REUSE compliant"
-			exit_code=${PKGCTL_LICENSE_CHECK_EXIT_FAILURE}
-
-			# re-execute reuse lint for human readable output
-			if (( verbose )); then
-				reuse lint
-			fi
-
-			popd >/dev/null
-			continue
-		fi
-
-		msg_success "${BOLD}${pkgbase}:${ALL_OFF} repository is REUSE compliant"
-		popd >/dev/null
-	done
-
-	# return status based on results
-	return "${exit_code}"
-}

src/lib/license/setup.sh

@@ -1,272 +0,0 @@
-#!/bin/bash
-#
-# SPDX-License-Identifier: GPL-3.0-or-later
-#
-[[ -z ${DEVTOOLS_INCLUDE_LICENSE_SETUP_SH:-} ]] || return 0
-DEVTOOLS_INCLUDE_LICENSE_SETUP_SH=1
-
-_DEVTOOLS_LIBRARY_DIR=${_DEVTOOLS_LIBRARY_DIR:-@pkgdatadir@}
-# shellcheck source=src/lib/common.sh
-source "${_DEVTOOLS_LIBRARY_DIR}"/lib/common.sh
-# shellcheck source=src/lib/license/check.sh
-source "${_DEVTOOLS_LIBRARY_DIR}"/lib/license/check.sh
-
-source /usr/share/makepkg/util/message.sh
-
-set -eo pipefail
-shopt -s nullglob
-
-
-pkgctl_license_setup_usage() {
-	local -r COMMAND=${_DEVTOOLS_COMMAND:-${BASH_SOURCE[0]##*/}}
-	cat <<- _EOF_
-		Usage: ${COMMAND} [OPTIONS] [PKGBASE]...
-
-		Automate the creation of a basic REUSE configuration by analyzing the
-		license array specified in the PKGBUILD file of a package.
-
-		If no PKGBASE is specified, the command defaults to using the current
-		working directory.
-
-		OPTIONS
-		    -f, --force   Overwrite existing REUSE configuration
-		    -h, --help    Show this help text
-		    --no-check    Do not run license check after setup
-
-		EXAMPLES
-		    $ ${COMMAND} neovim vim
-_EOF_
-}
-
-pkgctl_license_setup() {
-	local pkgbases=()
-	local force=0
-	local run_check=1
-
-	local path exit_code
-	local checks=()
-
-	while (( $# )); do
-		case $1 in
-			-h|--help)
-				pkgctl_license_setup_usage
-				exit 0
-				;;
-			-f|--force)
-				force=1
-				shift
-				;;
-			--no-check)
-				run_check=0
-				shift
-				;;
-			--)
-				shift
-				break
-				;;
-			-*)
-				die "invalid argument: %s" "$1"
-				;;
-			*)
-				pkgbases=("$@")
-				break
-				;;
-		esac
-	done
-
-	if ! command -v reuse &>/dev/null; then
-		die "The \"$_DEVTOOLS_COMMAND\" command requires the 'reuse' CLI tool"
-	fi
-
-	# Check if used without pkgbases in a packaging directory
-	if (( ${#pkgbases[@]} == 0 )); then
-		if [[ -f PKGBUILD ]]; then
-			pkgbases=(".")
-		else
-			pkgctl_license_setup_usage
-			exit 1
-		fi
-	fi
-
-	exit_code=0
-	for path in "${pkgbases[@]}"; do
-		# skip paths that are not directories
-		if [[ ! -d "${path}" ]]; then
-			continue
-		fi
-
-		pushd "${path}" >/dev/null
-		if license_setup "${path}" "${force}"; then
-			checks+=("${path}")
-		else
-			exit_code=1
-		fi
-		popd >/dev/null
-	done
-
-	# run checks on the setup targets
-	if (( run_check )) && (( ${#checks[@]} >= 1 )); then
-		echo
-		echo 📡 Running checks...
-		pkgctl_license_check "${checks[@]}" || true
-	fi
-
-	return "$exit_code"
-}
-
-license_setup() {
-	local path=$1
-	local force=$2
-	local pkgbase pkgname license
-
-	if [[ ! -f PKGBUILD ]]; then
-		msg_error "${BOLD}${path}:${ALL_OFF} no PKGBUILD found"
-		return 1
-	fi
-
-	# shellcheck source=contrib/makepkg/PKGBUILD.proto
-	if ! . ./PKGBUILD; then
-		msg_error "${BOLD}${path}:${ALL_OFF} failed to source PKGBUILD"
-		return 1
-	fi
-	pkgbase=${pkgbase:-$pkgname}
-
-	# setup LICENSE file
-	if ! license_file_setup "${pkgbase}" "${force}"; then
-		return 1
-	fi
-
-	# setup REUSE.toml
-	if ! reuse_setup "${pkgbase}" "${force}" "${license[@]}"; then
-		return 1
-	fi
-}
-
-is_valid_license() {
-	local license_name=$1
-	local supported_licenses
-	supported_licenses=$(reuse supported-licenses | awk '{print $1}')
-	if grep --quiet "^${license_name}$" <<< "$supported_licenses"; then
-		return 0
-	else
-		return 1
-	fi
-}
-
-license_file_setup() {
-	local pkgbase=$1
-	local force=$2
-
-	local license_text
-	license_text=$(< "${_DEVTOOLS_LIBRARY_DIR}"/data/LICENSE)
-
-	# Write LICENSE file, or check if it mismatches
-	if (( force )) || [[ ! -f LICENSE ]]; then
-		printf "%s\n" "${license_text}" > LICENSE
-		msg_success "${BOLD}${pkgbase}:${ALL_OFF} successfully configured LICENSE"
-	elif [[ -f LICENSE ]]; then
-		# if there is a license file, check whether it has the text we expect
-		existing_license="$(< LICENSE)"
-		if [[ "${license_text}" != "${existing_license}" ]]; then
-			msg_error "${BOLD}${pkgbase}:${ALL_OFF} existing LICENSE file doesn't have expected content, use --force to overwrite"
-			return 1
-		fi
-	fi
-
-	# make sure the LICENSE file is found by REUSE
-	mkdir --parents LICENSES
-	ln --symbolic --force ../LICENSE LICENSES/0BSD.txt
-}
-
-reuse_default_annotations() {
-	cat << EOF
-version = 1
-
-[[annotations]]
-path = [
-    "PKGBUILD",
-    "README.md",
-    "keys/**",
-    ".SRCINFO",
-    ".gitignore",
-    ".nvchecker.toml",
-    "*.install",
-    "*.sysusers",
-    "*sysusers.conf",
-    "*.tmpfiles",
-    "*tmpfiles.conf",
-    "*.logrotate",
-    "*.pam",
-    "*.service",
-    "*.socket",
-    "*.timer",
-    "*.desktop",
-    "*.hook",
-]
-SPDX-FileCopyrightText = "Arch Linux contributors"
-SPDX-License-Identifier = "0BSD"
-EOF
-}
-
-reuse_setup() {
-	local pkgbase=$1
-	local force=$2
-	shift 2
-	local license=("$@")
-
-	# Check if REUSE.toml already exists
-	if (( ! force )) && [[ -f REUSE.toml ]]; then
-		msg_error "${BOLD}${pkgbase}:${ALL_OFF} REUSE.toml already exists, use --force to overwrite"
-		return 1
-	fi
-
-	reuse_default_annotations > REUSE.toml
-
-	local warning_occurred=0
-	local patches=(*.patch)
-	# If there are patches and there's only a single well-known license listed in the package,
-	# we can generate the annotations for the patches, otherwise we will fail to do so and warn
-	# the user.
-	if (( ${#patches} )); then
-		# If there are multiple licenses, we can't make a good guess about which license the
-		# patches should have. In case the first element contains a space, we are dealing with
-		# a complex SPDX license identifier.
-		if (( ${#license[@]} > 1 )) || [[ ${license[0]} =~ [[:space:]] ]]; then
-			msg_warn "${BOLD}${pkgbase}:${ALL_OFF} .patch files were found but couldn't automatically guess a suitable license because PKGBUILD has multiple licenses"
-			patch_annotations "${pkgbase}" "TODO-Choose-a-license" "${patches[@]}" >> REUSE.toml
-			warning_occurred=1
-		elif ! is_valid_license "${license[0]}"; then
-			msg_warn "${BOLD}${pkgbase}:${ALL_OFF} .patch files were found but couldn't automatically guess a suitable license because the PKGBUILD license '${license[0]}' is not a recognized SPDX license"
-			patch_annotations "${pkgbase}" "TODO-Choose-a-license" "${patches[@]}" >> REUSE.toml
-			warning_occurred=1
-		else
-			patch_annotations "${pkgbase}" "${license[0]}" "${patches[@]}" >> REUSE.toml
-		fi
-	fi
-
-	if (( warning_occurred )); then
-		msg_warn "${BOLD}${pkgbase}:${ALL_OFF} configured REUSE but a warning occurred, manually edit and fix REUSE.toml"
-		return 1
-	else
-		reuse download --all
-		msg_success "${BOLD}${pkgbase}:${ALL_OFF} successfully configured REUSE.toml"
-	fi
-}
-
-patch_annotations() {
-	local pkgbase=$1
-	local license_identifier=$2
-	shift 2
-	local patches=("$@")
-
-	local annotations
-	annotations+="\n[[annotations]]\n"
-	annotations+="path = [\n"
-	for patch in "${patches[@]}"; do
-		annotations+="    \"$(basename "${patch}")\",\n"
-	done
-	annotations+="]\n"
-	annotations+="SPDX-FileCopyrightText = \"${pkgbase} contributors\"\n"
-	annotations+="SPDX-License-Identifier = \"${license_identifier}\""
-	echo -e "${annotations}"
-}

src/pkgctl.in

@@ -25,7 +25,6 @@ usage() {
 		    db      Pacman database modification for package update, move etc
 		    diff    Compare package files using different modes
 		    issue   Work with GitLab packaging issues
-		    license Check and manage package licenses
 		    release Release step to commit, tag and upload build artifacts
 		    repo    Manage Git packaging repositories and their configuration
 		    search  Search for an expression across the GitLab packaging group
@@ -114,14 +113,6 @@ while (( $# )); do
 			pkgctl_issue "$@"
 			exit 0
 			;;
-		license)
-			_DEVTOOLS_COMMAND+=" $1"
-			shift
-			# shellcheck source=src/lib/license.sh
-			source "${_DEVTOOLS_LIBRARY_DIR}"/lib/license.sh
-			pkgctl_license "$@"
-			exit 0
-			;;
 		release)
 			_DEVTOOLS_COMMAND+=" $1"
 			shift