123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199 |
- ## Script from https://github.com/tir38/android-lint-entropy-reducer at 07.05.2017
- # adapts to drone, use git username / token as parameter
- Encoding.default_external = Encoding::UTF_8
- Encoding.default_internal = Encoding::UTF_8
- puts "=================== starting Android Lint Entropy Reducer ===================="
- # get args
- git_user, git_token, git_branch = ARGV
- # ======================== SETUP ============================
- # User name for git commits made by this script.
- TRAVIS_GIT_USERNAME = String.new("Drone CI server")
- # File name and relative path of generated Lint report. Must match build.gradle file:
- # lintOptions {
- # htmlOutput file("[FILE_NAME].html")
- # }
- LINT_REPORT_FILE = String.new("build/reports/lint/lint.html")
- # File name and relative path of previous results of this script.
- PREVIOUS_LINT_RESULTS_FILE=String.new("scripts/analysis/lint-results.txt")
- # Flag to evaluate warnings. true = check warnings; false = ignore warnings
- CHECK_WARNINGS = true
- # File name and relative path to custom lint rules; Can be null or "".
- CUSTOM_LINT_FILE = String.new("")
- # ================ SETUP DONE; DON'T TOUCH ANYTHING BELOW ================
- require 'fileutils'
- require 'pathname'
- require 'open3'
- # since we need the xml-simple gem, and we want this script self-contained, let's grab it just when we need it
- begin
- gem "xml-simple"
- rescue LoadError
- system("gem install xml-simple")
- Gem.clear_paths
- end
- require 'xmlsimple'
- # add custom Lint jar
- if !CUSTOM_LINT_FILE.nil? &&
- CUSTOM_LINT_FILE.length > 0
- ENV["ANDROID_LINT_JARS"] = Dir.pwd + "/" + CUSTOM_LINT_FILE
- puts "adding custom lint rules to default set: "
- puts ENV["ANDROID_LINT_JARS"]
- end
- # run Lint
- puts "running Lint..."
- system './gradlew clean lint 1>/dev/null'
- # confirm that Lint ran w/out error
- result = $?.to_i
- if result != 0
- puts "FAIL: failed to run ./gradlew clean lint"
- exit 1
- end
- # find Lint report file
- lint_reports = Dir.glob(LINT_REPORT_FILE)
- if lint_reports.length == 0
- puts "Lint HTML report not found."
- exit 1
- end
- lint_report = String.new(lint_reports[0])
- # find error/warning count string in HTML report
- error_warning_string = ""
- File.open lint_report do |file|
- error_warning_string = file.find { |line| line =~ /([0-9]* error[s]? and )?[0-9]* warning[s]?/ }
- end
- # find number of errors
- error_string = error_warning_string.match(/[0-9]* error[s]?/)
- if (error_string.nil?)
- current_error_count = 0
- else
- current_error_count = error_string[0].match(/[0-9]*/)[0].to_i
- end
- puts "found errors: " + current_error_count.to_s
- # find number of warnings
- if CHECK_WARNINGS == true
- warning_string = error_warning_string.match(/[0-9]* warning[s]?/)[0]
- current_warning_count = warning_string.match(/[0-9]*/)[0].to_i
- puts "found warnings: " + current_warning_count.to_s
- end
- # get previous error and warning counts from last successful build
- previous_results = false
- previous_lint_reports = Dir.glob(PREVIOUS_LINT_RESULTS_FILE)
- if previous_lint_reports.nil? ||
- previous_lint_reports.length == 0
- previous_lint_report = File.new(PREVIOUS_LINT_RESULTS_FILE, "w") # create for writing to later
- else
- previous_lint_report = String.new(previous_lint_reports[0])
- previous_error_warning_string = ""
- File.open previous_lint_report do |file|
- previous_error_warning_string = file.find { |line| line =~ /([0-9]* error[s]? and )?[0-9]* warning[s]?/ }
- end
- unless previous_error_warning_string.nil?
- previous_results = true
- previous_error_string = previous_error_warning_string.match(/[0-9]* error[s]?/)
- if previous_error_string.nil?
- previous_error_string = "0 errors"
- else
- previous_error_string = previous_error_string[0]
- end
- previous_error_count = previous_error_string.match(/[0-9]*/)[0].to_i
- puts "previous errors: " + previous_error_count.to_s
- if CHECK_WARNINGS == true
- previous_warning_string = previous_error_warning_string.match(/[0-9]* warning[s]?/)
- if previous_warning_string.nil?
- previous_warning_string = "0 warnings"
- else
- previous_warning_string = previous_warning_string[0]
- end
- previous_warning_count = previous_warning_string.match(/[0-9]*/)[0].to_i
- puts "previous warnings: " + previous_warning_count.to_s
- end
- end
- end
- # compare previous error count with current error count
- if previous_results == true &&
- current_error_count > previous_error_count
- puts "FAIL: error count increased"
- exit 1
- end
- # compare previous warning count with current warning count
- if CHECK_WARNINGS == true &&
- previous_results == true &&
- current_warning_count > previous_warning_count
- puts "FAIL: warning count increased"
- exit 1
- end
- # check if warning and error count stayed the same
- if previous_results == true &&
- current_error_count == previous_error_count &&
- current_warning_count == previous_warning_count
- puts "SUCCESS: count stayed the same"
- exit 2
- end
- # either error count or warning count DECREASED
- # write new results to file (will overwrite existing, or create new)
- File.write(previous_lint_report, "DO NOT TOUCH; GENERATED BY DRONE\n" + error_warning_string)
- # push changes to github (if this script is run locally, we don't want to overwrite git username and email, so save temporarily)
- previous_git_username, _ = Open3.capture2('git config user.name')
- previous_git_username = previous_git_username.strip
- previous_git_email, _ = Open3.capture3('git config user.email')
- previous_git_email = previous_git_email.strip
- # update git user name and email for this script
- system ("git config --local user.name '" + git_user + "'")
- system ("git config --local user.email 'android@nextcloud.com'")
- system ("git remote rm origin")
- system ("git remote add origin https://" + git_user + ":" + git_token + "@github.com/nextcloud/android")
- # add previous Lint result file to git
- system ('git add ' + PREVIOUS_LINT_RESULTS_FILE)
- # commit changes; Add "skip ci" so that we don't accidentally trigger another Drone build
- system ('git commit -sm "Drone: update Lint results to reflect reduced error/warning count [skip ci]" ')
- # push to origin
- system ('git push origin HEAD:' + git_branch)
- # restore previous git user name and email
- system("git config --local user.name '#{previous_git_username}'")
- system("git config --local user.email '#{previous_git_email}'")
- puts "SUCCESS: count was reduced"
- exit 0 # success
|