Changeset a6fc88a in mainline for uspace/app/bdsh/cmds

Timestamp:

2017-04-03T21:15:17Z (7 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

8fe46a0

Parents:

ea4a3f0

Message:

Let vfs_link() and vfs_link_path() return the linked file handle

Add an output argument to both vfs_link() and vfs_link_path() so that the
client may receive the file handle of the linked file or directory. This
makes it easy to work with the new file or directory as it is ready to be
opened or in case of a directory to use it as a parent for further
operations without the need for additional vfs_lookup().

Location:

uspace/app/bdsh/cmds/modules

Files:

• cp/cp.c (modified) (2 diffs)
• mkdir/mkdir.c (modified) (3 diffs)

uspace/app/bdsh/cmds/modules/cp/cp.c

@@ -294 +294 @@
                                 merge_paths(dest_path, PATH_MAX, src_dirname);
 
-                                if (vfs_link_path(dest_path, KIND_DIRECTORY) != EOK) {
+                                if (vfs_link_path(dest_path, KIND_DIRECTORY,
+                                    NULL) != EOK) {
                                         printf("Unable to create "
                                             "dest directory %s\n", dest_path);
@@ -308 +309 @@
                          * e.g. cp -r /src /data/new_dir_src
                          */
-                        if (vfs_link_path(dest_path, KIND_DIRECTORY) != EOK) {
+                        if (vfs_link_path(dest_path, KIND_DIRECTORY,
+                            NULL) != EOK) {
                                 printf("Unable to create "
                                     "dest directory %s\n", dest_path);

uspace/app/bdsh/cmds/modules/mkdir/mkdir.c

@@ -96 +96 @@
 
         if (!create_parents) {
-                ret = vfs_link_path(path, KIND_DIRECTORY);
+                ret = vfs_link_path(path, KIND_DIRECTORY, NULL);
                 if (ret != EOK) {
                         cli_error(CL_EFAIL, "%s: could not create %s (%s)",
@@ -135 +135 @@
                         path[prev_off] = 0;
 
-                        ret = vfs_link_path(path, KIND_DIRECTORY);
+                        ret = vfs_link_path(path, KIND_DIRECTORY, NULL);
                         if (ret != EOK && ret != EEXIST) {
                                 cli_error(CL_EFAIL, "%s: could not create %s (%s)",
@@ -146 +146 @@
                 }
                 /* Create the final directory. */
-                ret = vfs_link_path(path, KIND_DIRECTORY);
+                ret = vfs_link_path(path, KIND_DIRECTORY, NULL);
                 if (ret != EOK) {
                         cli_error(CL_EFAIL, "%s: could not create %s (%s)",