Attachment 74122 Details for Bug 60095 – test to check that config.h is included

[patch] test to check that config.h is included

0002-Add-test-for-checking-that-each-source-file-includes.patch (text/plain), 1.09 KB, created by Adrian Johnson on 2013-02-03 08:09:17 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Adrian Johnson

Created: 2013-02-03 08:09:17 UTC

Size: 1.09 KB

patch

obsolete

>From 946dc79a28b7978b2e6aa3fd2b9d8a797e5fd55b Mon Sep 17 00:00:00 2001
>From: Adrian Johnson <ajohnson@redneon.com>
>Date: Sun, 3 Feb 2013 18:34:29 +1030
>Subject: [PATCH 2/2] Add test for checking that each source file includes
> config.h
>
>---
> test/check-includes.sh |   28 ++++++++++++++++++++++++++++
> 1 file changed, 28 insertions(+)
> create mode 100644 test/check-includes.sh
>
>diff --git a/test/check-includes.sh b/test/check-includes.sh
>new file mode 100644
>index 0000000..00023fe
>--- /dev/null
>+++ b/test/check-includes.sh
>@@ -0,0 +1,28 @@
>+#!/bin/sh
>+
>+# Check that first include of each source files is config.h
>+
>+echo "The following .cc files do not have config.h as the first include"
>+
>+
>+EXCLUDE_DIRS="../glib/demo/ ../qt4/demos/ ../qt4/tests/ ../cpp/tests/"
>+
>+
>+find .. $ -name "*.cc" -o -name "*.cpp" $ -exec awk -v exclude="$EXCLUDE_DIRS" '
>+{
>+    split(exclude, dirs, " ");
>+    for (d in dirs) {
>+        if (index(FILENAME, dirs[d]) == 1){
>+            exit;
>+        }
>+    }
>+}
>+
>+/^#include/ {
>+    if (!/[<"]config.h[>"]/)
>+        print FILENAME;
>+
>+    exit;
>+}
>+
>+' '{}' ';'
>-- 
>1.7.10.4
>

Actions: View | Splinter Review

Attachments on bug 60095: 74006 | 74007 | 74119 | 74120 | 74121 | 74122 | 74285

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.