Fix cp and mv so 'cp foo/README bar' where foo and bar are directories,

and README is a file.
 -Erik

6 files changed, 32 insertions, 5 deletions

diff --git a/Changelog b/Changelog
index 5c06ec1..52b34b6 100644
--- a/Changelog
+++ b/Changelog
@@ -1,3 +1,10 @@
+0.41
+	* Fixed a bug in both cp and mv preventing 'cp foo/README bar'
+	    type commands (file in a directory to another directory) 
+	    from working.
+
+	-Erik Andersen, 
+
 0.40
 	* New Apps: sort, uniq. -beppu
 	* New Apps: lsmod, rmmod -erik
diff --git a/Makefile b/Makefile
index f1f25cd..9a59c1f 100644
--- a/Makefile
+++ b/Makefile
@@ -22,7 +22,7 @@ BUILDTIME=$(shell date "+%Y%m%d-%H%M")
 
 # Comment out the following to make a debuggable build
 # Leave this off for production use.
-DODEBUG=false
+DODEBUG=true
 # If you want a static binary, turn this on.  I can't think
 # of many situations where anybody would ever want it static, 
 # but...
diff --git a/coreutils/cp.c b/coreutils/cp.c
index 4af73c2..e96012d 100644
--- a/coreutils/cp.c
+++ b/coreutils/cp.c
@@ -48,6 +48,7 @@ static int srcDirFlag = FALSE;
 static int fileAction(const char *fileName, struct stat* statbuf)
 {
     char newdestName[NAME_MAX];
+    char* newsrcName = NULL;
 
     strcpy(newdestName, destName);
     if ( srcDirFlag == TRUE ) {
@@ -62,7 +63,11 @@ static int fileAction(const char *fileName, struct stat* statbuf)
 	if (newdestName[strlen(newdestName)-1] != '/' ) {
 	    strcat(newdestName, "/");
 	}
-	strcat(newdestName, srcName);
+	newsrcName = strrchr(srcName, '/');
+	if (newsrcName && *newsrcName != '\0')
+	    strcat(newdestName, newsrcName);
+	else
+	    strcat(newdestName, srcName);
     }
     
     return (copyFile(fileName, newdestName, preserveFlag, followLinks));
diff --git a/coreutils/mv.c b/coreutils/mv.c
index 92c40c9..467a360 100644
--- a/coreutils/mv.c
+++ b/coreutils/mv.c
@@ -40,6 +40,7 @@ static int srcDirFlag = FALSE;
 static int fileAction(const char *fileName, struct stat* statbuf)
 {
     char newdestName[NAME_MAX];
+    char* newsrcName = NULL;
 
     strcpy(newdestName, destName);
     if ( srcDirFlag == TRUE ) {
@@ -50,7 +51,11 @@ static int fileAction(const char *fileName, struct stat* statbuf)
 	if (newdestName[strlen(newdestName)-1] != '/' ) {
 	    strcat(newdestName, "/");
 	}
-	strcat(newdestName, srcName);
+	newsrcName = strrchr(srcName, '/');
+	if (newsrcName && *newsrcName != '\0')
+	    strcat(newdestName, newsrcName);
+	else
+	    strcat(newdestName, srcName);
     }
     
     return (copyFile(fileName, newdestName, TRUE, TRUE));
diff --git a/cp.c b/cp.c
index 4af73c2..e96012d 100644
--- a/cp.c
+++ b/cp.c
@@ -48,6 +48,7 @@ static int srcDirFlag = FALSE;
 static int fileAction(const char *fileName, struct stat* statbuf)
 {
     char newdestName[NAME_MAX];
+    char* newsrcName = NULL;
 
     strcpy(newdestName, destName);
     if ( srcDirFlag == TRUE ) {
@@ -62,7 +63,11 @@ static int fileAction(const char *fileName, struct stat* statbuf)
 	if (newdestName[strlen(newdestName)-1] != '/' ) {
 	    strcat(newdestName, "/");
 	}
-	strcat(newdestName, srcName);
+	newsrcName = strrchr(srcName, '/');
+	if (newsrcName && *newsrcName != '\0')
+	    strcat(newdestName, newsrcName);
+	else
+	    strcat(newdestName, srcName);
     }
     
     return (copyFile(fileName, newdestName, preserveFlag, followLinks));
diff --git a/mv.c b/mv.c
index 92c40c9..467a360 100644
--- a/mv.c
+++ b/mv.c
@@ -40,6 +40,7 @@ static int srcDirFlag = FALSE;
 static int fileAction(const char *fileName, struct stat* statbuf)
 {
     char newdestName[NAME_MAX];
+    char* newsrcName = NULL;
 
     strcpy(newdestName, destName);
     if ( srcDirFlag == TRUE ) {
@@ -50,7 +51,11 @@ static int fileAction(const char *fileName, struct stat* statbuf)
 	if (newdestName[strlen(newdestName)-1] != '/' ) {
 	    strcat(newdestName, "/");
 	}
-	strcat(newdestName, srcName);
+	newsrcName = strrchr(srcName, '/');
+	if (newsrcName && *newsrcName != '\0')
+	    strcat(newdestName, newsrcName);
+	else
+	    strcat(newdestName, srcName);
     }
     
     return (copyFile(fileName, newdestName, TRUE, TRUE));