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

Reply via email to