On 4/8/24 15:29, Alin Serdean wrote:
>
> The spellchecker one:
> https://github.com/openvswitch/ovs/blob/master/utilities/checkpatch.py#L115
> <https://github.com/openvswitch/ovs/blob/master/utilities/checkpatch.py#L115>
>
> I don’t assume so but wanted to verify.
Ah, this one was just added today by Eelco. :)
And it's also in OVS repo, not in OVN.
>
>
>
>> On 8 Apr 2024, at 13:30, Ilya Maximets <i.maxim...@ovn.org> wrote:
>>
>> On 4/8/24 13:26, Alin Serdean wrote:
>>>
>>> Just one question: do we have to remove the xenserver reference in the
>>> checkpatch.py?
>>
>> Hmm. Which one?
>>
>>>
>>> Otherwise looks good.
>>>
>>> Acked-by: Alin Gabriel Serdean <aserd...@ovn.org>
>>>
>>>> On 8 Apr 2024, at 13:06, Ilya Maximets <i.maxim...@ovn.org> wrote:
>>>>
>>>> Remaining bits from the OVS/OVN split.
>>>>
>>>> Fixes: 1af37d11be73 ("Remove XenServer Code")
>>>> Signed-off-by: Ilya Maximets <i.maxim...@ovn.org>
>>>> ---
>>>> Documentation/tutorials/index.rst | 6 ------
>>>> README.rst | 3 ---
>>>> acinclude.m4 | 26 -----------------------
>>>> build-aux/initial-tab-whitelist | 1 -
>>>> debian/copyright.in | 34 -------------------------------
>>>> 5 files changed, 70 deletions(-)
>>>>
>>>> diff --git a/Documentation/tutorials/index.rst
>>>> b/Documentation/tutorials/index.rst
>>>> index 0b7f52d0b..865a64018 100644
>>>> --- a/Documentation/tutorials/index.rst
>>>> +++ b/Documentation/tutorials/index.rst
>>>> @@ -30,12 +30,6 @@ Tutorials
>>>> Getting started with Open vSwitch (OVS) and Open Virtual Network (OVN) for
>>>> Open
>>>> vSwitch.
>>>>
>>>> -.. TODO(stephenfin): We could really do with a few basic tutorials, along
>>>> with
>>>> - some more specialized ones (DPDK, XenServer, Windows). The latter could
>>>> - probably be formed from the install guides, but the former will need
>>>> to be
>>>> - produced from scratch or reproduced from blogs (with permission of the
>>>> - author)
>>>> -
>>>> .. toctree::
>>>> :maxdepth: 2
>>>>
>>>> diff --git a/README.rst b/README.rst
>>>> index 428cd8ee8..dce8c5bc8 100644
>>>> --- a/README.rst
>>>> +++ b/README.rst
>>>> @@ -91,9 +91,6 @@ license applies to the file in question.
>>>>
>>>> File build-aux/cccl is licensed under the GNU General Public License,
>>>> version 2.
>>>>
>>>> -Files under the xenserver directory are licensed on a file-by-file basis.
>>>> -Refer to each file for details.
>>>> -
>>>> Contact
>>>> -------
>>>>
>>>> diff --git a/acinclude.m4 b/acinclude.m4
>>>> index 1a80dfaa7..73a2a51be 100644
>>>> --- a/acinclude.m4
>>>> +++ b/acinclude.m4
>>>> @@ -177,32 +177,6 @@ AC_DEFUN([OVS_CONDITIONAL_CC_OPTION],
>>>> AM_CONDITIONAL([$2], [test $ovs_have_cc_option = yes])])
>>>> dnl ----------------------------------------------------------------------
>>>>
>>>> -dnl Check for too-old XenServer.
>>>> -AC_DEFUN([OVS_CHECK_XENSERVER_VERSION],
>>>> - [AC_CACHE_CHECK([XenServer release], [ovs_cv_xsversion],
>>>> - [if test -e /etc/redhat-release; then
>>>> - ovs_cv_xsversion=`sed -n 's/^XenServer DDK release
>>>> \([[^-]]*\)-.*/\1/p' /etc/redhat-release`
>>>> - fi
>>>> - if test -z "$ovs_cv_xsversion"; then
>>>> - ovs_cv_xsversion=none
>>>> - fi])
>>>> - case $ovs_cv_xsversion in
>>>> - none)
>>>> - ;;
>>>> -
>>>> - [[1-9]][[0-9]]* | dnl XenServer 10 or later
>>>> - [[6-9]]* | dnl XenServer 6 or later
>>>> - 5.[[7-9]]* | dnl XenServer 5.7 or later
>>>> - 5.6.[[1-9]][[0-9]][[0-9]][[0-9]]* | dnl XenServer 5.6.1000 or later
>>>> - 5.6.[[2-9]][[0-9]][[0-9]]* | dnl XenServer 5.6.200 or later
>>>> - 5.6.1[[0-9]][[0-9]]) dnl Xenserver 5.6.100 or later
>>>> - ;;
>>>> -
>>>> - *)
>>>> - AC_MSG_ERROR([This appears to be XenServer $ovs_cv_xsversion, but
>>>> only XenServer 5.6.100 or later is supported. (If you are really using a
>>>> supported version of XenServer, you may override this error message by
>>>> specifying 'ovs_cv_xsversion=5.6.100' on the "configure" command line.)])
>>>> - ;;
>>>> - esac])
>>>> -
>>>> dnl OVS_CHECK_SPARSE_TARGET
>>>> dnl
>>>> dnl The "cgcc" script from "sparse" isn't very good at detecting the
>>>> diff --git a/build-aux/initial-tab-whitelist
>>>> b/build-aux/initial-tab-whitelist
>>>> index 8ad43d616..eb5c1a23a 100644
>>>> --- a/build-aux/initial-tab-whitelist
>>>> +++ b/build-aux/initial-tab-whitelist
>>>> @@ -5,7 +5,6 @@
>>>> \.sln$
>>>> ^ovs/
>>>> ^third-party/
>>>> -^xenserver/
>>>> ^debian/rules.modules$
>>>> ^debian/rules$
>>>> ^\.gitmodules$
>>>> diff --git a/debian/copyright.in b/debian/copyright.in
>>>> index 9ad00340f..335fd6af8 100644
>>>> --- a/debian/copyright.in
>>>> +++ b/debian/copyright.in
>>>> @@ -27,40 +27,6 @@ Upstream Copyright Holders:
>>>>
>>>> License:
>>>>
>>>> -* The following components are licensed under the
>>>> - GNU Lesser General Public License version 2.1 only
>>>> - with the exception clause below as a pre-amble.
>>>> -
>>>> - xenserver/etc_xensource_scripts_vif
>>>> - xenserver/opt_xensource_libexec_InterfaceReconfigure.py
>>>> - xenserver/opt_xensource_libexec_InterfaceReconfigureBridge.py
>>>> - xenserver/opt_xensource_libexec_InterfaceReconfigureVswitch.py
>>>> - xenserver/opt_xensource_libexec_interface-reconfigure
>>>> - xenserver/usr_lib_xsconsole_plugins-base_XSFeatureVSwitch.py
>>>> -
>>>> - * These components are only distributed in the source package.
>>>> - They do not appear in any binary packages.
>>>> -
>>>> - On Debian systems, the complete text of the
>>>> - GNU Lesser General Public License version 2.1 can be found in
>>>> - `/usr/share/common-licenses/LGPL-2.1'
>>>> -
>>>> - The exception clause pre-amble reads:
>>>> -
>>>> - As a special exception to the GNU Lesser General Public License, you
>>>> - may link, statically or dynamically, a "work that uses the Library"
>>>> - with a publicly distributed version of the Library to produce an
>>>> - executable file containing portions of the Library, and distribute
>>>> - that executable file under terms of your choice, without any of the
>>>> - additional requirements listed in clause 6 of the GNU Lesser General
>>>> - Public License. By "a publicly distributed version of the Library",
>>>> - we mean either the unmodified Library as distributed, or a
>>>> - modified version of the Library that is distributed under the
>>>> - conditions defined in clause 3 of the GNU Library General Public
>>>> - License. This exception does not however invalidate any other reasons
>>>> - why the executable file might be covered by the GNU Lesser General
>>>> - Public License.
>>>> -
>>>> * The following components are licensed under the
>>>> GNU Lesser General Public License version 2.1.
>>>>
>>>> --
>>>> 2.44.0
>>>>
>>>> _______________________________________________
>>>> dev mailing list
>>>> d...@openvswitch.org
>>>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>>
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
