[arch-projects] [namcap][PATCH] add two rules which check if a PKGBUILD for vcs packages contains the correct makedepends and conflicts array
---
Namcap/rules/__init__.py | 3 ++-
Namcap/rules/vcspackages.py | 43 +++++++++++++++++++++++++++++++++++++++++++
namcap-tags | 3 +++
3 files changed, 48 insertions(+), 1 deletion(-)
create mode 100644 Namcap/rules/vcspackages.py
diff --git a/Namcap/rules/__init__.py b/Namcap/rules/__init__.py
index f7780d2..62c2617 100644
--- a/Namcap/rules/__init__.py
+++ b/Namcap/rules/__init__.py
@@ -60,7 +60,8 @@ from . import (
pkginfo,
pkgnameindesc,
sfurl,
- splitpkgbuild
+ splitpkgbuild,
+ vcspackages
)
all_rules = {}
diff --git a/Namcap/rules/vcspackages.py b/Namcap/rules/vcspackages.py
new file mode 100644
index 0000000..041ddf9
--- /dev/null
+++ b/Namcap/rules/vcspackages.py
@@ -0,0 +1,43 @@
+#
+# namcap rules - vcspackages
+# Copyright (C) 2013 Jelle van der Waa
2013/9/8 Jelle van der Waa
+ def analyze(self, pkginfo, tar): + if 'name' in pkginfo: + if any(pkginfo["name"].endswith(vcs) for vcs in vcspackages) and not "conflicts" in pkginfo: + self.warnings.append(("vcs-package-conflicts", ())) +
Python trick: you can write if pkginfo["name"].endswith(tuple(vcspackages)) and "conflicts" not in pkginfo: ... -- Rémy.
Adds a unit test
---
Namcap/rules/__init__.py | 3 ++-
Namcap/rules/vcspackages.py | 43 +++++++++++++++++++++++++++++++++++++++++++
namcap-tags | 3 +++
3 files changed, 48 insertions(+), 1 deletion(-)
create mode 100644 Namcap/rules/vcspackages.py
diff --git a/Namcap/rules/__init__.py b/Namcap/rules/__init__.py
index f7780d2..62c2617 100644
--- a/Namcap/rules/__init__.py
+++ b/Namcap/rules/__init__.py
@@ -60,7 +60,8 @@ from . import (
pkginfo,
pkgnameindesc,
sfurl,
- splitpkgbuild
+ splitpkgbuild,
+ vcspackages
)
all_rules = {}
diff --git a/Namcap/rules/vcspackages.py b/Namcap/rules/vcspackages.py
new file mode 100644
index 0000000..32bbbbb
--- /dev/null
+++ b/Namcap/rules/vcspackages.py
@@ -0,0 +1,43 @@
+#
+# namcap rules - vcspackages
+# Copyright (C) 2013 Jelle van der Waa
participants (2)
-
Jelle van der Waa
-
Rémy Oudompheng