Changes in uspace/app/getterm/getterm.c [1c635d6:884c56b] in mainline

File:

• uspace/app/getterm/getterm.c (modified) (4 diffs)

uspace/app/getterm/getterm.c

@@ -42 +42 @@
 #include <str_error.h>
 #include <errno.h>
-#include <loc.h>
 #include "version.h"
 #include "welcome.h"
@@ -50 +49 @@
 static void usage(void)
 {
-        printf("Usage: %s <terminal> <locfs> [--msg] [--wait] -- "
-            "<command> [<arguments...>]\n", APP_NAME);
-        printf(" <terminal>    Terminal device\n");
-        printf(" <locfs>       Mount point of locfs\n");
-        printf(" --msg         Print welcome message\n");
-        printf(" --wait        Wait for the terminal to be ready\n");
+        printf("Usage: %s <terminal> [-w] <command> [<arguments...>]\n", APP_NAME);
 }
 
-static void reopen(FILE **stream, int fd, const char *path, int flags,
-    const char *mode)
+static void reopen(FILE **stream, int fd, const char *path, int flags, const char *mode)
 {
         if (fclose(*stream))
@@ -83 +76 @@
 int main(int argc, char *argv[])
 {
+        int rc;
+        task_exit_t texit;
+        int retval;
+        task_id_t id;
+        char *fname, *term;
+        char **cmd_args;
+        bool print_wmsg;
+
         argv++;
         argc--;
-        if (argc < 4) {
+        if (argc < 1) {
                 usage();
-                return 1;
+                return -1;
         }
-        
-        char *term = *argv;
-        argv++;
-        argc--;
-        
-        char *locfs = *argv;
-        argv++;
-        argc--;
-        
-        bool print_msg = false;
-        bool wait = false;
-        
-        while ((argc > 0) && (str_cmp(*argv, "--") != 0)) {
-                if (str_cmp(*argv, "--msg") == 0) {
-                        print_msg = true;
-                } else if (str_cmp(*argv, "--wait") == 0) {
-                        wait = true;
-                } else {
-                        usage();
-                        return 2;
-                }
-                
+
+        if (str_cmp(*argv, "-w") == 0) {
+                print_wmsg = true;
                 argv++;
                 argc--;
+        } else {
+                print_wmsg = false;
         }
+
+        if (argc < 2) {
+                usage();
+                return -1;
+        }
+
+        term = *argv++;
+        fname = *argv;
+        cmd_args = argv;
         
-        if (argc < 1) {
-                usage();
-                return 3;
-        }
-        
-        /* Skip "--" */
-        argv++;
-        argc--;
-        
-        char *cmd = *argv;
-        char **args = argv;
-        
-        if (wait) {
-                /* Wait for the terminal service to be ready */
-                service_id_t service_id;
-                int rc = loc_service_get_id(term, &service_id, IPC_FLAG_BLOCKING);
-                if (rc != EOK) {
-                        printf("%s: Error waiting on %s (%s)\n", APP_NAME, term,
-                            str_error(rc));
-                        return rc;
-                }
-        }
-        
-        char term_node[LOC_NAME_MAXLEN];
-        snprintf(term_node, LOC_NAME_MAXLEN, "%s/%s", locfs, term);
-        
-        reopen(&stdin, 0, term_node, O_RDONLY, "r");
-        reopen(&stdout, 1, term_node, O_WRONLY, "w");
-        reopen(&stderr, 2, term_node, O_WRONLY, "w");
+        reopen(&stdin, 0, term, O_RDONLY, "r");
+        reopen(&stdout, 1, term, O_WRONLY, "w");
+        reopen(&stderr, 2, term, O_WRONLY, "w");
         
         if (stdin == NULL)
-                return 4;
+                return -2;
         
         if (stdout == NULL)
-                return 5;
+                return -3;
         
         if (stderr == NULL)
-                return 6;
+                return -4;
         
         /*
@@ -161 +128 @@
         
         version_print(term);
-        if (print_msg)
+        if (print_wmsg)
                 welcome_msg_print();
-        
-        task_id_t id;
-        task_wait_t twait;
-        
-        int rc = task_spawnv(&id, &twait, cmd, (const char * const *) args);
+
+        rc = task_spawnv(&id, fname, (const char * const *) cmd_args);
         if (rc != EOK) {
-                printf("%s: Error spawning %s (%s)\n", APP_NAME, cmd,
+                printf("%s: Error spawning %s (%s)\n", APP_NAME, fname,
                     str_error(rc));
-                return rc;
+                return -5;
         }
-        
-        task_exit_t texit;
-        int retval;
-        rc = task_wait(&twait, &texit, &retval);
+
+        rc = task_wait(id, &texit, &retval);
         if (rc != EOK) {
-                printf("%s: Error waiting for %s (%s)\n", APP_NAME, cmd,
+                printf("%s: Error waiting for %s (%s)\n", APP_NAME, fname,
                     str_error(rc));
-                return rc;
+                return -6;
         }
-        
+
         return 0;
 }