[ovs-dev] [PATCH] submitting-patches: Update test and documentation recommendations.
Ben Pfaff
blp at ovn.org
Thu Jul 13 22:42:07 UTC 2017
On Fri, Jul 07, 2017 at 12:18:43AM +0000, Darrell Ball wrote:
>
>
> On 5/18/17, 5:32 AM, "ovs-dev-bounces at openvswitch.org on behalf of Ben Pfaff" <ovs-dev-bounces at openvswitch.org on behalf of blp at ovn.org> wrote:
>
> Rationale:
>
> - "make distcheck" is not as necessary anymore because we have a build-time
> check that fails if files in the repository are not distributed.
>
> - xenserver has not been important for years, so remove the specific
> callout.
>
> - We already have an informal custom of adding tests for new feaures and
> bug fixes, so codify it.
>
> - Add note about updating NEWS.
>
> Signed-off-by: Ben Pfaff <blp at ovn.org>
> ---
> .../internals/contributing/submitting-patches.rst | 25 +++++++++++-----------
> 1 file changed, 12 insertions(+), 13 deletions(-)
>
> diff --git a/Documentation/internals/contributing/submitting-patches.rst b/Documentation/internals/contributing/submitting-patches.rst
> index edbc1aa8d9ff..4f7ca0304034 100644
> --- a/Documentation/internals/contributing/submitting-patches.rst
> +++ b/Documentation/internals/contributing/submitting-patches.rst
> @@ -44,29 +44,28 @@ particular:
> - A patch should make one logical change. Don't make multiple, logically
> unconnected changes to disparate subsystems in a single patch.
>
> -- A patch that adds or removes user-visible features should also update the
> - appropriate user documentation or manpages. Check "Feature Deprecation
> - Guidelines" section in this document if you intend to remove user-visible
> - feature.
> +- A patch that adds or removes user-visible features should also
> + update the appropriate user documentation or manpages. Consider
> + adding an item to NEWS for nontrivial changes. Check "Feature
> + Deprecation Guidelines" section in this document if you intend to
> + remove user-visible feature.
>
> Testing is also important:
>
> -- A patch that modifies existing code should be tested with ``make
> - check`` before submission. Refer to the `install guide`, under "Self-Tests",
> - for more information.
> +- Test a patch that modifies existing code with ``make check`` before
> + submission. Refer to the `install guide`, under "Self-Tests", for
> + more information.
>
> I think “Testing” is under Documentation and the label is “Unit Tests”
> I cannot find the title “Self-Tests”, but admittedly, I have little patience.
>
> Maybe checking the system tests (kernel and userspace) should be strongly suggested,
> if not an expectation.
Thanks, I fixed that stuff up and applied this to master.
More information about the dev
mailing list