[riot-notifications] [RIOT-OS/RIOT] pyterm: add /sleep function (#11095)

Gaƫtan Harter notifications at github.com
Mon Mar 11 16:25:59 CET 2019


cladmi commented on this pull request.



> @@ -303,6 +303,14 @@ class SerCmd(cmd.Cmd):
         date = time.strftime("%Y-%m-%d %H:%M:%S")
         return ["%s" % (date)]
 
+    def do_PYTERM_sleep(self, line):
+        """Pyterm command: Sleep for n seconds.
+        """
+        if line:

For the implementation, I would rather make the time mandatory instead of doing `1` by default.
When I do `sleep` in my shell, I get `sleep: missing operand`.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11095#discussion_r264286513
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190311/dff5fad9/attachment.html>


More information about the notifications mailing list