* [PATCH] Add support for the C variable in the coccicheck script
@ 2010-06-13 7:26 Nicolas Palix
0 siblings, 0 replies; 3+ messages in thread
From: Nicolas Palix @ 2010-06-13 7:26 UTC (permalink / raw)
To: Julia Lawall, Gilles Muller, Nicolas Palix, Michal Marek,
Sam Ravnborg, Joerg Roedel, cocci, linux-kernel
Cc: Nicolas Palix
This patch makes it possible to use the Coccinelle checker with the C
variable of the build system. To check only newly edited code, the
following command may be used:
'make C={1,2} CHECK="scripts/coccicheck"'
This runs every semantic patch in scripts/coccinelle by default. The
COCCI variable may additionally be used to only apply a single
semantic patch.
Signed-off-by: Nicolas Palix <npalix@diku.dk>
Signed-off-by: Julia Lawall <julia@diku.dk>
---
scripts/coccicheck | 76 +++++++++++++++++++++++++++++++++++-----------------
1 files changed, 51 insertions(+), 25 deletions(-)
diff --git a/scripts/coccicheck b/scripts/coccicheck
index 037424b..b8bcf1f 100755
--- a/scripts/coccicheck
+++ b/scripts/coccicheck
@@ -2,53 +2,79 @@
SPATCH="`which ${SPATCH:=spatch}`"
+if [ "$C" = "1" -o "$C" = "2" ]; then
+ ONLINE=1
+
+# This requires Coccinelle >= 0.2.3
+# FLAGS="-ignore_unknown_options -very_quiet"
+# OPTIONS=$*
+
+# Workaround for Coccinelle < 0.2.3
+ FLAGS="-I $srctree/include -very_quiet"
+ shift $(( $# - 1 ))
+ OPTIONS=$1
+else
+ ONLINE=0
+ FLAGS="-very_quiet"
+fi
+
if [ ! -x "$SPATCH" ]; then
echo 'spatch is part of the Coccinelle project and is available at http://coccinelle.lip6.fr/'
exit 1
fi
if [ "$MODE" = "" ] ; then
- echo 'You have not explicitly specify the mode to use. Fallback to "report".'
- echo 'You can specify the mode with "make coccicheck MODE=<mode>"'
- echo 'Available modes are: report, patch, context, org'
+ if [ "$ONLINE" = "0" ] ; then
+ echo 'You have not explicitly specify the mode to use. Fallback to "report".'
+ echo 'You can specify the mode with "make coccicheck MODE=<mode>"'
+ echo 'Available modes are: report, patch, context, org'
+ fi
MODE="report"
fi
-echo ''
-echo 'Please check for false positives in the output before submitting a patch.'
-echo 'When using "patch" mode, carefully review the patch before submitting it.'
-echo ''
+if [ "$ONLINE" = "0" ] ; then
+ echo ''
+ echo 'Please check for false positives in the output before submitting a patch.'
+ echo 'When using "patch" mode, carefully review the patch before submitting it.'
+ echo ''
+fi
-function coccinelle {
+coccinelle () {
COCCI="$1"
- DIR="$2"
OPT=`grep "Option" $COCCI | cut -d':' -f2`
- FILE=`echo $COCCI | sed "s|$DIR/||"`
- echo "Processing `basename $COCCI` with option(s) \"$OPT\""
- echo 'Message example to submit a patch:'
+# The option '-parse_cocci' can be used to syntaxically check the SmPL files.
+#
+# $SPATCH -D $MODE $FLAGS -parse_cocci $COCCI $OPT > /dev/null
- sed -e '/\/\/\//!d' -e 's|^///||' $COCCI
+ if [ "$ONLINE" = "0" ] ; then
- echo ' The semantic patch that makes this change is available'
- echo " in $FILE."
- echo ''
- echo ' More information about semantic patching is available at'
- echo ' http://coccinelle.lip6.fr/'
- echo ''
+ FILE=`echo $COCCI | sed "s|$srctree/||"`
-# The option '-parse_cocci' can be used to syntaxically check the SmPL files.
-#
-# $SPATCH -D $MODE -very_quiet -parse_cocci $COCCI $OPT > /dev/null
+ echo "Processing `basename $COCCI` with option(s) \"$OPT\""
+ echo 'Message example to submit a patch:'
+
+ sed -e '/\/\/\//!d' -e 's|^///||' $COCCI
+
+ echo ' The semantic patch that makes this change is available'
+ echo " in $FILE."
+ echo ''
+ echo ' More information about semantic patching is available at'
+ echo ' http://coccinelle.lip6.fr/'
+ echo ''
+
+ $SPATCH -D $MODE $FLAGS -sp_file $COCCI $OPT -dir $srctree || exit 1
+ else
+ $SPATCH -D $MODE $FLAGS -sp_file $COCCI $OPT $OPTIONS || exit 1
+ fi
- $SPATCH -D $MODE -very_quiet -sp_file $COCCI $OPT -dir $DIR
}
if [ "$COCCI" = "" ] ; then
for f in `find $srctree/scripts/coccinelle/ -name '*.cocci' -type f | sort`; do
- coccinelle $f $srctree;
+ coccinelle $f
done
else
- coccinelle $COCCI $srctree
+ coccinelle $COCCI
fi
--
1.7.0.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] Add support for the C variable in the coccicheck script
@ 2010-06-30 21:50 ` Nicolas Palix
0 siblings, 0 replies; 3+ messages in thread
From: Nicolas Palix @ 2010-06-30 21:50 UTC (permalink / raw)
To: Randy Dunlap, Roland Dreier, Joe Perches, Andrew Morton,
David S. Miller, Michal Marek, Sam Ravnborg, Julia Lawall,
Gilles Muller, linux-kernel, linux-kbuild, cocci, Wolfram Sang,
Kernel Janitors, Américo Wang, Andy Isaacson,
Linus Torvalds, Pekka Enberg, walter harms, Joerg Roedel,
Jan Engelhardt
Cc: Nicolas Palix
This patch makes it possible to use the Coccinelle checker with the C
variable of the build system. To check only newly edited code, the
following command may be used:
'make C={1,2} CHECK="scripts/coccicheck"'
This runs every semantic patch in scripts/coccinelle by default. The
COCCI variable may additionally be used to only apply a single
semantic patch.
Signed-off-by: Nicolas Palix <npalix@diku.dk>
Signed-off-by: Julia Lawall <julia@diku.dk>
---
scripts/coccicheck | 76 +++++++++++++++++++++++++++++++++++-----------------
1 files changed, 51 insertions(+), 25 deletions(-)
diff --git a/scripts/coccicheck b/scripts/coccicheck
index 037424b..b8bcf1f 100755
--- a/scripts/coccicheck
+++ b/scripts/coccicheck
@@ -2,53 +2,79 @@
SPATCH="`which ${SPATCH:=spatch}`"
+if [ "$C" = "1" -o "$C" = "2" ]; then
+ ONLINE=1
+
+# This requires Coccinelle >= 0.2.3
+# FLAGS="-ignore_unknown_options -very_quiet"
+# OPTIONS=$*
+
+# Workaround for Coccinelle < 0.2.3
+ FLAGS="-I $srctree/include -very_quiet"
+ shift $(( $# - 1 ))
+ OPTIONS=$1
+else
+ ONLINE=0
+ FLAGS="-very_quiet"
+fi
+
if [ ! -x "$SPATCH" ]; then
echo 'spatch is part of the Coccinelle project and is available at http://coccinelle.lip6.fr/'
exit 1
fi
if [ "$MODE" = "" ] ; then
- echo 'You have not explicitly specify the mode to use. Fallback to "report".'
- echo 'You can specify the mode with "make coccicheck MODE=<mode>"'
- echo 'Available modes are: report, patch, context, org'
+ if [ "$ONLINE" = "0" ] ; then
+ echo 'You have not explicitly specify the mode to use. Fallback to "report".'
+ echo 'You can specify the mode with "make coccicheck MODE=<mode>"'
+ echo 'Available modes are: report, patch, context, org'
+ fi
MODE="report"
fi
-echo ''
-echo 'Please check for false positives in the output before submitting a patch.'
-echo 'When using "patch" mode, carefully review the patch before submitting it.'
-echo ''
+if [ "$ONLINE" = "0" ] ; then
+ echo ''
+ echo 'Please check for false positives in the output before submitting a patch.'
+ echo 'When using "patch" mode, carefully review the patch before submitting it.'
+ echo ''
+fi
-function coccinelle {
+coccinelle () {
COCCI="$1"
- DIR="$2"
OPT=`grep "Option" $COCCI | cut -d':' -f2`
- FILE=`echo $COCCI | sed "s|$DIR/||"`
- echo "Processing `basename $COCCI` with option(s) \"$OPT\""
- echo 'Message example to submit a patch:'
+# The option '-parse_cocci' can be used to syntaxically check the SmPL files.
+#
+# $SPATCH -D $MODE $FLAGS -parse_cocci $COCCI $OPT > /dev/null
- sed -e '/\/\/\//!d' -e 's|^///||' $COCCI
+ if [ "$ONLINE" = "0" ] ; then
- echo ' The semantic patch that makes this change is available'
- echo " in $FILE."
- echo ''
- echo ' More information about semantic patching is available at'
- echo ' http://coccinelle.lip6.fr/'
- echo ''
+ FILE=`echo $COCCI | sed "s|$srctree/||"`
-# The option '-parse_cocci' can be used to syntaxically check the SmPL files.
-#
-# $SPATCH -D $MODE -very_quiet -parse_cocci $COCCI $OPT > /dev/null
+ echo "Processing `basename $COCCI` with option(s) \"$OPT\""
+ echo 'Message example to submit a patch:'
+
+ sed -e '/\/\/\//!d' -e 's|^///||' $COCCI
+
+ echo ' The semantic patch that makes this change is available'
+ echo " in $FILE."
+ echo ''
+ echo ' More information about semantic patching is available at'
+ echo ' http://coccinelle.lip6.fr/'
+ echo ''
+
+ $SPATCH -D $MODE $FLAGS -sp_file $COCCI $OPT -dir $srctree || exit 1
+ else
+ $SPATCH -D $MODE $FLAGS -sp_file $COCCI $OPT $OPTIONS || exit 1
+ fi
- $SPATCH -D $MODE -very_quiet -sp_file $COCCI $OPT -dir $DIR
}
if [ "$COCCI" = "" ] ; then
for f in `find $srctree/scripts/coccinelle/ -name '*.cocci' -type f | sort`; do
- coccinelle $f $srctree;
+ coccinelle $f
done
else
- coccinelle $COCCI $srctree
+ coccinelle $COCCI
fi
--
1.7.0.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] Add support for the C variable in the coccicheck script
@ 2010-06-30 21:50 ` Nicolas Palix
0 siblings, 0 replies; 3+ messages in thread
From: Nicolas Palix @ 2010-06-30 21:50 UTC (permalink / raw)
To: Randy Dunlap, Roland Dreier, Joe Perches, Andrew Morton,
David S. Miller, Michal Marek, Sam Ravnborg, Julia Lawall,
Gilles Muller, linux-kernel, linux-kbuild, cocci, Wolfram Sang,
Kernel Janitors, Américo Wang, Andy Isaacson,
Linus Torvalds, Pekka Enberg, walter harms, Joerg Roedel,
Jan Engelhardt
Cc: Nicolas Palix
This patch makes it possible to use the Coccinelle checker with the C
variable of the build system. To check only newly edited code, the
following command may be used:
'make C={1,2} CHECK="scripts/coccicheck"'
This runs every semantic patch in scripts/coccinelle by default. The
COCCI variable may additionally be used to only apply a single
semantic patch.
Signed-off-by: Nicolas Palix <npalix@diku.dk>
Signed-off-by: Julia Lawall <julia@diku.dk>
---
scripts/coccicheck | 76 +++++++++++++++++++++++++++++++++++-----------------
1 files changed, 51 insertions(+), 25 deletions(-)
diff --git a/scripts/coccicheck b/scripts/coccicheck
index 037424b..b8bcf1f 100755
--- a/scripts/coccicheck
+++ b/scripts/coccicheck
@@ -2,53 +2,79 @@
SPATCH="`which ${SPATCH:=spatch}`"
+if [ "$C" = "1" -o "$C" = "2" ]; then
+ ONLINE=1
+
+# This requires Coccinelle >= 0.2.3
+# FLAGS="-ignore_unknown_options -very_quiet"
+# OPTIONS=$*
+
+# Workaround for Coccinelle < 0.2.3
+ FLAGS="-I $srctree/include -very_quiet"
+ shift $(( $# - 1 ))
+ OPTIONS=$1
+else
+ ONLINE=0
+ FLAGS="-very_quiet"
+fi
+
if [ ! -x "$SPATCH" ]; then
echo 'spatch is part of the Coccinelle project and is available at http://coccinelle.lip6.fr/'
exit 1
fi
if [ "$MODE" = "" ] ; then
- echo 'You have not explicitly specify the mode to use. Fallback to "report".'
- echo 'You can specify the mode with "make coccicheck MODE=<mode>"'
- echo 'Available modes are: report, patch, context, org'
+ if [ "$ONLINE" = "0" ] ; then
+ echo 'You have not explicitly specify the mode to use. Fallback to "report".'
+ echo 'You can specify the mode with "make coccicheck MODE=<mode>"'
+ echo 'Available modes are: report, patch, context, org'
+ fi
MODE="report"
fi
-echo ''
-echo 'Please check for false positives in the output before submitting a patch.'
-echo 'When using "patch" mode, carefully review the patch before submitting it.'
-echo ''
+if [ "$ONLINE" = "0" ] ; then
+ echo ''
+ echo 'Please check for false positives in the output before submitting a patch.'
+ echo 'When using "patch" mode, carefully review the patch before submitting it.'
+ echo ''
+fi
-function coccinelle {
+coccinelle () {
COCCI="$1"
- DIR="$2"
OPT=`grep "Option" $COCCI | cut -d':' -f2`
- FILE=`echo $COCCI | sed "s|$DIR/||"`
- echo "Processing `basename $COCCI` with option(s) \"$OPT\""
- echo 'Message example to submit a patch:'
+# The option '-parse_cocci' can be used to syntaxically check the SmPL files.
+#
+# $SPATCH -D $MODE $FLAGS -parse_cocci $COCCI $OPT > /dev/null
- sed -e '/\/\/\//!d' -e 's|^///||' $COCCI
+ if [ "$ONLINE" = "0" ] ; then
- echo ' The semantic patch that makes this change is available'
- echo " in $FILE."
- echo ''
- echo ' More information about semantic patching is available at'
- echo ' http://coccinelle.lip6.fr/'
- echo ''
+ FILE=`echo $COCCI | sed "s|$srctree/||"`
-# The option '-parse_cocci' can be used to syntaxically check the SmPL files.
-#
-# $SPATCH -D $MODE -very_quiet -parse_cocci $COCCI $OPT > /dev/null
+ echo "Processing `basename $COCCI` with option(s) \"$OPT\""
+ echo 'Message example to submit a patch:'
+
+ sed -e '/\/\/\//!d' -e 's|^///||' $COCCI
+
+ echo ' The semantic patch that makes this change is available'
+ echo " in $FILE."
+ echo ''
+ echo ' More information about semantic patching is available at'
+ echo ' http://coccinelle.lip6.fr/'
+ echo ''
+
+ $SPATCH -D $MODE $FLAGS -sp_file $COCCI $OPT -dir $srctree || exit 1
+ else
+ $SPATCH -D $MODE $FLAGS -sp_file $COCCI $OPT $OPTIONS || exit 1
+ fi
- $SPATCH -D $MODE -very_quiet -sp_file $COCCI $OPT -dir $DIR
}
if [ "$COCCI" = "" ] ; then
for f in `find $srctree/scripts/coccinelle/ -name '*.cocci' -type f | sort`; do
- coccinelle $f $srctree;
+ coccinelle $f
done
else
- coccinelle $COCCI $srctree
+ coccinelle $COCCI
fi
--
1.7.0.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-06-30 21:51 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-06-13 7:26 [PATCH] Add support for the C variable in the coccicheck script Nicolas Palix
2010-06-30 21:50 Nicolas Palix
2010-06-30 21:50 ` Nicolas Palix
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.