Skip to content

Commit a56793e

Browse files
committed
Migrate bom command to use a Go workspace
Remove `GOFLAGS=-mod=mod` from the execution of license-bill-of-materials. Signed-off-by: Ivan Valdes <[email protected]>
1 parent 573134f commit a56793e

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

scripts/test.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -505,7 +505,7 @@ function bom_pass {
505505
# BOM file should be generated for linux. Otherwise running this command on other operating systems such as OSX
506506
# results in certain dependencies being excluded from the BOM file, such as procfs.
507507
# For more info, https://github.com/etcd-io/etcd/issues/19665
508-
output=$(GOOS=linux GOFLAGS=-mod=mod run_go_tool github.com/appscodelabs/license-bill-of-materials \
508+
output=$(GOOS=linux run_go_tool github.com/appscodelabs/license-bill-of-materials \
509509
--override-file ./bill-of-materials.override.json \
510510
"${modules[@]}")
511511
code="$?"

scripts/updatebom.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ function bom_fixlet {
1717
# BOM file should be generated for linux. Otherwise running this command on other operating systems such as OSX
1818
# results in certain dependencies being excluded from the BOM file, such as procfs.
1919
# For more info, https://github.com/etcd-io/etcd/issues/19665
20-
if GOOS=linux GOFLAGS=-mod=mod run_go_tool "github.com/appscodelabs/license-bill-of-materials" \
20+
if GOOS=linux run_go_tool "github.com/appscodelabs/license-bill-of-materials" \
2121
--override-file ./bill-of-materials.override.json \
2222
"${modules[@]}" > ./bill-of-materials.json.tmp; then
2323
cp ./bill-of-materials.json.tmp ./bill-of-materials.json

0 commit comments

Comments
 (0)