<div dir="auto">Sure. Push it.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 20, 2021, 6:36 PM Vijay Kumar Banerjee <<a href="mailto:vijay@rtems.org">vijay@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
In the last review, the commit message wasn't clear, I fixed it in this version.<br>
Does this look Ok to be merged?<br>
<br>
Best regards,<br>
Vijay<br>
<br>
On Thu, Apr 22, 2021 at 8:02 PM Vijay Kumar Banerjee <<a href="mailto:vijay@rtems.org" target="_blank" rel="noreferrer">vijay@rtems.org</a>> wrote:<br>
><br>
> telnetd01 test cannot be run without a network stack, so this test is being<br>
> moved to the rtems-net-legacy repository.<br>
> ---<br>
>  spec/build/testsuites/libtests/grp.yml       |   2 -<br>
>  spec/build/testsuites/libtests/telnetd01.yml |  22 ----<br>
>  testsuites/libtests/telnetd01/init.c         | 120 -------------------<br>
>  testsuites/libtests/telnetd01/telnetd01.doc  |  24 ----<br>
>  testsuites/libtests/telnetd01/telnetd01.scn  |  11 --<br>
>  5 files changed, 179 deletions(-)<br>
>  delete mode 100644 spec/build/testsuites/libtests/telnetd01.yml<br>
>  delete mode 100644 testsuites/libtests/telnetd01/init.c<br>
>  delete mode 100644 testsuites/libtests/telnetd01/telnetd01.doc<br>
>  delete mode 100644 testsuites/libtests/telnetd01/telnetd01.scn<br>
><br>
> diff --git a/spec/build/testsuites/libtests/grp.yml b/spec/build/testsuites/libtests/grp.yml<br>
> index 5695fc7f06..20a593a5d3 100644<br>
> --- a/spec/build/testsuites/libtests/grp.yml<br>
> +++ b/spec/build/testsuites/libtests/grp.yml<br>
> @@ -258,8 +258,6 @@ links:<br>
>    uid: tar02<br>
>  - role: build-dependency<br>
>    uid: tar03<br>
> -- role: build-dependency<br>
> -  uid: telnetd01<br>
>  - role: build-dependency<br>
>    uid: termios<br>
>  - role: build-dependency<br>
> diff --git a/spec/build/testsuites/libtests/telnetd01.yml b/spec/build/testsuites/libtests/telnetd01.yml<br>
> deleted file mode 100644<br>
> index 9f5bda84d9..0000000000<br>
> --- a/spec/build/testsuites/libtests/telnetd01.yml<br>
> +++ /dev/null<br>
> @@ -1,22 +0,0 @@<br>
> -SPDX-License-Identifier: CC-BY-SA-4.0 OR BSD-2-Clause<br>
> -build-type: test-program<br>
> -cflags: []<br>
> -copyrights:<br>
> -- Copyright (C) 2020 embedded brains GmbH (<a href="http://www.embedded-brains.de" rel="noreferrer noreferrer" target="_blank">http://www.embedded-brains.de</a>)<br>
> -cppflags: []<br>
> -cxxflags: []<br>
> -enabled-by:<br>
> -- RTEMS_NETWORKING<br>
> -features: c cprogram<br>
> -includes:<br>
> -- cpukit/libnetworking<br>
> -ldflags: []<br>
> -links: []<br>
> -source:<br>
> -- testsuites/libtests/telnetd01/init.c<br>
> -stlib: []<br>
> -target: testsuites/libtests/telnetd01.exe<br>
> -type: build<br>
> -use-after:<br>
> -- telnetd<br>
> -use-before: []<br>
> diff --git a/testsuites/libtests/telnetd01/init.c b/testsuites/libtests/telnetd01/init.c<br>
> deleted file mode 100644<br>
> index a17126bf41..0000000000<br>
> --- a/testsuites/libtests/telnetd01/init.c<br>
> +++ /dev/null<br>
> @@ -1,120 +0,0 @@<br>
> -/*<br>
> - * Copyright (c) 2018 embedded brains GmbH.  All rights reserved.<br>
> - *<br>
> - *  embedded brains GmbH<br>
> - *  Dornierstr. 4<br>
> - *  82178 Puchheim<br>
> - *  Germany<br>
> - *  <<a href="mailto:rtems@embedded-brains.de" target="_blank" rel="noreferrer">rtems@embedded-brains.de</a>><br>
> - *<br>
> - * The license and distribution terms for this file may be<br>
> - * found in the file LICENSE in this distribution or at<br>
> - * <a href="http://www.rtems.org/license/LICENSE" rel="noreferrer noreferrer" target="_blank">http://www.rtems.org/license/LICENSE</a>.<br>
> - */<br>
> -<br>
> -#ifdef HAVE_CONFIG_H<br>
> -#include "config.h"<br>
> -#endif<br>
> -<br>
> -#include <sys/stat.h><br>
> -#include <fcntl.h><br>
> -#include <string.h><br>
> -<br>
> -#include <rtems.h><br>
> -#include <rtems/rtems_bsdnet.h><br>
> -#include <rtems/telnetd.h><br>
> -<br>
> -#include <tmacros.h><br>
> -<br>
> -const char rtems_test_name[] = "TELNETD 1";<br>
> -<br>
> -struct rtems_bsdnet_config rtems_bsdnet_config;<br>
> -<br>
> -static void command(char *device_name, void *arg)<br>
> -{<br>
> -}<br>
> -<br>
> -static void test_command_null(void)<br>
> -{<br>
> -  static const rtems_telnetd_config_table config = {<br>
> -    .command = NULL<br>
> -  };<br>
> -  rtems_status_code sc;<br>
> -<br>
> -  sc = rtems_telnetd_start(&config);<br>
> -  rtems_test_assert(sc == RTEMS_INVALID_ADDRESS);<br>
> -}<br>
> -<br>
> -static void test_cannot_start_server_task(void)<br>
> -{<br>
> -  static const rtems_telnetd_config_table config = {<br>
> -    .command = command,<br>
> -    .priority = UINT32_MAX<br>
> -  };<br>
> -  rtems_status_code sc;<br>
> -<br>
> -  sc = rtems_telnetd_start(&config);<br>
> -  rtems_test_assert(sc == RTEMS_UNSATISFIED);<br>
> -}<br>
> -<br>
> -static void test_successful_start(void)<br>
> -{<br>
> -  static const rtems_telnetd_config_table config = {<br>
> -    .command = command,<br>
> -    .stack_size = RTEMS_MINIMUM_STACK_SIZE<br>
> -  };<br>
> -  rtems_status_code sc;<br>
> -<br>
> -  sc = rtems_telnetd_start(&config);<br>
> -  rtems_test_assert(sc == RTEMS_SUCCESSFUL);<br>
> -}<br>
> -<br>
> -static void test_already_started(void)<br>
> -{<br>
> -  static const rtems_telnetd_config_table config = {<br>
> -    .command = command<br>
> -  };<br>
> -  rtems_status_code sc;<br>
> -<br>
> -  sc = rtems_telnetd_start(&config);<br>
> -  rtems_test_assert(sc == RTEMS_RESOURCE_IN_USE);<br>
> -}<br>
> -<br>
> -static rtems_task Init(rtems_task_argument argument)<br>
> -{<br>
> -  int rv;<br>
> -<br>
> -  TEST_BEGIN();<br>
> -<br>
> -  rv = rtems_bsdnet_initialize_network();<br>
> -  rtems_test_assert(rv == 0);<br>
> -<br>
> -  test_command_null();<br>
> -  test_cannot_start_server_task();<br>
> -  test_successful_start();<br>
> -  test_already_started();<br>
> -<br>
> -  TEST_END();<br>
> -  rtems_test_exit(0);<br>
> -}<br>
> -<br>
> -#define CONFIGURE_INIT<br>
> -<br>
> -#define CONFIGURE_MICROSECONDS_PER_TICK 10000<br>
> -<br>
> -#define CONFIGURE_APPLICATION_NEEDS_CLOCK_DRIVER<br>
> -#define CONFIGURE_APPLICATION_NEEDS_SIMPLE_CONSOLE_DRIVER<br>
> -<br>
> -#define CONFIGURE_MAXIMUM_FILE_DESCRIPTORS (3 + 1 + 5 * 4)<br>
> -<br>
> -#define CONFIGURE_MAXIMUM_TASKS 8<br>
> -<br>
> -#define CONFIGURE_MAXIMUM_POSIX_KEYS 1<br>
> -<br>
> -#define CONFIGURE_INITIAL_EXTENSIONS RTEMS_TEST_INITIAL_EXTENSION<br>
> -<br>
> -#define CONFIGURE_RTEMS_INIT_TASKS_TABLE<br>
> -<br>
> -#define CONFIGURE_INIT_TASK_ATTRIBUTES RTEMS_FLOATING_POINT<br>
> -<br>
> -#include <rtems/confdefs.h><br>
> diff --git a/testsuites/libtests/telnetd01/telnetd01.doc b/testsuites/libtests/telnetd01/telnetd01.doc<br>
> deleted file mode 100644<br>
> index fe1a4d31a3..0000000000<br>
> --- a/testsuites/libtests/telnetd01/telnetd01.doc<br>
> +++ /dev/null<br>
> @@ -1,24 +0,0 @@<br>
> -#<br>
> -# Copyright (c) 2018 embedded brains GmbH.  All rights reserved.<br>
> -#<br>
> -#  embedded brains GmbH<br>
> -#  Dornierstr. 4<br>
> -#  82178 Puchheim<br>
> -#  Germany<br>
> -#  <<a href="mailto:rtems@embedded-brains.de" target="_blank" rel="noreferrer">rtems@embedded-brains.de</a>><br>
> -#<br>
> -# The license and distribution terms for this file may be<br>
> -# found in the file LICENSE in this distribution or at<br>
> -# <a href="http://www.rtems.org/license/LICENSE" rel="noreferrer noreferrer" target="_blank">http://www.rtems.org/license/LICENSE</a>.<br>
> -<br>
> -This file describes the directives and concepts tested by this test set.<br>
> -<br>
> -test set name: telnetd01<br>
> -<br>
> -directives:<br>
> -<br>
> -  - rtems_telnetd_start()<br>
> -<br>
> -concepts:<br>
> -<br>
> -+ Check if Telnet server works.<br>
> diff --git a/testsuites/libtests/telnetd01/telnetd01.scn b/testsuites/libtests/telnetd01/telnetd01.scn<br>
> deleted file mode 100644<br>
> index 3e9cc0007a..0000000000<br>
> --- a/testsuites/libtests/telnetd01/telnetd01.scn<br>
> +++ /dev/null<br>
> @@ -1,11 +0,0 @@<br>
> -*** BEGIN OF TEST TELNETD 1 ***<br>
> -*** TEST VERSION: 5.0.0.dc32b6aa0807fb70f9b26bc0bc6e164ddb49bd3a<br>
> -*** TEST STATE: EXPECTED_PASS<br>
> -*** TEST BUILD: RTEMS_NETWORKING<br>
> -*** TEST TOOLS: 7.3.0 20180125 (RTEMS 5, RSB 9670d7541e0621915e521fe76e7bb33de8cee661, Newlib d13c84eb07e35984bf7a974cd786a6cdac29e6b9)<br>
> -syslog: telnetd: configuration with invalid command<br>
> -syslog: telnetd: cannot create session task<br>
> -syslog: telnetd: started successfully on port 23<br>
> -syslog: telnetd: cannot bind server socket<br>
> -<br>
> -*** END OF TEST TELNETD 1 ***<br>
> --<br>
> 2.26.2<br>
><br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div>