Dangerfile

.travis.yml

@@ -0,0 +1,7 @@
+language: ruby
+
+before_script:
+  - gem install danger --version '~> 4.0'
+
+script:
+  - danger 

Dangerfile

@@ -0,0 +1,96 @@
+# Make it more obvious that a PR is a work in progress and shouldn't be merged yet
+if github.pr_title.include? "WIP"
+    warn("PR is classed as Work in Progress")
+end
+
+# Ensure a clean commits history
+if git.commits.any? { |c| c.message =~ /^Merge branch/ }
+  fail('Please rebase to get rid of the merge commits in this PR')
+end
+
+# Which parts of the repository did the PR touch?
+has_build_system_changes = !git.modified_files.grep(/make-rules\/*.mk/).empty?
+has_doc_changes = !git.modified_files.grep(/doc\//).empty?
+has_transform_changes = !git.modified_files.grep(/transforms\//).empty?
+has_tools_changes = !git.modified_files.grep(/tools\//).empty?
+has_component_changes = !git.modified_files.grep(/components\//).empty?
+
+# Checker functions
+def check_depend_mk(file_name)
+    return !`grep '^include.*depend.mk' #{file_name}`.empty?
+end
+
+def check_BUILD_PKG_DEPENDENCIES(file_name)
+    return !`grep '^BUILD_PKG_DEPENDENCIES = .*' #{file_name}`.empty?
+end
+
+def check_required_packages(file_name)
+    return `grep '^REQUIRED_PACKAGES \+= ' #{file_name}`.empty?
+end
+
+def check_ws_make_rules(file_name)
+    return !`grep '^include .*' #{file_name} | grep -v 'shared-macros.mk' | grep '$(WS_MAKE_RULES)'`.empty?
+end
+
+def check_copyright(file_name)
+    return !`grep '^# Copyright 2017 <contributor>' #{file_name}`.empty?
+end
+
+# Helper functions
+def check_base_file(file_name)
+    github.review.warn('Please add your copyright to ' + file_name) if check_copyright(file_name)
+end
+
+def check_component_makefile(file_name)
+
+    check_base_file(file_name)
+
+    github.review.warn('Remove depend.mk line from ' + file_name) if check_depend_mk(file_name)
+    github.review.warn('Remove BUILD_PKG_DEPENDENCIES line from ' + file_name) if check_BUILD_PKG_DEPENDENCIES(file_name)
+    github.review.warn('Switch Makefile include to use $(WS_MAKE_RULES) variable in ' + file_name) if check_ws_make_rules(file_name)
+    github.review.warn('Consider adding REQUIRED_PACKAGES to ' + file_name) if check_required_packages(file_name)
+end
+
+github.review.start
+
+# Component-related checks
+if has_component_changes
+    git.added_files.each do |file_name|
+        is_component_makefile = file_name.match(/Makefile$/)
+        is_component_ips_manifest = file_name.match(/.p5m$/)
+        is_component_patch = file_name.match(/.patch$/)
+        is_component_license = file_name.match(/.license$/)
+    end
+
+    git.modified_files.each do |file_name|
+        is_dangerfile = file_name.match(/Dangerfile$/)
+        is_component_makefile = file_name.match(/Makefile$/)
+        is_component_ips_manifest = file_name.match(/.p5m$/)
+        is_component_patch = file_name.match(/.patch$/)
+        is_component_license = file_name.match(/.license$/)
+
+        if is_component_makefile
+            check_component_makefile(file_name)
+            github.review.warn('Bump COMPONENT_REVISION or COMPONENT_VERSION in ' + file_name + ' as it was modified')
+        end
+
+        if is_component_ips_manifest
+            component_makefile = File.join(File.dirname(file_name), 'Makefile')
+            github.review.warn('Bump COMPONENT_REVISION or COMPONENT_VERSION in ' + component_makefile + ' as IPS manifest ' + file_name + ' was modified')
+        end
+    end
+
+    git.deleted_files.each do |file_name|
+        is_dangerfile = file_name.match(/Dangerfile$/)
+        is_component_makefile = file_name.match(/Makefile$/)
+        is_component_ips_manifest = file_name.match(/.p5m$/)
+        is_component_patch = file_name.match(/.patch$/)
+        is_component_license = file_name.match(/.license$/)
+
+        if is_component_patch
+            github.review.warn('Bump COMPONENT_REVISION or COMPONENT_VERSION as ' + file_name + ' was dropped')
+        end
+    end
+end
+
+github.review.submit

components/shell/bash-completion/Makefile

@@ -44,5 +44,3 @@ install: $(INSTALL_32)
 BUILD_PKG_DEPENDENCIES = $(BUILD_TOOLS)
 
 include $(WS_MAKE_RULES)/depend.mk
-
-REQUIRED_PACKAGES += shell/bash

components/web/nginx/Makefile

@@ -15,7 +15,7 @@
 include ../../../make-rules/shared-macros.mk
 
 COMPONENT_NAME=		nginx
-COMPONENT_VERSION=	1.11.12
+COMPONENT_VERSION=	1.11.13
 COMPONENT_LICENSE=	BSD
 COMPONENT_PROJECT_URL=	http://nginx.net/
 COMPONENT_SUMMARY=	Nginx Webserver