* scripts/checkpatch.pl is deprecated
@ 2017-01-11 19:35 Vicente Bergas
2017-01-11 20:07 ` Uwe Kleine-König
0 siblings, 1 reply; 8+ messages in thread
From: Vicente Bergas @ 2017-01-11 19:35 UTC (permalink / raw)
To: barebox
Hello,
when using scripts/checkpatch.pl it may report:
Unescaped left brace in regex is deprecated,
passed through in regex at scripts/checkpatch.pl line 2267.
This error is shown if a patch contains a left brace with
no preceding space like in:
if(condition){
Should the script be fixed or updated?
Regards,
Vicente.
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: scripts/checkpatch.pl is deprecated 2017-01-11 19:35 scripts/checkpatch.pl is deprecated Vicente Bergas @ 2017-01-11 20:07 ` Uwe Kleine-König 2017-01-12 10:27 ` Ian Abbott 0 siblings, 1 reply; 8+ messages in thread From: Uwe Kleine-König @ 2017-01-11 20:07 UTC (permalink / raw) To: Vicente Bergas; +Cc: barebox On Wed, Jan 11, 2017 at 08:35:11PM +0100, Vicente Bergas wrote: > Hello, > when using scripts/checkpatch.pl it may report: > Unescaped left brace in regex is deprecated, > passed through in regex at scripts/checkpatch.pl line 2267. I bet this scripts originates from the kernel. So I'd recommend to test if their version is affected to. If yes, report to them, if not, update barebox to latest Linux version. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: scripts/checkpatch.pl is deprecated 2017-01-11 20:07 ` Uwe Kleine-König @ 2017-01-12 10:27 ` Ian Abbott 2017-01-15 19:41 ` Vicente Bergas 2017-01-15 19:42 ` Vicente Bergas 0 siblings, 2 replies; 8+ messages in thread From: Ian Abbott @ 2017-01-12 10:27 UTC (permalink / raw) To: Uwe Kleine-König, Vicente Bergas; +Cc: barebox On 11/01/17 20:07, Uwe Kleine-König wrote: > On Wed, Jan 11, 2017 at 08:35:11PM +0100, Vicente Bergas wrote: >> Hello, >> when using scripts/checkpatch.pl it may report: >> Unescaped left brace in regex is deprecated, >> passed through in regex at scripts/checkpatch.pl line 2267. > > I bet this scripts originates from the kernel. So I'd recommend to test > if their version is affected to. If yes, report to them, if not, update > barebox to latest Linux version. In the Linux kernel, this was fixed in commit 6b8c69e4384b0bcf1936c2137804840212daaf69 ("checkpatch: fix another left brace warning"), which made it into the the 4.6 kernel. -- -=( Ian Abbott @ MEV Ltd. E-mail: <abbotti@mev.co.uk> )=- -=( Web: http://www.mev.co.uk/ )=- _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: scripts/checkpatch.pl is deprecated 2017-01-12 10:27 ` Ian Abbott @ 2017-01-15 19:41 ` Vicente Bergas 2017-01-15 19:42 ` Vicente Bergas 1 sibling, 0 replies; 8+ messages in thread From: Vicente Bergas @ 2017-01-15 19:41 UTC (permalink / raw) To: Ian Abbott; +Cc: barebox, Uwe Kleine-König On Thu, Jan 12, 2017 at 11:27 AM, Ian Abbott <abbotti@mev.co.uk> wrote: > On 11/01/17 20:07, Uwe Kleine-König wrote: >> >> On Wed, Jan 11, 2017 at 08:35:11PM +0100, Vicente Bergas wrote: >>> >>> Hello, >>> when using scripts/checkpatch.pl it may report: >>> Unescaped left brace in regex is deprecated, >>> passed through in regex at scripts/checkpatch.pl line 2267. >> >> >> I bet this scripts originates from the kernel. So I'd recommend to test >> if their version is affected to. If yes, report to them, if not, update >> barebox to latest Linux version. > > > In the Linux kernel, this was fixed in commit > 6b8c69e4384b0bcf1936c2137804840212daaf69 ("checkpatch: fix another left > brace warning"), which made it into the the 4.6 kernel. > > -- > -=( Ian Abbott @ MEV Ltd. E-mail: <abbotti@mev.co.uk> )=- > -=( Web: http://www.mev.co.uk/ )=- _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: scripts/checkpatch.pl is deprecated 2017-01-12 10:27 ` Ian Abbott 2017-01-15 19:41 ` Vicente Bergas @ 2017-01-15 19:42 ` Vicente Bergas 2017-01-16 7:36 ` Sascha Hauer 1 sibling, 1 reply; 8+ messages in thread From: Vicente Bergas @ 2017-01-15 19:42 UTC (permalink / raw) To: Ian Abbott; +Cc: barebox, Uwe Kleine-König On Thu, Jan 12, 2017 at 11:27 AM, Ian Abbott <abbotti@mev.co.uk> wrote: > On 11/01/17 20:07, Uwe Kleine-König wrote: >> >> On Wed, Jan 11, 2017 at 08:35:11PM +0100, Vicente Bergas wrote: >>> >>> Hello, >>> when using scripts/checkpatch.pl it may report: >>> Unescaped left brace in regex is deprecated, >>> passed through in regex at scripts/checkpatch.pl line 2267. >> >> >> I bet this scripts originates from the kernel. So I'd recommend to test >> if their version is affected to. If yes, report to them, if not, update >> barebox to latest Linux version. > > > In the Linux kernel, this was fixed in commit > 6b8c69e4384b0bcf1936c2137804840212daaf69 ("checkpatch: fix another left > brace warning"), which made it into the the 4.6 kernel. This commit applies as is in barebox, so, can it be applied? https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/patch/?id=6b8c69e4384b0bcf1936c2137804840212daaf69 > > -- > -=( Ian Abbott @ MEV Ltd. E-mail: <abbotti@mev.co.uk> )=- > -=( Web: http://www.mev.co.uk/ )=- _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: scripts/checkpatch.pl is deprecated 2017-01-15 19:42 ` Vicente Bergas @ 2017-01-16 7:36 ` Sascha Hauer 2017-01-16 17:34 ` Vicente Bergas 0 siblings, 1 reply; 8+ messages in thread From: Sascha Hauer @ 2017-01-16 7:36 UTC (permalink / raw) To: Vicente Bergas; +Cc: barebox, Ian Abbott, Uwe Kleine-König On Sun, Jan 15, 2017 at 08:42:42PM +0100, Vicente Bergas wrote: > On Thu, Jan 12, 2017 at 11:27 AM, Ian Abbott <abbotti@mev.co.uk> wrote: > > On 11/01/17 20:07, Uwe Kleine-König wrote: > >> > >> On Wed, Jan 11, 2017 at 08:35:11PM +0100, Vicente Bergas wrote: > >>> > >>> Hello, > >>> when using scripts/checkpatch.pl it may report: > >>> Unescaped left brace in regex is deprecated, > >>> passed through in regex at scripts/checkpatch.pl line 2267. > >> > >> > >> I bet this scripts originates from the kernel. So I'd recommend to test > >> if their version is affected to. If yes, report to them, if not, update > >> barebox to latest Linux version. > > > > > > In the Linux kernel, this was fixed in commit > > 6b8c69e4384b0bcf1936c2137804840212daaf69 ("checkpatch: fix another left > > brace warning"), which made it into the the 4.6 kernel. > > This commit applies as is in barebox, so, can it be applied? > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/patch/?id=6b8c69e4384b0bcf1936c2137804840212daaf69 Yes. Can you send this as patch to the list? Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: scripts/checkpatch.pl is deprecated 2017-01-16 7:36 ` Sascha Hauer @ 2017-01-16 17:34 ` Vicente Bergas 2017-01-18 7:46 ` Sascha Hauer 0 siblings, 1 reply; 8+ messages in thread From: Vicente Bergas @ 2017-01-16 17:34 UTC (permalink / raw) To: Sascha Hauer; +Cc: barebox, Ian Abbott, Uwe Kleine-König On Mon, Jan 16, 2017 at 8:36 AM, Sascha Hauer <s.hauer@pengutronix.de> wrote: > On Sun, Jan 15, 2017 at 08:42:42PM +0100, Vicente Bergas wrote: >> On Thu, Jan 12, 2017 at 11:27 AM, Ian Abbott <abbotti@mev.co.uk> wrote: >> > On 11/01/17 20:07, Uwe Kleine-König wrote: >> >> >> >> On Wed, Jan 11, 2017 at 08:35:11PM +0100, Vicente Bergas wrote: >> >>> >> >>> Hello, >> >>> when using scripts/checkpatch.pl it may report: >> >>> Unescaped left brace in regex is deprecated, >> >>> passed through in regex at scripts/checkpatch.pl line 2267. >> >> >> >> >> >> I bet this scripts originates from the kernel. So I'd recommend to test >> >> if their version is affected to. If yes, report to them, if not, update >> >> barebox to latest Linux version. >> > >> > >> > In the Linux kernel, this was fixed in commit >> > 6b8c69e4384b0bcf1936c2137804840212daaf69 ("checkpatch: fix another left >> > brace warning"), which made it into the the 4.6 kernel. >> >> This commit applies as is in barebox, so, can it be applied? >> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/patch/?id=6b8c69e4384b0bcf1936c2137804840212daaf69 > > Yes. Can you send this as patch to the list? Why does it need to be in the list? can not it be applied directly? > > Sascha > > -- > Pengutronix e.K. | | > Industrial Linux Solutions | http://www.pengutronix.de/ | > Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: scripts/checkpatch.pl is deprecated 2017-01-16 17:34 ` Vicente Bergas @ 2017-01-18 7:46 ` Sascha Hauer 0 siblings, 0 replies; 8+ messages in thread From: Sascha Hauer @ 2017-01-18 7:46 UTC (permalink / raw) To: Vicente Bergas; +Cc: barebox, Ian Abbott, Uwe Kleine-König On Mon, Jan 16, 2017 at 06:34:19PM +0100, Vicente Bergas wrote: > On Mon, Jan 16, 2017 at 8:36 AM, Sascha Hauer <s.hauer@pengutronix.de> wrote: > > On Sun, Jan 15, 2017 at 08:42:42PM +0100, Vicente Bergas wrote: > >> On Thu, Jan 12, 2017 at 11:27 AM, Ian Abbott <abbotti@mev.co.uk> wrote: > >> > On 11/01/17 20:07, Uwe Kleine-König wrote: > >> >> > >> >> On Wed, Jan 11, 2017 at 08:35:11PM +0100, Vicente Bergas wrote: > >> >>> > >> >>> Hello, > >> >>> when using scripts/checkpatch.pl it may report: > >> >>> Unescaped left brace in regex is deprecated, > >> >>> passed through in regex at scripts/checkpatch.pl line 2267. > >> >> > >> >> > >> >> I bet this scripts originates from the kernel. So I'd recommend to test > >> >> if their version is affected to. If yes, report to them, if not, update > >> >> barebox to latest Linux version. > >> > > >> > > >> > In the Linux kernel, this was fixed in commit > >> > 6b8c69e4384b0bcf1936c2137804840212daaf69 ("checkpatch: fix another left > >> > brace warning"), which made it into the the 4.6 kernel. > >> > >> This commit applies as is in barebox, so, can it be applied? > >> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/patch/?id=6b8c69e4384b0bcf1936c2137804840212daaf69 > > > > Yes. Can you send this as patch to the list? > > Why does it need to be in the list? can not it be applied directly? Could be, but I am lazy ;) Anyway, in this case it was easy enough to wget/git-am the patch. Just did that. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-01-18 7:46 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-01-11 19:35 scripts/checkpatch.pl is deprecated Vicente Bergas 2017-01-11 20:07 ` Uwe Kleine-König 2017-01-12 10:27 ` Ian Abbott 2017-01-15 19:41 ` Vicente Bergas 2017-01-15 19:42 ` Vicente Bergas 2017-01-16 7:36 ` Sascha Hauer 2017-01-16 17:34 ` Vicente Bergas 2017-01-18 7:46 ` Sascha Hauer
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox