Re: [dpdk-dev] [PATCH v2] devtools: add script to get maintainers from patch

[Ferruh Yigit <ferruh.yigit@intel.com>]

On 11/7/2017 3:35 PM, Thomas Monjalon wrote:
> 04/08/2017 16:01, Ferruh Yigit:
>> This is a wrapper to Linux kernel get_maintainer.pl file and only
>> supports parsing MAINTAINERS file (no git fallback etc..)
>>
>> Requires DPDK_GETMAINTAINER_PATH devel config option set, please check
>> devtools/load-devel-config.
>>
>> DPDK_GETMAINTAINER_PATH should be full path to the get_maintainer.pl
>> script, like:
>> DPDK_GETMAINTAINER_PATH=~/linux/scripts/get_maintainer.pl
>>
>> Can be used individually:
>> ./devtools/get_maintainer.sh <my.patch>
>>
>> Or via git send-email, to add maintainers automatically:
>> git send-email --to-cmd ./devtools/get_maintainer.sh \
>> --cc dev@dpdk.org HEAD -4
>>
>> Currently there is an ugly workaround to be able to use Linux script out
>> of the kernel tree, later better method can replace it.
>>
>> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> 
> Sorry for having missed this very convenient patch for months.
> 
> Applied, thanks
> 
> Have you tried to send a patch to Linux
> in order to avoid the workaround below?

No, not tried, but that list looks like put intentionally so not sure if a patch
helps.

> 
>> +FILES="COPYING CREDITS Kbuild"
>> +FOLDERS="Documentation arch include fs init ipc kernel scripts"
>> +
>> +# Kernel script checks for some files and folders to run
>> +workaround () {
>> +	for f in $FILES; do
>> +		if [ ! -f $f ]; then touch $f; fi
>> +	done
>> +
>> +	for d in $FOLDERS; do
>> +		if [ ! -d $d ]; then mkdir $d; fi
>> +	done
>> +}