[dpdk-ci] [PATCH v4 7/7] tests: add checkpatch

[Thomas Monjalon]

2016-12-08 09:02, Wei, FangfangX:
> Hi Thomas,
> Following is the debug information today, please help to review and give some suggestion.
> 
> 1. replace "$sendmail -t" by "cat":
> >echo $report | tools/send-patch-report.sh -t "pw17762  [PATCH, 1/2] eth: get rid of goto's in rte_eth_dev_detach" -f "fangfangx.wei at intel.com" -m "20161208014751.24285-2-stephen at networkplumber.org" -p "dev.dpdk.org" -l "Intel Per-patch compilation check" -s "SUCCESS" -d "Compilation OK"
> 
> usage: pwclient [-h]
>                 {apply,git-am,get,info,projects,check-list,check-info,check-create,states,view,update,list,search}
>                 ...
> pwclient: error: No default project configured in ~/.pwclientrc
> 
> 
> There is no ~/.pwclientrc, should I copy config/pwclientrc to ~/.pwclientrc?

Yes
We could add a section about configuration in the README.

> 2. replace "$sendmail -t" by "cat" then copy config/pwclientrc to ~/.pwclientrc:
> >echo $report | tools/send-patch-report.sh -t "pw17762  [PATCH, 1/2] eth: get rid of goto's in rte_eth_dev_detach" -f "fangfangx.wei at intel.com" -m "20161208014751.24285-2-stephen at networkplumber.org" -p "dev.dpdk.org" -l "Intel Per-patch compilation check" -s "SUCCESS" -d "Compilation OK"
> 
> Traceback (most recent call last):
[...]
> socket.error: [Errno 110] Connection timed out

Is your Internet connection working fine? Do you need some proxy configuration?

> 3. The ci.config hasn't been loaded, because the environment $DPDK_CI_PWCLIENT and $DPDK_CI_MAILER are None.

They are not needed.
Some default values are used in send-patch-report.sh:
sendmail=${DPDK_CI_MAILER:-/usr/sbin/sendmail}
pwclient=${DPDK_CI_PWCLIENT:-$(dirname $(readlink -m $0))/pwclient}

> 4. replace "$sendmail -t" by "cat" then replace " pwid=$($pwclient list -f '%{id}' -m "$msgid " by "#pwid=$($pwclient list -f '%{id}' -m "$msgid")":
> >echo $report | tools/send-patch-report.sh -t "pw17762  [PATCH, 1/2] eth: get rid of goto's in rte_eth_dev_detach" -f "fangfangx.wei at intel.com" -m "20161208014751.24285-2-stephen at networkplumber.org" -p "dev.dpdk.org" -l "Intel Per-patch compilation check" -s "SUCCESS" -d "Compilation OK"
> 
> Subject: |SUCCESS| pw17762 [PATCH, 1/2] eth: get rid of goto's in rte_eth_dev_detach
> In-Reply-To: 20161208014751.24285-2-stephen at networkplumber.org
> References: 20161208014751.24285-2-stephen at networkplumber.org
> To: test-report at dpdk.org
> Cc: fangfangx.wei at intel.com
> 
> Test-Label: Intel Per-patch compilation check
> Test-Status: SUCCESS
> http://dpdk.org/patch/?
> 
> _Compilation OK_
> 
> Test-Label: Intel Per-patch compilation check Test-Status: SUCCESS http://www.dpdk.org/dev/patchwork/patch/17762 Submitter: Stephen Hemminger <stephen at networkplumber.org> Date: Wed, 7 Dec 2016 17:47:50 -0800 DPDK git baseline: Repo:dpdk, Branch:master, CommitID:c431384c8fbf8503693bcae1bdcd58d6fa459b8a Patch17762-17762 --> compile pass Build Summary: 18 Builds Done, 18 Successful, 0 Failures Test environment and configuration as below: OS: FreeBSD10.3_64 Kernel Version:10.3-RELEASE CPU info:CPU: Intel(R) Xeon(R) CPU E5-2699 v4 @ 2.20GHz (2194.97-MHz K8-class CPU) GCC Version:gcc (FreeBSD Ports Collection) 4.8.5 Clang Version:3.4.1 x86_64-native-bsdapp-clang x86_64-native-bsdapp-gcc OS: RHEL7.2_64 Kernel Version:3.10.0-327.el7.x86_64 CPU info:Intel(R) Xeon(R) CPU E5-2699 v4 @ 2.20GHz GCC Version:gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-4) Clang Version:3.4.2 i686-native-linuxapp-gcc x86_64-native-linuxapp-gcc x86_64-native-linuxapp-gcc-shared OS: UB1604_64 Kernel Version:4.4.0-47-generic CPU info:Intel(R) Xeon(R) CPU E5-2699 v3 @ 2.30GHz GCC Version:gcc (Ubuntu 5.4.0-6ubuntu1~16.04.4) 5.4.0 20160609 Clang Version:3.8.0 i686-native-linuxapp-gcc x86_64-native-linuxapp-clang x86_64-native-linuxapp-gcc-shared x86_64-native-linuxapp-gcc OS: CentOS7_64 Kernel Version:3.10.0-327.el7.x86_64 CPU info:Intel(R) Xeon(R) CPU E5-2699 v3 @ 2.30GHz GCC Version:gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-4) Clang Version:3.4.2 i686-native-linuxapp-gcc x86_64-native-linuxapp-clang x86_64-native-linuxapp-gcc-shared x86_64-native-linuxapp-gcc OS: FC24_64 Kernel Version:4.8.6-201.fc24.x86_64 CPU info:Intel(R) Xeon(R) CPU E5-2699 v3 @ 2.30GHz GCC Version:gcc (GCC) 6.2.1 20160916 (Red Hat 6.2.1-2) Clang Version:3.8.0 x86_64-native-linuxapp-gcc i686-native-linuxapp-gcc x86_64-native-linuxapp-clang x86_64-native-linuxapp-gcc-shared x86_64-native-linuxapp-gcc-debug

You have a wrapping issue ;)
You can remove Test-Label, Test-Status and other lines as they are already
printed above.