From 849e2f3d4e1975bf02cfc7d2b32534294780a53b Mon Sep 17 00:00:00 2001 From: Valentin Rothberg Date: Mon, 8 Jul 2019 15:28:39 +0200 Subject: analyses: add dependency-tree.sh Add a new analysis script to print the dependency tree. Signed-off-by: Valentin Rothberg --- dependencies/analyses/dependency-tree.sh | 13 +++++++++++++ 1 file changed, 13 insertions(+) create mode 100755 dependencies/analyses/dependency-tree.sh (limited to 'dependencies/analyses/dependency-tree.sh') diff --git a/dependencies/analyses/dependency-tree.sh b/dependencies/analyses/dependency-tree.sh new file mode 100755 index 000000000..3c9dccc51 --- /dev/null +++ b/dependencies/analyses/dependency-tree.sh @@ -0,0 +1,13 @@ +#!/usr/bin/bash + +if test "$#" -ne 1; then + echo "invalid arguments: usage: $0 path to package" + exit 1 +fi + +DATA=$(go list $1/... \ + | xargs -d '\n' go list -f '{{ .ImportPath }}: {{ join .Imports ", " }}' \ + | awk '{ printf "%s\n\n", $0 }' \ + ) + +echo "$DATA" -- cgit v1.2.3-54-g00ecf From 25d63f009dfb0bd646b761591a08989cfd9d4fda Mon Sep 17 00:00:00 2001 From: Valentin Rothberg Date: Fri, 12 Jul 2019 11:35:48 +0200 Subject: dependency-tree analysis: direct and transitive Change the script to generate two files. One including direct dependencies, the other including direct and transitive dependencies. Signed-off-by: Valentin Rothberg --- dependencies/analyses/README.md | 10 ++++++---- dependencies/analyses/dependency-tree.sh | 9 ++++++++- 2 files changed, 14 insertions(+), 5 deletions(-) (limited to 'dependencies/analyses/dependency-tree.sh') diff --git a/dependencies/analyses/README.md b/dependencies/analyses/README.md index e05bc3e8f..a440a0ebd 100644 --- a/dependencies/analyses/README.md +++ b/dependencies/analyses/README.md @@ -71,15 +71,17 @@ Running the script can help identify sources of bloat and reveal potential candi Use the `dependency-tree.sh` script to figure out which package includes which packages. The output of the script has the format `package: dependency_1, dependency_2, ...`. Each line is followed by a blank line to make it easier to read. -Note that the list of dependencies includes only the direct dependencies and not all transitive dependencies. -The transitive dependencies of a given package can be examined by running `go list -f '{{ .Name }}: {{ join .Deps ", " }}' $PACKAGE` or by browsing through the output of `dependency-tree.sh`. +The script generates two files: + + - `direct-tree.txt` - listing direct dependencies + - `transitive-tree.txt` - listing direct and transitive dependencies Running such a dependency-tree analysis may look as follows: ``` -[libpod]$ ./dependencies/analyses/dependency-tree.sh github.com/containers/libpod > tree.txt -[libpod]$ grep "^github.com/containers/libpod/pkg/registries" tree.txt +[libpod]$ ./dependencies/analyses/dependency-tree.sh github.com/containers/libpod +[libpod]$ grep "^github.com/containers/libpod/pkg/registries" direct-tree.txt github.com/containers/libpod/pkg/registries: github.com/containers/libpod/vendor/github.com/containers/image/pkg/sysregistriesv2, github.com/containers/libpod/vendor/github.com/containers/image/types, github.com/containers/libpod/pkg/rootless, github.com/containers/libpod/vendor/github.com/docker/distribution/reference, github.com/containers/libpod/vendor/github.com/pkg/errors, os, path/filepath, strings ``` diff --git a/dependencies/analyses/dependency-tree.sh b/dependencies/analyses/dependency-tree.sh index 3c9dccc51..9a2e3282d 100755 --- a/dependencies/analyses/dependency-tree.sh +++ b/dependencies/analyses/dependency-tree.sh @@ -10,4 +10,11 @@ DATA=$(go list $1/... \ | awk '{ printf "%s\n\n", $0 }' \ ) -echo "$DATA" +echo "$DATA" > direct-tree.txt + +DATA=$(go list $1/... \ + | xargs -d '\n' go list -f '{{ .ImportPath }}: {{ join .Deps ", " }}' \ + | awk '{ printf "%s\n\n", $0 }' \ + ) + +echo "$DATA" > transitive-tree.txt -- cgit v1.2.3-54-g00ecf From 294ddc472047f5a13bcf1d9b41407e18b1fad984 Mon Sep 17 00:00:00 2001 From: Valentin Rothberg Date: Tue, 16 Jul 2019 14:14:48 +0200 Subject: dependency/analyses: simplify scripts Credits to bash wizard @edsantiago for the changes. Signed-off-by: Valentin Rothberg --- dependencies/analyses/dependency-tree.sh | 11 ++++------- dependencies/analyses/nm-symbols-analysis.sh | 10 ++-------- 2 files changed, 6 insertions(+), 15 deletions(-) (limited to 'dependencies/analyses/dependency-tree.sh') diff --git a/dependencies/analyses/dependency-tree.sh b/dependencies/analyses/dependency-tree.sh index 9a2e3282d..84085a50d 100755 --- a/dependencies/analyses/dependency-tree.sh +++ b/dependencies/analyses/dependency-tree.sh @@ -5,16 +5,13 @@ if test "$#" -ne 1; then exit 1 fi -DATA=$(go list $1/... \ +go list $1/... \ | xargs -d '\n' go list -f '{{ .ImportPath }}: {{ join .Imports ", " }}' \ | awk '{ printf "%s\n\n", $0 }' \ - ) + > direct-tree.tmp.$$ && mv -f direct-tree.tmp.$$ direct-tree.txt -echo "$DATA" > direct-tree.txt -DATA=$(go list $1/... \ +go list $1/... \ | xargs -d '\n' go list -f '{{ .ImportPath }}: {{ join .Deps ", " }}' \ | awk '{ printf "%s\n\n", $0 }' \ - ) - -echo "$DATA" > transitive-tree.txt + > transitive-tree.tmp.$$ && mv -f transitive-tree.tmp.$$ transitive-tree.txt diff --git a/dependencies/analyses/nm-symbols-analysis.sh b/dependencies/analyses/nm-symbols-analysis.sh index 924246715..361b746e4 100755 --- a/dependencies/analyses/nm-symbols-analysis.sh +++ b/dependencies/analyses/nm-symbols-analysis.sh @@ -5,11 +5,5 @@ if test "$#" -ne 1; then exit 1 fi -DATA=$(go tool nm -size "$1" \ - | awk 'NF==4 {printf "%s\t%s\t%s\n", $2, $3, $4}' \ - | grep -v -P "\t_\t" \ - | grep -P "\tt\t" \ - | awk ' {printf "%s\t\t%s\n", $1, $3} ' \ - ) - -echo "$DATA" +go tool nm -size "$1" \ + | awk 'NF==4 && $3=="t" {printf "%s\t\t%s\n", $2, $4}' -- cgit v1.2.3-54-g00ecf