Timothy Brownawell wrote: > 'mtn drop -R' is recursive. (hey, we do have a --recursive option. Can > we add a --non-recursive and kill --depth?)
Attached is a patch that changes mtn default behavior to add non-recursively and uses the -R option to reenable old behavior. This patch makes the add and drop commands more similar.
--depth is defined on commit, any other commands?It would seem that it made sense to add --depth to commit at some point or for a certain use-case.
Cheers, -- Ulf
#
# old_revision [970b5cf42c0202a0b1a236ed7034f47c117c2f79]
#
# patch "cmd_ws_commit.cc"
# from [d91beffea3f99a325b152c0a7160ba966f1592f9]
# to [5a78b364b4281679e14e1282934c1ea2ce0b30e7]
#
============================================================
--- cmd_ws_commit.cc d91beffea3f99a325b152c0a7160ba966f1592f9
+++ cmd_ws_commit.cc 5a78b364b4281679e14e1282934c1ea2ce0b30e7
@@ -250,10 +250,11 @@ CMD(add, N_("workspace"), N_("[PATH]..."
CMD(add, N_("workspace"), N_("[PATH]..."),
- N_("add files to workspace"), option::unknown)
+ N_("add files to workspace"), option::unknown % option::recursive)
{
if (!app.unknown && (args.size() < 1))
throw usage(name);
+ N(!app.unknown || !app.recursive, F("cannot set unknown and recursive at the same time"));
app.require_workspace();
@@ -279,7 +280,7 @@ CMD(add, N_("workspace"), N_("[PATH]..."
paths.insert(sp);
}
- bool add_recursive = !app.unknown;
+ bool add_recursive = app.recursive;
app.work.perform_additions(paths, add_recursive);
}
smime.p7s
Description: S/MIME Cryptographic Signature
_______________________________________________ Monotone-devel mailing list [email protected] http://lists.nongnu.org/mailman/listinfo/monotone-devel
