Re: [LTP] [PATCH v2] Add runltp-ng to upstream

[Petr Vorel <pvorel@suse.cz>]

Hi all,

> runltp-ng is the next generation runner for Linux Testing Project and it
> will replace the current obsolete runltp script in the next future.

changes v1->v2
* repository moved to LTP github namespace

> Signed-off-by: Andrea Cervesato <andrea.cervesato@suse.com>
> ---
>  .gitmodules     | 3 +++
>  tools/runltp-ng | 1 +
>  2 files changed, 4 insertions(+)
>  create mode 160000 tools/runltp-ng

> diff --git a/.gitmodules b/.gitmodules
> index a3c34af4b..810eac395 100644
> --- a/.gitmodules
> +++ b/.gitmodules
> @@ -4,3 +4,6 @@
>  [submodule "tools/sparse/sparse-src"]
>  	path = tools/sparse/sparse-src
>  	url = git://git.kernel.org/pub/scm/devel/sparse/sparse.git
> +[submodule "tools/runltp-ng"]
> +	path = tools/runltp-ng
> +	url = git@github.com:linux-test-project/runltp-ng.git
Wouldn't be better to use https ?

-	url = git@github.com:linux-test-project/runltp-ng.git
+	url = https://github.com/linux-test-project/runltp-ng.git

Or are there problems with certificates? If agree on https,
we of course should migrate other

Otherwise LGTM

Acked-by: Petr Vorel <pvorel@suse.cz>

Kind regards,
Petr

> diff --git a/tools/runltp-ng b/tools/runltp-ng
> new file mode 160000
> index 000000000..af20a864e
> --- /dev/null
> +++ b/tools/runltp-ng
> @@ -0,0 +1 @@
> +Subproject commit af20a864ec01674ce4f5a21d3308dc3a76da9303

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp