Re: [PATCH 1/3] Coccicheck: Add the rep+ctxt mode
Dne 11.10.2012 15:46, Nicolas Palix napsal(a): > Hi, > > On Thu, Oct 11, 2012 at 3:34 PM, Michal Marek wrote: >> On Thu, Sep 20, 2012 at 10:30:46PM +0200, Nicolas Palix wrote: >>> This adds a 'rep+ctxt' mode which prints the warning >>> message followed by the context. >>> >>> Signed-off-by: Nicolas Palix >>> --- >>> scripts/coccicheck | 3 +++ >>> 1 file changed, 3 insertions(+) >>> >>> diff --git a/scripts/coccicheck b/scripts/coccicheck >>> index 823e972..eb43e40 100755 >>> --- a/scripts/coccicheck >>> +++ b/scripts/coccicheck >>> @@ -95,6 +95,9 @@ coccinelle () { >>> $SPATCH -D report $FLAGS -sp_file $COCCI $OPT $OPTIONS -no_show_diff >>> || \ >>> $SPATCH -D context $FLAGS -sp_file $COCCI $OPT $OPTIONS >>> || \ >>> $SPATCH -D org $FLAGS -sp_file $COCCI $OPT $OPTIONS >>> -no_show_diff || exit 1 >> >> Your mailer broke this line into two, please try to avoid this next >> time. Also, please CC lkml when submitting patches. I applied the patch >> to kbuild.git#misc now. > > Ooops.. Sorry. I didn't notice. > > The patches are publish on github in my coccicheck branch. > https://github.com/npalix/linux/commits/coccicheck > > Do you prefer a pull request next time to avoid such things ? Yes, if you base the branch on some tagged release, preferably the latest -rc1. Michal -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/3] Coccicheck: Add the rep+ctxt mode
Hi, On Thu, Oct 11, 2012 at 3:34 PM, Michal Marek wrote: > On Thu, Sep 20, 2012 at 10:30:46PM +0200, Nicolas Palix wrote: >> This adds a 'rep+ctxt' mode which prints the warning >> message followed by the context. >> >> Signed-off-by: Nicolas Palix >> --- >> scripts/coccicheck | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/scripts/coccicheck b/scripts/coccicheck >> index 823e972..eb43e40 100755 >> --- a/scripts/coccicheck >> +++ b/scripts/coccicheck >> @@ -95,6 +95,9 @@ coccinelle () { >> $SPATCH -D report $FLAGS -sp_file $COCCI $OPT $OPTIONS -no_show_diff >> || \ >> $SPATCH -D context $FLAGS -sp_file $COCCI $OPT $OPTIONS >> || \ >> $SPATCH -D org $FLAGS -sp_file $COCCI $OPT $OPTIONS >> -no_show_diff || exit 1 > > Your mailer broke this line into two, please try to avoid this next > time. Also, please CC lkml when submitting patches. I applied the patch > to kbuild.git#misc now. Ooops.. Sorry. I didn't notice. The patches are publish on github in my coccicheck branch. https://github.com/npalix/linux/commits/coccicheck Do you prefer a pull request next time to avoid such things ? Nicolas > > Michal > >> +elif [ "$MODE" = "rep+ctxt" ] ; then >> + $SPATCH -D report $FLAGS -sp_file $COCCI $OPT $OPTIONS -no_show_diff >> && \ >> + $SPATCH -D context $FLAGS -sp_file $COCCI $OPT $OPTIONS >> || exit 1 >> else >> $SPATCH -D $MODE $FLAGS -sp_file $COCCI $OPT $OPTIONS || exit 1 >> fi >> -- >> 1.7.11.3 -- Nicolas Palix -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/3] Coccicheck: Add the rep+ctxt mode
On Thu, Sep 20, 2012 at 10:30:46PM +0200, Nicolas Palix wrote: > This adds a 'rep+ctxt' mode which prints the warning > message followed by the context. > > Signed-off-by: Nicolas Palix > --- > scripts/coccicheck | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/scripts/coccicheck b/scripts/coccicheck > index 823e972..eb43e40 100755 > --- a/scripts/coccicheck > +++ b/scripts/coccicheck > @@ -95,6 +95,9 @@ coccinelle () { > $SPATCH -D report $FLAGS -sp_file $COCCI $OPT $OPTIONS -no_show_diff > || \ > $SPATCH -D context $FLAGS -sp_file $COCCI $OPT $OPTIONS > || \ > $SPATCH -D org $FLAGS -sp_file $COCCI $OPT $OPTIONS > -no_show_diff || exit 1 Your mailer broke this line into two, please try to avoid this next time. Also, please CC lkml when submitting patches. I applied the patch to kbuild.git#misc now. Michal > +elif [ "$MODE" = "rep+ctxt" ] ; then > + $SPATCH -D report $FLAGS -sp_file $COCCI $OPT $OPTIONS -no_show_diff > && \ > + $SPATCH -D context $FLAGS -sp_file $COCCI $OPT $OPTIONS > || exit 1 > else > $SPATCH -D $MODE $FLAGS -sp_file $COCCI $OPT $OPTIONS || exit 1 > fi > -- > 1.7.11.3 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/3] Coccicheck: Add the rep+ctxt mode
On Thu, Sep 20, 2012 at 10:30:46PM +0200, Nicolas Palix wrote: This adds a 'rep+ctxt' mode which prints the warning message followed by the context. Signed-off-by: Nicolas Palix nicolas.pa...@imag.fr --- scripts/coccicheck | 3 +++ 1 file changed, 3 insertions(+) diff --git a/scripts/coccicheck b/scripts/coccicheck index 823e972..eb43e40 100755 --- a/scripts/coccicheck +++ b/scripts/coccicheck @@ -95,6 +95,9 @@ coccinelle () { $SPATCH -D report $FLAGS -sp_file $COCCI $OPT $OPTIONS -no_show_diff || \ $SPATCH -D context $FLAGS -sp_file $COCCI $OPT $OPTIONS || \ $SPATCH -D org $FLAGS -sp_file $COCCI $OPT $OPTIONS -no_show_diff || exit 1 Your mailer broke this line into two, please try to avoid this next time. Also, please CC lkml when submitting patches. I applied the patch to kbuild.git#misc now. Michal +elif [ $MODE = rep+ctxt ] ; then + $SPATCH -D report $FLAGS -sp_file $COCCI $OPT $OPTIONS -no_show_diff \ + $SPATCH -D context $FLAGS -sp_file $COCCI $OPT $OPTIONS || exit 1 else $SPATCH -D $MODE $FLAGS -sp_file $COCCI $OPT $OPTIONS || exit 1 fi -- 1.7.11.3 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/3] Coccicheck: Add the rep+ctxt mode
Hi, On Thu, Oct 11, 2012 at 3:34 PM, Michal Marek mma...@suse.cz wrote: On Thu, Sep 20, 2012 at 10:30:46PM +0200, Nicolas Palix wrote: This adds a 'rep+ctxt' mode which prints the warning message followed by the context. Signed-off-by: Nicolas Palix nicolas.pa...@imag.fr --- scripts/coccicheck | 3 +++ 1 file changed, 3 insertions(+) diff --git a/scripts/coccicheck b/scripts/coccicheck index 823e972..eb43e40 100755 --- a/scripts/coccicheck +++ b/scripts/coccicheck @@ -95,6 +95,9 @@ coccinelle () { $SPATCH -D report $FLAGS -sp_file $COCCI $OPT $OPTIONS -no_show_diff || \ $SPATCH -D context $FLAGS -sp_file $COCCI $OPT $OPTIONS || \ $SPATCH -D org $FLAGS -sp_file $COCCI $OPT $OPTIONS -no_show_diff || exit 1 Your mailer broke this line into two, please try to avoid this next time. Also, please CC lkml when submitting patches. I applied the patch to kbuild.git#misc now. Ooops.. Sorry. I didn't notice. The patches are publish on github in my coccicheck branch. https://github.com/npalix/linux/commits/coccicheck Do you prefer a pull request next time to avoid such things ? Nicolas Michal +elif [ $MODE = rep+ctxt ] ; then + $SPATCH -D report $FLAGS -sp_file $COCCI $OPT $OPTIONS -no_show_diff \ + $SPATCH -D context $FLAGS -sp_file $COCCI $OPT $OPTIONS || exit 1 else $SPATCH -D $MODE $FLAGS -sp_file $COCCI $OPT $OPTIONS || exit 1 fi -- 1.7.11.3 -- Nicolas Palix -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/3] Coccicheck: Add the rep+ctxt mode
Dne 11.10.2012 15:46, Nicolas Palix napsal(a): Hi, On Thu, Oct 11, 2012 at 3:34 PM, Michal Marek mma...@suse.cz wrote: On Thu, Sep 20, 2012 at 10:30:46PM +0200, Nicolas Palix wrote: This adds a 'rep+ctxt' mode which prints the warning message followed by the context. Signed-off-by: Nicolas Palix nicolas.pa...@imag.fr --- scripts/coccicheck | 3 +++ 1 file changed, 3 insertions(+) diff --git a/scripts/coccicheck b/scripts/coccicheck index 823e972..eb43e40 100755 --- a/scripts/coccicheck +++ b/scripts/coccicheck @@ -95,6 +95,9 @@ coccinelle () { $SPATCH -D report $FLAGS -sp_file $COCCI $OPT $OPTIONS -no_show_diff || \ $SPATCH -D context $FLAGS -sp_file $COCCI $OPT $OPTIONS || \ $SPATCH -D org $FLAGS -sp_file $COCCI $OPT $OPTIONS -no_show_diff || exit 1 Your mailer broke this line into two, please try to avoid this next time. Also, please CC lkml when submitting patches. I applied the patch to kbuild.git#misc now. Ooops.. Sorry. I didn't notice. The patches are publish on github in my coccicheck branch. https://github.com/npalix/linux/commits/coccicheck Do you prefer a pull request next time to avoid such things ? Yes, if you base the branch on some tagged release, preferably the latest -rc1. Michal -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/