Re: [dpdk-dev] [PATCH] devtools: avoid config backup during test build setup

2019-12-16 Thread Thomas Monjalon
16/12/2019 17:59, Bruce Richardson: > On Mon, Dec 16, 2019 at 05:47:03PM +0100, Thomas Monjalon wrote: > > 16/12/2019 17:40, Bruce Richardson: > > > On Mon, Dec 16, 2019 at 05:22:42PM +0100, Thomas Monjalon wrote: > > > > The config options are updated with sed and the short inline option. > > > >

Re: [dpdk-dev] [PATCH] devtools: avoid config backup during test build setup

2019-12-16 Thread Bruce Richardson
On Mon, Dec 16, 2019 at 05:47:03PM +0100, Thomas Monjalon wrote: > 16/12/2019 17:40, Bruce Richardson: > > On Mon, Dec 16, 2019 at 05:22:42PM +0100, Thomas Monjalon wrote: > > > The config options are updated with sed and the short inline option. > > > In FreeBSD, the suffix argument of the inline

Re: [dpdk-dev] [PATCH] devtools: avoid config backup during test build setup

2019-12-16 Thread Thomas Monjalon
16/12/2019 17:40, Bruce Richardson: > On Mon, Dec 16, 2019 at 05:22:42PM +0100, Thomas Monjalon wrote: > > The config options are updated with sed and the short inline option. > > In FreeBSD, the suffix argument of the inline option is mandatory. > > In order to avoid creating a backup file when us

Re: [dpdk-dev] [PATCH] devtools: avoid config backup during test build setup

2019-12-16 Thread Bruce Richardson
On Mon, Dec 16, 2019 at 05:22:42PM +0100, Thomas Monjalon wrote: > The config options are updated with sed and the short inline option. > In FreeBSD, the suffix argument of the inline option is mandatory. > In order to avoid creating a backup file when using sed inline, > the argument is set as an

[dpdk-dev] [PATCH] devtools: avoid config backup during test build setup

2019-12-16 Thread Thomas Monjalon
The config options are updated with sed and the short inline option. In FreeBSD, the suffix argument of the inline option is mandatory. In order to avoid creating a backup file when using sed inline, the argument is set as an empty suffix. The short option '-i' does not expect '=' before the suffi