From 45369acf791426004f7f0f5da7346523fe279399 Mon Sep 17 00:00:00 2001 From: "W. Trevor King" Date: Wed, 3 Jan 2018 10:46:57 -0800 Subject: [PATCH] check-approval: Fix missing license_ids variable Avoid: $ ./script/check-approval MIT ./script/check-approval:55:in `
': undefined local variable or method `license_ids' for main:Object (NameError) The license_ids line was added in e505eb8f (check if license is aleady a license, 2016-01-18, #318), but license_ids was removed from the helper in b99e7ab0 (replace 'id' variables with 'spdx_lcase' to minimize confusion, 2016-06-01, #424). This commit restores the old code locally, since this script is the only consumer. --- script/check-approval | 1 + 1 file changed, 1 insertion(+) diff --git a/script/check-approval b/script/check-approval index 6b491db..6451818 100755 --- a/script/check-approval +++ b/script/check-approval @@ -52,6 +52,7 @@ approvals.each do |approver, licenses| rows << ["#{approver} approved", licenses.include?(license)] end +license_ids = licenses.map { |l| l['id'] } current = license_ids.include?(license) rows << ['Current license', current]