summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMattes D <github@xoft.cz>2016-07-02 00:17:18 +0200
committerMattes D <github@xoft.cz>2016-07-02 00:30:22 +0200
commit0eea5c9fc5c064036cf54f833266667ed8e07885 (patch)
tree24c6c10c53303c6a8da0890f5a0d33199bdf8c07
parentMerge pull request #3246 from cuberite/revert_redstone_handler (diff)
downloadcuberite-0eea5c9fc5c064036cf54f833266667ed8e07885.tar
cuberite-0eea5c9fc5c064036cf54f833266667ed8e07885.tar.gz
cuberite-0eea5c9fc5c064036cf54f833266667ed8e07885.tar.bz2
cuberite-0eea5c9fc5c064036cf54f833266667ed8e07885.tar.lz
cuberite-0eea5c9fc5c064036cf54f833266667ed8e07885.tar.xz
cuberite-0eea5c9fc5c064036cf54f833266667ed8e07885.tar.zst
cuberite-0eea5c9fc5c064036cf54f833266667ed8e07885.zip
-rw-r--r--circle.yml4
-rw-r--r--src/Bindings/CheckBindingsDependencies.lua108
2 files changed, 110 insertions, 2 deletions
diff --git a/circle.yml b/circle.yml
index 341c87a1d..f13d02ce8 100644
--- a/circle.yml
+++ b/circle.yml
@@ -8,5 +8,5 @@ dependencies:
test:
override:
- - cd src && find . -name \*.cpp -or -name \*.h > AllFiles.lst
- - cd src && lua CheckBasicStyle.lua
+ - cd src && find . -name \*.cpp -or -name \*.h > AllFiles.lst && lua CheckBasicStyle.lua
+ - cd src/Bindings && lua CheckBindingsDependencies.lua
diff --git a/src/Bindings/CheckBindingsDependencies.lua b/src/Bindings/CheckBindingsDependencies.lua
new file mode 100644
index 000000000..c0565ead8
--- /dev/null
+++ b/src/Bindings/CheckBindingsDependencies.lua
@@ -0,0 +1,108 @@
+-- CheckBindingsDependencies.lua
+
+-- This script checks whether all files listed in AllToLua.pkg are also in the dependencies for bindings regeneration.
+-- This script is run as part of the CircleCI tests
+
+
+
+
+
+--- Filenames that should be ignored in the AllToLua.pkg file:
+-- Dictionary of "filename" -> true for each ignored filename
+local g_ShouldIgnorePkg =
+{
+ -- ["../IniFile.h"] = true,
+}
+
+--- Filenames that should be ignored in the CMakeLists.txt file:
+-- Dictionary of "filename" -> true for each ignored filename
+local g_ShouldIgnoreCMake =
+{
+ ["tolua"] = true,
+ ["../Bindings/AllToLua.pkg"] = true,
+ ["../Bindings/BindingsProcessor.lua"] = true,
+}
+
+
+
+
+
+--- Returns a sorted list of all files listed in AllToLua.pkg
+-- The returned table has both an array part (list of files) and a dictionary part ("filename" -> true)
+local function getAllToLuaPkgFiles()
+ local res = {}
+ for line in io.lines("AllToLua.pkg") do
+ line:gsub("$cfile \"(.+)\"", -- Parse each line with a $cfile directive
+ function (a_FileName)
+ if (g_ShouldIgnorePkg[a_FileName]) then
+ return
+ end
+ a_FileName = a_FileName:gsub("../Bindings/", "") -- Normalize the path
+ table.insert(res, a_FileName)
+ res[a_FileName] = true
+ end
+ )
+ end
+ table.sort(res)
+ return res
+end
+
+
+
+
+
+--- Returns a sorted list of all files listed as dependencies in CMakeLists.txt
+-- The returned table has both an array part (list of files) and a dictionary part ("filename" -> true)
+local function getCMakeListsFiles()
+ local f = assert(io.open("CMakeLists.txt", "r"))
+ local contents = f:read("*all")
+ f:close()
+ local res = {}
+ contents:gsub("set%s*(%b())", -- Process each CMake's "set" statement
+ function (a_SetStatement)
+ if not(a_SetStatement:find("%(BINDING_DEPENDENCIES")) then
+ return
+ end
+ -- This is the statement setting the dependencies, parse the files:
+ a_SetStatement:gsub("%s(%S+)%s",
+ function (a_FileName)
+ if (g_ShouldIgnoreCMake[a_FileName]) then
+ return
+ end
+ a_FileName = a_FileName:gsub("../Bindings/", "") -- Normalize the path
+ table.insert(res, a_FileName)
+ res[a_FileName] = true
+ end
+ )
+ end
+ )
+ table.sort(res)
+ return res
+end
+
+
+
+
+
+-- Check each set of files against the other:
+local pkgFiles = getAllToLuaPkgFiles()
+local cmakeFiles = getCMakeListsFiles()
+local numMissingFiles = 0
+for _, fnam in ipairs(pkgFiles) do
+ if not(cmakeFiles[fnam]) then
+ io.stderr:write("Bindings dependency file ", fnam, " is not listed in src/Bindings/CMakeLists.txt\n")
+ numMissingFiles = numMissingFiles + 1
+ end
+end
+for _, fnam in ipairs(cmakeFiles) do
+ if not(pkgFiles[fnam]) then
+ io.stderr:write("Bindings dependency file ", fnam, " is not listed in src/Bindings/AllToLua.pkg\n")
+ numMissingFiles = numMissingFiles + 1
+ end
+end
+
+-- If any mismatch was found, exit with an error code:
+if (numMissingFiles > 0) then
+ io.stderr:write("Bindings dependency mismatches found: ", numMissingFiles, "\n")
+ os.exit(1)
+end