Re: [dpdk-dev] [PATCH v2 2/2] devtools: drop '+' from the section name

2018-11-30 Thread Ferruh Yigit
On 11/30/2018 3:28 PM, Neil Horman wrote: > On Fri, Nov 30, 2018 at 01:32:02PM +0100, David Marchand wrote: >> The incriminated commit did relax the condition to catch all sections >> but dropped the + removal which can triggers false detection of the >> special EXPERIMENTAL section when adding sym

Re: [dpdk-dev] [PATCH v2 2/2] devtools: drop '+' from the section name

2018-11-30 Thread David Marchand
Thomas, On Fri, Nov 30, 2018 at 4:28 PM Neil Horman wrote: > On Fri, Nov 30, 2018 at 01:32:02PM +0100, David Marchand wrote: > > The incriminated commit did relax the condition to catch all sections > > but dropped the + removal which can triggers false detection of the > > special EXPERIMENTAL

Re: [dpdk-dev] [PATCH v2 2/2] devtools: drop '+' from the section name

2018-11-30 Thread Neil Horman
On Fri, Nov 30, 2018 at 01:32:02PM +0100, David Marchand wrote: > The incriminated commit did relax the condition to catch all sections > but dropped the + removal which can triggers false detection of the > special EXPERIMENTAL section when adding symbols and the section in the > same patch. > >

[dpdk-dev] [PATCH v2 2/2] devtools: drop '+' from the section name

2018-11-30 Thread David Marchand
The incriminated commit did relax the condition to catch all sections but dropped the + removal which can triggers false detection of the special EXPERIMENTAL section when adding symbols and the section in the same patch. Fixes: 7281cf520f89 ("devtools: relax rule for identifying symbol section")