mirror of
https://github.com/nextcloud/talk-android.git
synced 2024-11-26 23:25:20 +03:00
bad5b5f9bc
The updated libwebrtc will be load from the talk-clients-webrtc repository [1] and not from a dependency repository. The 'lint' task must now depend on the 'perBuild' task to make sure that the WebRTC library is downloaded from the configured URL. To get the new version to work a change is needed because of this announcement: > Google is planning to transition Chrome’s WebRTC implementation from the > current SDP format (called “Plan B”) to a standards conformant format > (“Unified Plan”, draft-ietf-rtcweb-jsep) over the next couple of quarters. > > - [2] Currently we are in phase 4 of the transition [3]. So actually 'Plan B' must be used to avoid bigger changes to our code base at this time. But 'Plan B' is depracted and will be removed in phase 5 of the transition [4]. Resolves: #1773 See: - [1] https://github.com/nextcloud-releases/talk-clients-webrtc - [2] https://webrtc.org/getting-started/unified-plan-transition-guide - [3] https://webrtc.org/getting-started/unified-plan-transition-guide#phase_4_make_%E2%80%9Cplan_b%E2%80%9D_throw - [4] https://webrtc.org/getting-started/unified-plan-transition-guide#phase_5_remove_%E2%80%9Cplan_b%E2%80%9D Signed-off-by: Tim Krüger <t@timkrueger.me>
199 lines
6.5 KiB
Ruby
199 lines
6.5 KiB
Ruby
## 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("app/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 --console=plain lintGplayDebug'
|
|
|
|
# confirm that Lint ran w/out error
|
|
result = $?.to_i
|
|
if result != 0
|
|
puts "FAIL: failed to run ./gradlew --console=plain lintGplayDebug"
|
|
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 '.'") # set email blank
|
|
system ("git remote rm origin")
|
|
system ("git remote add origin https://" + git_user + ":" + git_token + "@github.com/nextcloud/talk-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_COMMITTER_EMAIL" => "drone@nextcloud.com", "GIT_AUTHOR_EMAIL" => "drone@nextcloud.com"}, '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
|