Hi Martin, please fix the problems reported by the robot.
Thanks,
Ben.
On Thu, Sep 26, 2019 at 09:58:22PM -0400, 0-day Robot wrote:
> Bleep bloop. Greetings Martin Zhang, I am a robot and I have tried out your
> patch.
> Thanks for your contribution.
>
> I encountered some error that I wasn't expecting. See the details below.
>
>
> checkpatch:
> WARNING: Line has non-spaces leading whitespace
> #40 FILE: lib/odp-util.c:1527:
> memset(ip, 0, sizeof(*ip));
>
> WARNING: Line has non-spaces leading whitespace
> #48 FILE: lib/odp-util.c:1541:
> ip->ip_csum = csum(ip, ip_len);
>
> Lines checked: 78, Warnings: 2, Errors: 0
>
>
> build:
> mv tests/ovsdb-cluster-testsuite.tmp tests/ovsdb-cluster-testsuite
> \
> { sed -n -e '/%AUTHORS%/q' -e p < ./debian/copyright.in; \
> sed '34,/^$/d' ./AUTHORS.rst | \
> sed -n -e '/^$/q' -e 's/^/ /p'; \
> sed -e '34,/%AUTHORS%/d' ./debian/copyright.in; \
> } > debian/copyright
> (printf '\043 Generated automatically -- do not modify! -*-
> buffer-read-only: t -*-\n' && sed -e 's,[@]VERSION[@],2.12.90,g') <
> ./rhel/openvswitch-dkms.spec.in > openvswitch-dkms.spec.tmp || exit 1; if cmp
> -s openvswitch-dkms.spec.tmp rhel/openvswitch-dkms.spec; then touch
> rhel/openvswitch-dkms.spec; rm openvswitch-dkms.spec.tmp; else mv
> openvswitch-dkms.spec.tmp rhel/openvswitch-dkms.spec; fi
> (printf '\043 Generated automatically -- do not modify! -*-
> buffer-read-only: t -*-\n' && sed -e 's,[@]VERSION[@],2.12.90,g') <
> ./rhel/kmod-openvswitch-rhel6.spec.in > kmod-openvswitch-rhel6.spec.tmp ||
> exit 1; if cmp -s kmod-openvswitch-rhel6.spec.tmp
> rhel/kmod-openvswitch-rhel6.spec; then touch
> rhel/kmod-openvswitch-rhel6.spec; rm kmod-openvswitch-rhel6.spec.tmp; else mv
> kmod-openvswitch-rhel6.spec.tmp rhel/kmod-openvswitch-rhel6.spec; fi
> (printf '\043 Generated automatically -- do not modify! -*-
> buffer-read-only: t -*-\n' && sed -e 's,[@]VERSION[@],2.12.90,g') <
> ./rhel/openvswitch-kmod-fedora.spec.in > openvswitch-kmod-fedora.spec.tmp ||
> exit 1; if cmp -s openvswitch-kmod-fedora.spec.tmp
> rhel/openvswitch-kmod-fedora.spec; then touch
> rhel/openvswitch-kmod-fedora.spec; rm openvswitch-kmod-fedora.spec.tmp; else
> mv openvswitch-kmod-fedora.spec.tmp rhel/openvswitch-kmod-fedora.spec; fi
> (printf '\043 Generated automatically -- do not modify! -*-
> buffer-read-only: t -*-\n' && sed -e 's,[@]VERSION[@],2.12.90,g') <
> ./rhel/openvswitch.spec.in > openvswitch.spec.tmp || exit 1; if cmp -s
> openvswitch.spec.tmp rhel/openvswitch.spec; then touch rhel/openvswitch.spec;
> rm openvswitch.spec.tmp; else mv openvswitch.spec.tmp rhel/openvswitch.spec;
> fi
> (printf '\043 Generated automatically -- do not modify! -*-
> buffer-read-only: t -*-\n' && sed -e 's,[@]VERSION[@],2.12.90,g') <
> ./rhel/openvswitch-fedora.spec.in > openvswitch-fedora.spec.tmp || exit 1; if
> cmp -s openvswitch-fedora.spec.tmp rhel/openvswitch-fedora.spec; then touch
> rhel/openvswitch-fedora.spec; rm openvswitch-fedora.spec.tmp; else mv
> openvswitch-fedora.spec.tmp rhel/openvswitch-fedora.spec; fi
> (printf '\043 Generated automatically -- do not modify! -*-
> buffer-read-only: t -*-\n' && sed -e 's,[@]VERSION[@],2.12.90,g') \
> < ./xenserver/openvswitch-xen.spec.in > openvswitch-xen.spec.tmp ||
> exit 1; \
> if cmp -s openvswitch-xen.spec.tmp xenserver/openvswitch-xen.spec; then touch
> xenserver/openvswitch-xen.spec; rm openvswitch-xen.spec.tmp; else mv
> openvswitch-xen.spec.tmp xenserver/openvswitch-xen.spec; fi
> make[3]: Entering directory
> `/var/lib/jenkins/jobs/upstream_build_from_pw/workspace/datapath'
> make[3]: Leaving directory
> `/var/lib/jenkins/jobs/upstream_build_from_pw/workspace/datapath'
> lib/odp-util.c
> See above for files that use tabs for indentation.
> Please use spaces instead.
> make[2]: *** [check-tabs] Error 1
> make[2]: Leaving directory
> `/var/lib/jenkins/jobs/upstream_build_from_pw/workspace'
> make[1]: *** [all-recursive] Error 1
> make[1]: Leaving directory
> `/var/lib/jenkins/jobs/upstream_build_from_pw/workspace'
> make: *** [all] Error 2
>
>
> Please check this out. If you feel there has been an error, please email
> [email protected]
>
> Thanks,
> 0-day Robot
> _______________________________________________
> dev mailing list
> [email protected]
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
