CLI: 'put src-dir' vs 'archive put src-dir' usability

Signed-off-by: Alex Aizman <[email protected]>

cmd/cli/cli/arch_hdlr.go

@@ -289,10 +289,16 @@ func putApndArchHandler(c *cli.Context) (err error) {
 	// multi-file cases
 	//
 	if !a.appendOnly && !a.appendOrPut {
-		warn := fmt.Sprintf("multi-file 'archive put' operation requires either %s or %s",
+		warn := fmt.Sprintf("multi-file 'archive put' operation requires either %s or %s (command line)",
 			qflprn(archAppendOnlyFlag), qflprn(archAppendOrPutFlag))
 		actionWarn(c, warn)
-		fmt.Fprintf(c.App.ErrWriter, "Assuming %s - proceeding to execute...\n\n", qflprn(archAppendOrPutFlag))
+		if flagIsSet(c, yesFlag) {
+			fmt.Fprintf(c.App.ErrWriter, "Assuming %s - proceeding to execute...\n\n", qflprn(archAppendOrPutFlag))
+		} else {
+			if ok := confirm(c, fmt.Sprintf("Proceed to execute 'archive put %s'?", flprn(archAppendOrPutFlag))); !ok {
+				return
+			}
+		}
 		a.appendOrPut = true
 	}
 

cmd/cli/cli/object_hdlr.go

@@ -13,6 +13,7 @@ import (
 	"github.com/NVIDIA/aistore/api"
 	"github.com/NVIDIA/aistore/cmd/cli/teb"
 	"github.com/NVIDIA/aistore/cmn"
+	"github.com/NVIDIA/aistore/cmn/archive"
 	"github.com/NVIDIA/aistore/cmn/cos"
 	"github.com/NVIDIA/aistore/cmn/debug"
 	"github.com/urfave/cli"
@@ -307,6 +308,22 @@ func putHandler(c *cli.Context) (err error) {
 	default: // one directory
 		var ndir int
 
+		if a.dst.oname != "" {
+			warn := fmt.Sprintf("'%s' will be used as a destination name prefix for all files from '%s'",
+				a.dst.oname, a.src.arg)
+			actionWarn(c, warn)
+			if _, err := archive.Mime(a.dst.oname, ""); err == nil {
+				warn := fmt.Sprintf("did you want to use 'archive put' instead, with %q as the destination?",
+					a.dst.oname)
+				actionWarn(c, warn)
+			}
+			if !flagIsSet(c, yesFlag) {
+				if ok := confirm(c, "Proceed anyway?"); !ok {
+					return
+				}
+			}
+		}
+
 		fobjs, err := lsFobj(c, a.src.abspath, "", a.dst.oname, &ndir, a.src.recurs, incl)
 		if err != nil {
 			return err