add --license option to print (SPDX) license tag in the `installed-packages` overview. This is useful for manifest generation, as it show all licenses used within an image. In case the license is not available, a question mark (?) is printed instead.
base-files - 198-r10203+1-c12bd3a21b - GPL-2.0 bnx2-firmware - 20190416-1 - ? busybox - 1.30.1-4 - GPL-2.0 dnsmasq - 2.80-13 - GPL-2.0 dropbear - 2019.78-2 - MIT e2fsprogs - 1.44.5-1 - GPL-2.0 firewall - 2019-01-02-70f8785b-2 - ISC fstools - 2019-03-28-ff1ded63-4 - GPL-2.0 fwtool - 1 - ? getrandom - 2019-04-07-5130fa4d-2 - GPL-2.0 ip6tables - 1.8.2-3 - GPL-2.0 iptables - 1.8.2-3 - GPL-2.0 jshn - 2019-02-27-eeef7b50-1 - ISC jsonfilter - 2018-02-04-c7e938d6-1 - ISC ... CC: Jo-Philipp Wich <j...@mein.io> Signed-off-by: Paul Spooren <m...@aparcar.org> --- v2 <- v1 ignore license via PFM_LICENSE in opkg_cmd.c when not actually needed libopkg/opkg_cmd.c | 51 +++++++++++++++++++++++++-------------------- libopkg/opkg_conf.c | 1 + libopkg/opkg_conf.h | 1 + libopkg/pkg.c | 11 ++++++++++ libopkg/pkg.h | 1 + libopkg/pkg_parse.c | 5 +++++ libopkg/pkg_parse.h | 1 + src/opkg-cl.c | 7 +++++++ 8 files changed, 55 insertions(+), 23 deletions(-) diff --git a/libopkg/opkg_cmd.c b/libopkg/opkg_cmd.c index c823df8..5fc9ae5 100644 --- a/libopkg/opkg_cmd.c +++ b/libopkg/opkg_cmd.c @@ -45,9 +45,12 @@ static void print_pkg(pkg_t * pkg) { char *version = pkg_version_str_alloc(pkg); char *description = pkg_get_string(pkg, PKG_DESCRIPTION); + char *license = pkg_get_string(pkg, PKG_LICENSE); printf("%s - %s", pkg->name, version); if (conf->size) printf(" - %lu", (unsigned long) pkg_get_int(pkg, PKG_SIZE)); + if (conf->license) + printf(" - %s", license ? license : "?"); if (description) printf(" - %s", description); printf("\n"); @@ -1255,10 +1258,10 @@ static int opkg_print_architecture_cmd(int argc, char **argv) array for easier maintenance */ static opkg_cmd_t cmds[] = { {"update", 0, (opkg_cmd_fun_t) opkg_update_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"upgrade", 1, (opkg_cmd_fun_t) opkg_upgrade_cmd, PFM_DESCRIPTION | PFM_SOURCE}, - {"list", 0, (opkg_cmd_fun_t) opkg_list_cmd, PFM_SOURCE}, + {"list", 0, (opkg_cmd_fun_t) opkg_list_cmd, PFM_SOURCE | PFM_LICENSE}, {"list_installed", 0, (opkg_cmd_fun_t) opkg_list_installed_cmd, PFM_SOURCE}, {"list-installed", 0, (opkg_cmd_fun_t) opkg_list_installed_cmd, @@ -1268,55 +1271,57 @@ static opkg_cmd_t cmds[] = { {"list-upgradable", 0, (opkg_cmd_fun_t) opkg_list_upgradable_cmd, PFM_SOURCE}, {"list_changed_conffiles", 0, - (opkg_cmd_fun_t) opkg_list_changed_conffiles_cmd, PFM_SOURCE}, + (opkg_cmd_fun_t) opkg_list_changed_conffiles_cmd, + PFM_SOURCE | PFM_LICENSE}, {"list-changed-conffiles", 0, - (opkg_cmd_fun_t) opkg_list_changed_conffiles_cmd, PFM_SOURCE}, + (opkg_cmd_fun_t) opkg_list_changed_conffiles_cmd, + PFM_SOURCE | PFM_LICENSE}, {"info", 0, (opkg_cmd_fun_t) opkg_info_cmd, 0}, {"flag", 1, (opkg_cmd_fun_t) opkg_flag_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"status", 0, (opkg_cmd_fun_t) opkg_status_cmd, PFM_DESCRIPTION | PFM_SOURCE}, {"install", 1, (opkg_cmd_fun_t) opkg_install_cmd, PFM_DESCRIPTION | PFM_SOURCE}, {"remove", 1, (opkg_cmd_fun_t) opkg_remove_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"configure", 0, (opkg_cmd_fun_t) opkg_configure_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"files", 1, (opkg_cmd_fun_t) opkg_files_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"search", 1, (opkg_cmd_fun_t) opkg_search_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, - {"find", 1, (opkg_cmd_fun_t) opkg_find_cmd, PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, + {"find", 1, (opkg_cmd_fun_t) opkg_find_cmd, PFM_SOURCE | PFM_LICENSE}, {"download", 1, (opkg_cmd_fun_t) opkg_download_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"compare_versions", 1, (opkg_cmd_fun_t) opkg_compare_versions_cmd, 0}, {"compare-versions", 1, (opkg_cmd_fun_t) opkg_compare_versions_cmd, 0}, {"print-architecture", 0, (opkg_cmd_fun_t) opkg_print_architecture_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"print_architecture", 0, (opkg_cmd_fun_t) opkg_print_architecture_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"print-installation-architecture", 0, (opkg_cmd_fun_t) opkg_print_architecture_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"print_installation_architecture", 0, (opkg_cmd_fun_t) opkg_print_architecture_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"depends", 1, (opkg_cmd_fun_t) opkg_depends_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"whatdepends", 1, (opkg_cmd_fun_t) opkg_whatdepends_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"whatdependsrec", 1, (opkg_cmd_fun_t) opkg_whatdepends_recursively_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"whatrecommends", 1, (opkg_cmd_fun_t) opkg_whatrecommends_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"whatsuggests", 1, (opkg_cmd_fun_t) opkg_whatsuggests_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"whatprovides", 1, (opkg_cmd_fun_t) opkg_whatprovides_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"whatreplaces", 1, (opkg_cmd_fun_t) opkg_whatreplaces_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, {"whatconflicts", 1, (opkg_cmd_fun_t) opkg_whatconflicts_cmd, - PFM_DESCRIPTION | PFM_SOURCE}, + PFM_DESCRIPTION | PFM_SOURCE | PFM_LICENSE}, }; opkg_cmd_t *opkg_cmd_find(const char *name) diff --git a/libopkg/opkg_conf.c b/libopkg/opkg_conf.c index 08855eb..40b1734 100644 --- a/libopkg/opkg_conf.c +++ b/libopkg/opkg_conf.c @@ -70,6 +70,7 @@ opkg_option_t options[] = { {"proxy_user", OPKG_OPT_TYPE_STRING, &_conf.proxy_user}, {"query-all", OPKG_OPT_TYPE_BOOL, &_conf.query_all}, {"size", OPKG_OPT_TYPE_BOOL, &_conf.size}, + {"license", OPKG_OPT_TYPE_BOOL, &_conf.license}, {"tmp_dir", OPKG_OPT_TYPE_STRING, &_conf.tmp_dir}, {"verbosity", OPKG_OPT_TYPE_INT, &_conf.verbosity}, {NULL, 0, NULL} diff --git a/libopkg/opkg_conf.h b/libopkg/opkg_conf.h index 37f95a1..0413ccd 100644 --- a/libopkg/opkg_conf.h +++ b/libopkg/opkg_conf.h @@ -87,6 +87,7 @@ struct opkg_conf { int verbosity; int noaction; int size; + int license; int download_only; char *cache; diff --git a/libopkg/pkg.c b/libopkg/pkg.c index e5bfe6f..fb1b2f1 100644 --- a/libopkg/pkg.c +++ b/libopkg/pkg.c @@ -777,6 +777,15 @@ void pkg_formatted_field(FILE * fp, pkg_t * pkg, const char *field) } } break; + case 'l': + case 'L': + if (strcasecmp(field, "License") == 0) { + p = pkg_get_string(pkg, PKG_LICENSE); + if (p && *p) { + fprintf(fp, "License: %s\n", p); + } + } + break; case 'm': case 'M': if (strcasecmp(field, "Maintainer") == 0) { @@ -926,6 +935,7 @@ void pkg_formatted_info(FILE * fp, pkg_t * pkg) pkg_formatted_field(fp, pkg, "Package"); pkg_formatted_field(fp, pkg, "Version"); pkg_formatted_field(fp, pkg, "Depends"); + pkg_formatted_field(fp, pkg, "License"); pkg_formatted_field(fp, pkg, "Recommends"); pkg_formatted_field(fp, pkg, "Suggests"); pkg_formatted_field(fp, pkg, "Provides"); @@ -956,6 +966,7 @@ void pkg_print_status(pkg_t * pkg, FILE * file) pkg_formatted_field(file, pkg, "Package"); pkg_formatted_field(file, pkg, "Version"); pkg_formatted_field(file, pkg, "Depends"); + pkg_formatted_field(file, pkg, "License"); pkg_formatted_field(file, pkg, "Recommends"); pkg_formatted_field(file, pkg, "Suggests"); pkg_formatted_field(file, pkg, "Provides"); diff --git a/libopkg/pkg.h b/libopkg/pkg.h index 600fc9e..0065043 100644 --- a/libopkg/pkg.h +++ b/libopkg/pkg.h @@ -79,6 +79,7 @@ typedef enum pkg_state_status pkg_state_status_t; enum pkg_fields { PKG_MAINTAINER, + PKG_LICENSE, PKG_PRIORITY, PKG_SOURCE, PKG_TAGS, diff --git a/libopkg/pkg_parse.c b/libopkg/pkg_parse.c index 0baa4db..87db2fa 100644 --- a/libopkg/pkg_parse.c +++ b/libopkg/pkg_parse.c @@ -269,6 +269,11 @@ int pkg_parse_line(void *ptr, char *line, uint mask) pkg_set_int(pkg, PKG_INSTALLED_TIME, strtoul(line + strlen("Installed-Time") + 1, NULL, 0)); } break; + case 'l': + case 'L': + if ((mask & PFM_LICENSE) && is_field("License:", line)) + pkg_set_string(pkg, PKG_LICENSE, line + strlen("License") + 1); + break; case 'M': if ((mask & PFM_MD5SUM) && (is_field("MD5sum:", line) || is_field("MD5Sum:", line))) diff --git a/libopkg/pkg_parse.h b/libopkg/pkg_parse.h index d1f901a..ccb97aa 100644 --- a/libopkg/pkg_parse.h +++ b/libopkg/pkg_parse.h @@ -54,6 +54,7 @@ int pkg_parse_line(void *ptr, char *line, uint mask); #define PFM_SUGGESTS (1 << 25) #define PFM_TAGS (1 << 26) #define PFM_VERSION (1 << 27) +#define PFM_LICENSE (1 << 28) #define PFM_ALL (~(uint)0) diff --git a/src/opkg-cl.c b/src/opkg-cl.c index a3ea5c1..b867487 100644 --- a/src/opkg-cl.c +++ b/src/opkg-cl.c @@ -54,6 +54,7 @@ enum { ARGS_OPT_FORCE_SIGNATURE, ARGS_OPT_NO_CHECK_CERTIFICATE, ARGS_OPT_SIZE, + ARGS_OPT_LICENSE, }; static struct option long_options[] = { @@ -103,6 +104,7 @@ static struct option long_options[] = { {"add-arch", 1, 0, ARGS_OPT_ADD_ARCH}, {"add-dest", 1, 0, ARGS_OPT_ADD_DEST}, {"size", 0, 0, ARGS_OPT_SIZE}, + {"license", 0, 0, ARGS_OPT_LICENSE}, {"test", 0, 0, ARGS_OPT_NOACTION}, {"tmp-dir", 1, 0, 't'}, {"tmp_dir", 1, 0, 't'}, @@ -220,6 +222,9 @@ static int args_parse(int argc, char *argv[]) case ARGS_OPT_SIZE: conf->size = 1; break; + case ARGS_OPT_LICENSE: + conf->license = 1; + break; case ARGS_OPT_NOACTION: conf->noaction = 1; break; @@ -349,6 +354,8 @@ static void usage() ("\t--nocase Perform case insensitive pattern matching\n"); printf ("\t--size Print package size when listing available packages\n"); + printf + ("\t--license Print package license when listing available packages\n"); printf("\t--force-removal-of-dependent-packages\n"); printf("\t Remove package and all dependencies\n"); printf("\t--autoremove Remove packages that were installed\n"); -- 2.20.1 _______________________________________________ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/mailman/listinfo/openwrt-devel