On 4.10.2010 22:45, Nicolas Palix wrote: > Improve user information with a new kind of comment > about semantic patch output. > > Fix spelling. > > Signed-off-by: Nicolas Palix <[email protected]> > Signed-off-by: Julia Lawall <[email protected]> > --- > scripts/coccicheck | 9 +++++++-- > 1 files changed, 7 insertions(+), 2 deletions(-) > > diff --git a/scripts/coccicheck b/scripts/coccicheck > index b8bcf1f..4655551 100755 > --- a/scripts/coccicheck > +++ b/scripts/coccicheck > @@ -25,7 +25,7 @@ fi > > if [ "$MODE" = "" ] ; then > if [ "$ONLINE" = "0" ] ; then > - echo 'You have not explicitly specify the mode to use. Fallback to > "report".' > + echo 'You have not explicitly specified 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 > @@ -52,7 +52,12 @@ coccinelle () { > > FILE=`echo $COCCI | sed "s|$srctree/||"` > > - echo "Processing `basename $COCCI` with option(s) \"$OPT\"" > + echo "Processing `basename $COCCI`" > + echo "with option(s) \"$OPT\"" > + echo 'Semantic patch information:' > + > + sed -e '/\/\/#/!d' -e 's|^//#||' $COCCI
sed -ne 's|^//#||p' $COCCI :) But I would put this after the "Message example to submit a patch". free.cocci has the information in this order and it makes more sense this way IMO (first general information, then details). And it would be nice to only display the "Semantic patch information:" line if there actually is some additional information in the semantic patch file. Otherwise, the series looks OK to me. > + > echo 'Message example to submit a patch:' > > sed -e '/\/\/\//!d' -e 's|^///||' $COCCI Michal _______________________________________________ Cocci mailing list [email protected] http://lists.diku.dk/mailman/listinfo/cocci (Web access from inside DIKUs LAN only)
