Bleep bloop. Greetings Terry Wilson, 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
#27 FILE: utilities/ovn-ctl:315:
set "$@" $extra_args
WARNING: Line has non-spaces leading whitespace
#49 FILE: utilities/ovn-ctl:575:
set "$@" $extra_args
WARNING: Line has non-spaces leading whitespace
#60 FILE: utilities/ovn-ctl:606:
set "$@" $extra_args
WARNING: Line is 98 characters long (recommended limit is 79)
#97 FILE: utilities/ovn-ctl.8.xml:7:
<p><code>ovn-ctl</code> [<var>options</var>] <var>command</var> [--
<var>extra_args</var>]</p>
WARNING: Line is 84 characters long (recommended limit is 79)
#107 FILE: utilities/ovn-ctl.8.xml:161:
Any options after '--' will be passed on to the binary run by
<var>command</var>
WARNING: Line is 102 characters long (recommended limit is 79)
#108 FILE: utilities/ovn-ctl.8.xml:162:
with the exception of start_northd, which can have options specified in
ovn-northd-db-params.conf.
WARNING: Line is 91 characters long (recommended limit is 79)
#109 FILE: utilities/ovn-ctl.8.xml:163:
Any <var>extra_args</var> passed to start_northd will be passed to the
ovsdb-servers if
Lines checked: 118, Warnings: 7, Errors: 0
build:
/usr/bin/python3 ./build-aux/dpdkstrip.py | \
sed \
-e 's,[@]PKIDIR[@],/usr/local/var/lib/openvswitch/pki,g' \
-e 's,[@]LOGDIR[@],/usr/local/var/log/ovn,g' \
-e 's,[@]DBDIR[@],/usr/local/etc/ovn,g' \
-e 's,[@]PYTHON3[@],/usr/bin/python3,g' \
-e 's,[@]OVN_RUNDIR[@],/usr/local/var/run/ovn,g' \
-e
's,[@]OVSBUILDDIR[@],/var/lib/jenkins/jobs/0day_robot_upstream_build_ovn_from_pw/workspace/ovs,g'
\
-e 's,[@]VERSION[@],22.06.90,g' \
-e 's,[@]OVSVERSION[@],2.17.90,g' \
-e 's,[@]localstatedir[@],/usr/local/var,g' \
-e 's,[@]pkgdatadir[@],/usr/local/share/ovn,g' \
-e 's,[@]sysconfdir[@],/usr/local/etc,g' \
-e 's,[@]bindir[@],/usr/local/bin,g' \
-e 's,[@]sbindir[@],/usr/local/sbin,g' \
-e
's,[@]abs_builddir[@],/var/lib/jenkins/jobs/0day_robot_upstream_build_ovn_from_pw/workspace,g'
\
-e
's,[@]abs_top_srcdir[@],/var/lib/jenkins/jobs/0day_robot_upstream_build_ovn_from_pw/workspace,g'
\
> rhel/ovn-fedora.spec.tmp
mv rhel/ovn-fedora.spec.tmp rhel/ovn-fedora.spec
utilities/ovn-ctl
See above for files that use tabs for indentation.
Please use spaces instead.
make[1]: *** [check-tabs] Error 1
make[1]: Leaving directory
`/var/lib/jenkins/jobs/0day_robot_upstream_build_ovn_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
