[Buildroot] [PATCH 1/1] check-package: Flag usage of '\t \\$'.

[Buildroot] [PATCH 1/1] check-package: Flag usage of '\t \\$'.

[Nicolas Cavallari]

check-package would flag tabs before a backslash ('\t\\'),
two spaces before a backslash ('  \\') but would not flag a tab before space
before backslash ('\t \\'), allowing someone to bypass the check.

Signed-off-by: Nicolas Cavallari <nicolas.cavallari@green-communications.fr>

---
There is no occurence of this in buildroot.
 utils/checkpackagelib/lib_mk.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/utils/checkpackagelib/lib_mk.py b/utils/checkpackagelib/lib_mk.py
index 2df2e1a884..cfd4d05117 100644
--- a/utils/checkpackagelib/lib_mk.py
+++ b/utils/checkpackagelib/lib_mk.py
@@ -123,7 +123,7 @@ class RemoveDefaultPackageSourceVariable(_CheckFunction):
 
 
 class SpaceBeforeBackslash(_CheckFunction):
-    TAB_OR_MULTIPLE_SPACES_BEFORE_BACKSLASH = re.compile(r"^.*(  |\t)\\$")
+    TAB_OR_MULTIPLE_SPACES_BEFORE_BACKSLASH = re.compile(r"^.*(  |\t ?)\\$")
 
     def check_line(self, lineno, text):
         if self.TAB_OR_MULTIPLE_SPACES_BEFORE_BACKSLASH.match(text.rstrip()):
-- 
2.16.2

[Buildroot] [PATCH 1/1] check-package: Flag usage of '\t \\$'.

[Ricardo Martincoski]

Hello,

> check-package would flag tabs before a backslash ('\t\\'),
> two spaces before a backslash ('  \\') but would not flag a tab before space
> before backslash ('\t \\'), allowing someone to bypass the check.
> 
> Signed-off-by: Nicolas Cavallari <nicolas.cavallari@green-communications.fr>

Reviewed-by: Ricardo Martincoski <ricardo.martincoski@datacom.ind.br>

Regards,
Ricardo

[Buildroot] [PATCH 1/1] check-package: Flag usage of '\t \\$'.

[Thomas Petazzoni]

Hello,

On Wed,  7 Mar 2018 10:48:12 +0100, Nicolas Cavallari wrote:
> check-package would flag tabs before a backslash ('\t\\'),
> two spaces before a backslash ('  \\') but would not flag a tab before space
> before backslash ('\t \\'), allowing someone to bypass the check.
> 
> Signed-off-by: Nicolas Cavallari <nicolas.cavallari@green-communications.fr>
> 
> ---
> There is no occurence of this in buildroot.
>  utils/checkpackagelib/lib_mk.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com