monotone-commits-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Monotone-commits-diffs] net.venge.monotone.string-sanitization: eaaefe


From: code
Subject: [Monotone-commits-diffs] net.venge.monotone.string-sanitization: eaaefe1f4eb90f39a86e9058c1603408bd4295d9
Date: Wed, 2 Mar 2011 11:01:25 +0100 (CET)

revision:            eaaefe1f4eb90f39a86e9058c1603408bd4295d9
date:                2011-03-02T09:59:42
author:              address@hidden
branch:              net.venge.monotone.string-sanitization
changelog:
merge of '438c41ce95f720b5b110091876fab266481b453d'
     and '8a84cbfb78635c15928397fa96b020927eb52f7a'

manifest:
format_version "1"

new_manifest [f2bc35b325566acd07253b8445c454be3a453f98]

old_revision [438c41ce95f720b5b110091876fab266481b453d]

patch "Makefile.am"
 from [8dd83453a5567b7657c91a0e375433564c218402]
   to [215c640d1a8e1f974513605fe38351258b810a80]

patch "src/cmd_conflicts.cc"
 from [fe887aac3fd84b1048360551dd8c8638d059ea7d]
   to [ad9dc74c27d4d8c50a74e5825ec9e0a12cdc1b09]

patch "src/cmd_merging.cc"
 from [98408f1a05aecbe60293c0607be8f6ae65f83940]
   to [31b3be7d444652ec5ac6d937a987c6a65157c313]

patch "src/options_list.hh"
 from [4af93dbcc45e76d732d073ced7ce21d20472cd00]
   to [3b758ede56a0a9fbd435f5eb5d5c1d9dd4418ca0]

patch "src/packet.cc"
 from [c7972e5821369ca4d8279fb2e7d62196fb06645f]
   to [f61360ed2524fdf53411bd24d022a2a3c9e6e9c7]

patch "src/paths.hh"
 from [52cf28b7d0b27dc3dcc58086196d682c73129c8d]
   to [2939f11db21e23d68530ed7415b9bdfa8dfd5a20]

patch "src/pcrewrap.cc"
 from [53efd25cf048ddb118c0c21d413be62b57fe68e0]
   to [7f5bd4e7d8d9e02f2b0271fa9a0f64dcbce2467d]

patch "src/rcs_import.cc"
 from [332bdeb73dfd9e4e64f533bd930c9d6fdc7299ec]
   to [c2d6d57e4896d8c3e53821e7619933029c83f7e7]

patch "src/revision.cc"
 from [e646c35dae34ea0399ec89c3a86fff6088415efd]
   to [c4c2ae8c94e1bbe97952f8601fe8edf4f36af18e]

patch "src/safe_map.hh"
 from [72cfed654bd458c527b1abef1a691ab3f9ebfb4b]
   to [d6d0b1e34fd93bc95a47f65f48b3f652e6034ee5]

patch "src/selectors.cc"
 from [8325a670b116d87a325075ebe75bb20dbced8de4]
   to [3efba0ce3139290b4088cea0d6e82a912565471e]

patch "src/transforms.cc"
 from [5ddd45879695306827570bd595789564990540fd]
   to [cdfb6854ef4992faba21074145f3c8269b845b11]

patch "src/ui.cc"
 from [08a1162f01fedf3ebaa2c6ff2fcb80fcf8c08180]
   to [8bdaebf0da3e90cbe0026d140c2000e92998693f]

patch "src/work.cc"
 from [7973e15b907a675216583f414415738a14f4da00]
   to [fdfc68740e9492e7fa0c4651e0759de7a231c1c9]

patch "test/func/addition_of_files_and_directories/__driver__.lua"
 from [e29d6372557546b569771cc3191cbbc4a78400a1]
   to [0051df02ca4bdb2c88e5931b087aaa6b3b0e67e5]

patch "test/func/automate_log/__driver__.lua"
 from [58b89a2de80cf56d9bdc3983123aecf71b9475d0]
   to [686e1a3bb354f293e228e11dd630ed88fb6358ba]

patch "test/func/drop_missing_and_unknown_files/__driver__.lua"
 from [6e145212c1085b0f6dc31673386efffdb680d23c]
   to [dd3bde9ef250bed48c56e5f0ae0cb454ddcf9228]

patch "test/func/drop_with_actual_removal/__driver__.lua"
 from [c9d54979b1aa971f6d81159e82160a771c381dec]
   to [898696063d078885fbc42143a8813ec6a7c37bde]

patch "test/func/rename_warns_on_subdir_move/__driver__.lua"
 from [54f3a635476643fd907b987f474d9e70fa93719f]
   to [e8b6134155c2b559c97d153b42280f15c9db4006]

patch "test/func/renaming_a_file/__driver__.lua"
 from [e50cef4ee28f3032e8468da7d41548b0bd1a78e9]
   to [95457ea61436f717087e3334871aa2edeca75f29]

patch "test/func/resolve_conflicts_content/update-1"
 from [d46c485d67feccba79540fc6cd164d963ffdf500]
   to [e7dc8443c2b1baa9f8a7eda7cbcbbd2685cbbab3]

patch "test/func/resolve_conflicts_none/update-1"
 from [9e140e34be5c104056d061ed65dd2efdf9a56c44]
   to [3d979297e64c6051a701f706b977eeda1d3d92d0]

patch "test/func/skip_invalid_paths/__driver__.lua"
 from [18ccf9b67e40c8297e2638e49bbf300770145efc]
   to [1b91c6e9ed33dd1066d9920a9c2a578767f13af9]

patch "test/func/two_parent_workspace_add_drop_rename/__driver__.lua"
 from [f2edf6e4783c744259d791f4452fdabd211451e6]
   to [3caddee7e154c62729fa5f31820a5ac1915c525a]

patch "test/func/undrop/__driver__.lua"
 from [05b13418ee4c4522954a3f2534d548c8019c30fe]
   to [a732b394c0292d56e3b62b151d6085bbdd3e0944]

patch "test/func/ws_ops_with_wrong_node_type/__driver__.lua"
 from [67e446cbe85a052f948cb65a30114d21359c50b9]
   to [5134453dc18f0f438b44c7ae71ef3a8c6548b592]

old_revision [8a84cbfb78635c15928397fa96b020927eb52f7a]

patch "src/migrate_schema.cc"
 from [a613cb26783685a29dd39905551b3ddc7fdaf891]
   to [778717ded91866aaabea3752ebd5d44400ce8595]
============================================================
--- Makefile.am	8dd83453a5567b7657c91a0e375433564c218402
+++ Makefile.am	215c640d1a8e1f974513605fe38351258b810a80
@@ -653,8 +653,8 @@ test/func.status : mtn$(EXEEXT) test/bin
 
 # Creates the needed artifacts for manual test execution
 .PHONY: testers
-testers: mtn$(EXEEXT) $(check_PROGRAMS)
-	run_tester_tests run_func_tests run_unit_tests run_extra_tests
+testers: mtn$(EXEEXT) $(check_PROGRAMS)	\
+	 run_tester_tests run_func_tests run_unit_tests run_extra_tests
 
 # We want the tests re-run even if the .status files already exist.
 # .PHONY does not work for that (bad interaction with pattern rules),
============================================================
--- src/packet.cc	c7972e5821369ca4d8279fb2e7d62196fb06645f
+++ src/packet.cc	f61360ed2524fdf53411bd24d022a2a3c9e6e9c7
@@ -332,7 +332,7 @@ namespace
         privkey_packet(args, body);
       else
         {
-          W(F("unknown packet type: '%s'") % type);
+          W(F("unknown packet type '%s'") % type);
           return;
         }
       ++count;
============================================================
--- src/rcs_import.cc	332bdeb73dfd9e4e64f533bd930c9d6fdc7299ec
+++ src/rcs_import.cc	c2d6d57e4896d8c3e53821e7619933029c83f7e7
@@ -711,10 +711,10 @@ test_parse_rcs_file(system_path const & 
   I(! filename.empty());
   assert_path_is_file(filename);
 
-  P(F("parsing RCS file %s") % filename);
+  P(F("parsing RCS file '%s'") % filename);
   rcs_file r;
   parse_rcs_file(filename.as_external(), r);
-  P(F("parsed RCS file %s OK") % filename);
+  P(F("parsed RCS file '%s' OK") % filename);
 }
 
 
@@ -903,11 +903,11 @@ public:
           }
         catch (oops const & o)
           {
-            W(F("error reading RCS file %s: %s") % file % o.what());
+            W(F("error reading RCS file '%s': %s") % file % o.what());
           }
       }
     else
-      L(FL("skipping non-RCS file %s") % file);
+      L(FL("skipping non-RCS file '%s'") % file);
   }
   virtual ~cvs_tree_walker() {}
 };
@@ -1207,8 +1207,8 @@ import_cvs_repo(project_t & project,
 
 {
   E(!directory_exists(cvsroot / "CVSROOT"), origin::user,
-    F("%s appears to be a CVS repository root directory\n"
-      "try importing a module instead, with 'cvs_import %s/<module_name>")
+    F("'%s' appears to be a CVS repository root directory\n"
+      "try importing a module instead, with 'cvs_import %s/<module_name>'")
     % cvsroot % cvsroot);
 
   cvs_history cvs;
============================================================
--- src/transforms.cc	5ddd45879695306827570bd595789564990540fd
+++ src/transforms.cc	cdfb6854ef4992faba21074145f3c8269b845b11
@@ -81,10 +81,10 @@ error_in_transform(Botan::Exception & e,
 
       E(false, caused_by,
         F("%s\n"
-          "this may be due to a memory glitch, data corruption during\n"
+          "This may be due to a memory glitch, data corruption during\n"
           "a network transfer, corruption of your database or workspace,\n"
-          "or a bug in monotone.  if the error persists, please contact\n"
-          "%s for assistance.\n")
+          "or a bug in monotone.  If the error persists, please report\n"
+          "it to '%s'.")
         % w % PACKAGE_BUGREPORT);
     }
   else
============================================================
--- src/work.cc	7973e15b907a675216583f414415738a14f4da00
+++ src/work.cc	fdfc68740e9492e7fa0c4651e0759de7a231c1c9
@@ -225,7 +225,7 @@ workspace::get_work_rev(revision_t & rev
   catch(exception & e)
     {
       E(false, origin::system,
-        F("workspace is corrupt: reading %s: %s")
+        F("workspace is corrupt: reading '%s': %s")
         % rev_path % e.what());
     }
 
@@ -469,7 +469,7 @@ read_options_file(any_path const & optsp
     }
   catch (exception & e)
     {
-      W(F("Failed to read options file %s: %s") % optspath % e.what());
+      W(F("Failed to read options file '%s': %s") % optspath % e.what());
       return;
     }
 
@@ -518,11 +518,11 @@ read_options_file(any_path const & optsp
           opts.key_dir_given = true;
         }
       else
-        W(F("unrecognized key '%s' in options file %s - ignored")
+        W(F("unrecognized key '%s' in options file '%s' - ignored")
           % opt % optspath);
     }
   E(src.lookahead == EOF, src.made_from,
-    F("Could not parse entire options file %s") % optspath);
+    F("Could not parse entire options file '%s'") % optspath);
 }
 
 static void
@@ -557,7 +557,7 @@ write_options_file(bookkeeping_path cons
     }
   catch(exception & e)
     {
-      W(F("Failed to write options file %s: %s") % optspath % e.what());
+      W(F("Failed to write options file '%s': %s") % optspath % e.what());
     }
 }
 
@@ -1088,13 +1088,13 @@ addition_builder::add_nodes_for(file_pat
   if (ros.has_node(path))
     {
       E(is_dir_t(ros.get_node(path)), origin::user,
-        F("cannot add %s, because %s is recorded as a file "
+        F("cannot add '%s', because '%s' is recorded as a file "
           "in the workspace manifest") % goal % path);
       return;
     }
 
   add_nodes_for(path.dirname(), goal);
-  P(F("adding %s to workspace manifest") % path);
+  P(F("adding '%s' to workspace manifest") % path);
 
   node_id nid = the_null_node;
   switch (get_path_status(path))
@@ -1184,14 +1184,14 @@ addition_builder::visit_file(file_path c
 {
   if ((respect_ignore && work.ignore_file(path)) || db.is_dbfile(path))
     {
-      P(F("skipping ignorable file %s") % path);
+      P(F("skipping ignorable file '%s'") % path);
       return;
     }
 
   if (ros.has_node(path))
     {
       if (!path.empty())
-        P(F("skipping %s, already accounted for in workspace") % path);
+        P(F("skipping '%s', already accounted for in workspace") % path);
       return;
     }
 
@@ -1347,7 +1347,7 @@ editable_working_tree::drop_detached_nod
   map<bookkeeping_path, file_path>::const_iterator i
     = rename_add_drop_map.find(pth);
   I(i != rename_add_drop_map.end());
-  P(F("dropping %s") % i->second);
+  P(F("dropping '%s'") % i->second);
   safe_erase(rename_add_drop_map, pth);
   delete_file_or_dir_shallow(pth);
 }
@@ -1358,7 +1358,7 @@ editable_working_tree::create_dir_node()
   node_id nid = next_nid++;
   bookkeeping_path pth = path_for_detached_nid(nid);
   require_path_is_nonexistent(pth,
-                              F("path %s already exists") % pth);
+                              F("path '%s' already exists") % pth);
   mkdir_p(pth);
   return nid;
 }
@@ -1369,7 +1369,7 @@ editable_working_tree::create_file_node(
   node_id nid = next_nid++;
   bookkeeping_path pth = path_for_detached_nid(nid);
   require_path_is_nonexistent(pth,
-                              F("path %s already exists") % pth);
+                              F("path '%s' already exists") % pth);
   file_data dat;
   source.get_version(content, dat);
   write_data(pth, dat.inner());
@@ -1387,11 +1387,11 @@ editable_working_tree::attach_node(node_
   if (i != rename_add_drop_map.end())
     {
       if (messages)
-        P(F("renaming %s to %s") % i->second % dst_pth);
+        P(F("renaming '%s' to '%s'") % i->second % dst_pth);
       safe_erase(rename_add_drop_map, src_pth);
     }
   else if (messages)
-     P(F("adding %s") % dst_pth);
+     P(F("adding '%s'") % dst_pth);
 
   if (dst_pth == file_path())
     {
@@ -1428,7 +1428,7 @@ editable_working_tree::apply_delta(file_
   calculate_ident(pth, curr_id);
   E(curr_id == old_id, origin::system,
     F("content of file '%s' has changed, not overwriting") % pth);
-  P(F("updating %s") % pth);
+  P(F("updating '%s'") % pth);
 
   file_data dat;
   source.get_version(new_id, dat);
@@ -1603,7 +1603,7 @@ move_conflicting_paths_into_bookkeeping(
   // now().as_iso_8601_extended to eliminate the colons, or some appropriate
   // format for now().as_formatted_localtime would be simple and
   // probably adequate.
-  bookkeeping_path leftover_path = bookkeeping_root / "resolutions";
+  bookkeeping_path leftover_path = bookkeeping_resolutions_dir;
 
   mkdir_p(leftover_path);
 
@@ -1624,7 +1624,7 @@ move_conflicting_paths_into_bookkeeping(
       else
         I(false);
 
-      P(F("moved conflicting path %s to %s") % *i % new_path);
+      P(F("moved conflicting path '%s' to '%s'") % *i % new_path);
     }
 }
 
@@ -1889,7 +1889,7 @@ workspace::perform_deletions(database & 
         F("unable to drop the root directory"));
 
       if (!new_roster.has_node(name))
-        P(F("skipping %s, not currently tracked") % name);
+        P(F("skipping '%s', not currently tracked") % name);
       else
         {
           const_node_t n = new_roster.get_node(name);
@@ -1899,7 +1899,7 @@ workspace::perform_deletions(database & 
               if (!d->children.empty())
                 {
                   E(recursive, origin::user,
-                    F("cannot remove %s/, it is not empty") % name);
+                    F("cannot remove '%s/', it is not empty") % name);
                   for (dir_map::const_iterator j = d->children.begin();
                        j != d->children.end(); ++j)
                     todo.push_front(name / j->first);
@@ -1914,7 +1914,7 @@ workspace::perform_deletions(database & 
                   if (directory_empty(name))
                     delete_file_or_dir_shallow(name);
                   else
-                    W(F("directory %s not empty - "
+                    W(F("directory '%s' not empty - "
                         "it will be dropped but not deleted") % name);
                 }
               else
@@ -1925,11 +1925,11 @@ workspace::perform_deletions(database & 
                   if (file->content == fid)
                     delete_file_or_dir_shallow(name);
                   else
-                    W(F("file %s changed - "
+                    W(F("file '%s' changed - "
                         "it will be dropped but not deleted") % name);
                 }
             }
-          P(F("dropping %s from workspace manifest") % name);
+          P(F("dropping '%s' from workspace manifest") % name);
           new_roster.drop_detached_node(new_roster.detach_node(name));
         }
       todo.pop_front();
@@ -1973,14 +1973,14 @@ workspace::perform_rename(database & db,
         F("cannot rename the workspace root (try '%s pivot_root' instead)")
         % prog_name);
       E(new_roster.has_node(src), origin::user,
-        F("source file %s is not versioned") % src);
+        F("source file '%s' is not versioned") % src);
 
       if (src == dst || dst.is_beneath_of(src))
         {
           if (get_path_status(dst) == path::directory)
-            W(F("cannot move `%s' to a subdirectory of itself, `%s/%s'") % src % dst % src);
+            W(F("cannot move '%s' to a subdirectory of itself, '%s/%s'") % src % dst % src);
           else
-            W(F("`%s' and `%s' are the same file") % src % dst);
+            W(F("'%s' and '%s' are the same file") % src % dst);
         }
       else
         {
@@ -1996,7 +1996,7 @@ workspace::perform_rename(database & db,
           // mtn mv foo bar/foo where bar doesn't exist
           file_path parent = dst.dirname();
           E(get_path_status(parent) == path::directory, origin::user,
-            F("destination path's parent directory %s/ doesn't exist") % parent);
+            F("destination path's parent directory '%s/' doesn't exist") % parent);
 
           renames.insert(make_pair(src, dpath));
           add_parent_dirs(db, nis, *this, dpath, new_roster);
@@ -2013,7 +2013,7 @@ workspace::perform_rename(database & db,
       //    mtn mv --bookkeep-only foo bar
 
       E(get_path_status(dst) == path::directory, origin::user,
-        F("destination %s/ is not a directory") % dst);
+        F("destination '%s/' is not a directory") % dst);
 
       for (set<file_path>::const_iterator i = srcs.begin();
            i != srcs.end(); i++)
@@ -2022,7 +2022,7 @@ workspace::perform_rename(database & db,
             F("cannot rename the workspace root (try '%s pivot_root' instead)")
             % prog_name);
           E(new_roster.has_node(*i), origin::user,
-            F("source file %s is not versioned") % *i);
+            F("source file '%s' is not versioned") % *i);
 
           file_path d = dst / i->basename();
           if (bookkeep_only &&
@@ -2039,12 +2039,12 @@ workspace::perform_rename(database & db,
               d = dst / i->basename();
 
               E(!new_roster.has_node(d), origin::user,
-                F("destination %s already exists in the workspace manifest") % d);
+                F("destination '%s' already exists in the workspace manifest") % d);
             }
 
           if (*i == dst || dst.is_beneath_of(*i))
             {
-              W(F("cannot move `%s' to a subdirectory of itself, `%s/%s'")
+              W(F("cannot move '%s' to a subdirectory of itself, '%s/%s'")
                 % *i % dst % *i);
             }
           else
@@ -2062,7 +2062,7 @@ workspace::perform_rename(database & db,
     {
       node_id nid = new_roster.detach_node(i->first);
       new_roster.attach_node(nid, i->second);
-      P(F("renaming %s to %s in workspace manifest") % i->first % i->second);
+      P(F("renaming '%s' to '%s' in workspace manifest") % i->first % i->second);
     }
 
   parent_map parents;
@@ -2087,16 +2087,16 @@ workspace::perform_rename(database & db,
           }
         else if (!have_src && !have_dst)
           {
-            W(F("%s doesn't exist in workspace, skipping") % s);
+            W(F("'%s' doesn't exist in workspace, skipping") % s);
           }
         else if (have_src && have_dst)
           {
-            W(F("destination %s already exists in workspace, "
+            W(F("destination '%s' already exists in workspace, "
                 "skipping filesystem rename") % d);
           }
         else
           {
-            W(F("%s doesn't exist in workspace and %s does, "
+            W(F("'%s' doesn't exist in workspace and '%s' does, "
                 "skipping filesystem rename") % s % d);
           }
       }
@@ -2123,7 +2123,7 @@ workspace::perform_pivot_root(database &
     F("proposed new root directory '%s' is not a directory") % new_root);
   {
     E(!old_roster.has_node(new_root / bookkeeping_root_component), origin::user,
-      F("proposed new root directory '%s' contains illegal path %s")
+      F("proposed new root directory '%s' contains illegal path '%s'")
       % new_root % bookkeeping_root);
   }
 
@@ -2235,7 +2235,7 @@ workspace::perform_content_update(roster
   delete_dir_shallow(detached);
 
   if (moved_conflicting)
-    P(F("moved some conflicting files into %s/%s") % bookkeeping_root % "resolutions");
+    P(F("moved some conflicting files into '%s'") % bookkeeping_resolutions_dir);
 }
 
 // Local Variables:
============================================================
--- src/ui.cc	08a1162f01fedf3ebaa2c6ff2fcb80fcf8c08180
+++ src/ui.cc	8bdaebf0da3e90cbe0026d140c2000e92998693f
@@ -656,9 +656,9 @@ user_interface::fatal(string const & fat
 user_interface::fatal(string const & fatal)
 {
   inform(F("fatal: %s\n"
-           "this is almost certainly a bug in monotone.\n"
-           "please send this error message, the output of '%s version --full',\n"
-           "and a description of what you were doing to <%s>.")
+           "This is almost certainly a bug in monotone.\n"
+           "Please report this error message, the output of '%s version --full',\n"
+           "and a description of what you were doing to '%s'.")
          % fatal % prog_name % PACKAGE_BUGREPORT);
   global_sanity.dump_buffer();
 }
@@ -669,9 +669,9 @@ user_interface::fatal_db(string const & 
 user_interface::fatal_db(string const & fatal)
 {
   inform(F("fatal: %s\n"
-           "this is almost certainly a bug in monotone.\n"
-           "please send this error message, the output of '%s version --full',\n"
-           "and a description of what you were doing to <%s>.\n"
+           "This is almost certainly a bug in monotone.\n"
+           "Please report this error message, the output of '%s version --full',\n"
+           "and a description of what you were doing to '%s'.\n"
            "This error appears to have been triggered by something in the\n"
            "database you were using, so please preserve it in case it can\n"
            "help in finding the bug.")
============================================================
--- src/revision.cc	e646c35dae34ea0399ec89c3a86fff6088415efd
+++ src/revision.cc	c4c2ae8c94e1bbe97952f8601fe8edf4f36af18e
@@ -376,8 +376,8 @@ parse_revision(basic_io::parser & parser
   parser.esym(syms::format_version);
   parser.str(tmp);
   E(tmp == "1", parser.tok.in.made_from,
-    F("encountered a revision with unknown format, version '%s'\n"
-      "I only know how to understand the version '1' format\n"
+    F("encountered a revision with unknown format, version %s\n"
+      "I only know how to understand the version 1 format\n"
       "a newer version of monotone is required to complete this operation")
     % tmp);
   parser.esym(syms::new_manifest);
============================================================
--- src/selectors.cc	8325a670b116d87a325075ebe75bb20dbced8de4
+++ src/selectors.cc	3efba0ce3139290b4088cea0d6e82a912565471e
@@ -149,7 +149,7 @@ string preprocess_date_for_selector(stri
   if (lua.hook_exists("expand_date"))
     {
       E(lua.hook_expand_date(sel, tmp), origin::user,
-        F("selector '%s' is not a valid date\n") % sel);
+        F("selector '%s' is not a valid date") % sel);
     }
   else
     {
@@ -165,11 +165,11 @@ string preprocess_date_for_selector(stri
   else if (tmp.size()<11 && !equals)
     tmp += "T00:00:00";
   E(tmp.size()==19 || equals, origin::user,
-    F("selector '%s' is not a valid date (%s)") % sel % tmp);
+    F("selector '%s' is not a valid date (internally completed to '%s')") % sel % tmp);
 
   if (sel != tmp)
     {
-      P (F ("expanded date '%s' -> '%s'\n") % sel % tmp);
+      P(F("expanded date '%s' -> '%s'") % sel % tmp);
       sel = tmp;
     }
   if (equals && sel.size() < 19)
@@ -835,7 +835,7 @@ shared_ptr<selector> selector::create(op
               shared_ptr<selector> lhs = idx(items, items.size() - 3).sel;
               shared_ptr<selector> rhs = idx(items, items.size() - 1).sel;
               E(lhs, origin::user,
-                F("selector '%s is invalid, because there is a '%s' someplace it shouldn't be")
+                F("selector '%s' is invalid, because there is a '%s' someplace it shouldn't be")
                 % orig % op);
               shared_ptr<or_selector> lhs_as_or = boost::dynamic_pointer_cast<or_selector>(lhs);
               shared_ptr<and_selector> lhs_as_and = boost::dynamic_pointer_cast<and_selector>(lhs);
============================================================
--- src/paths.hh	52cf28b7d0b27dc3dcc58086196d682c73129c8d
+++ src/paths.hh	2939f11db21e23d68530ed7415b9bdfa8dfd5a20
@@ -97,7 +97,7 @@
 //
 //       all this means that when you want to print out a path, you usually
 //       want to just say:
-//           F("my path is %s") % my_path
+//           F("my path is '%s'") % my_path
 //       i.e., nothing fancy necessary, for purposes of F() just treat it like
 //       it were a string
 
@@ -351,6 +351,8 @@ private:
 
 #define bookkeeping_root (bookkeeping_path("_MTN"))
 #define bookkeeping_root_component (path_component("_MTN"))
+#define bookkeeping_conflicts_file (bookkeeping_path("_MTN/conflicts"))
+#define bookkeeping_resolutions_dir (bookkeeping_path("_MTN/resolutions"))
 // for migration
 #define old_bookkeeping_root_component (path_component("MT"))
 
============================================================
--- src/safe_map.hh	72cfed654bd458c527b1abef1a691ab3f9ebfb4b
+++ src/safe_map.hh	d6d0b1e34fd93bc95a47f65f48b3f652e6034ee5
@@ -22,7 +22,7 @@ do_safe_erase(T & container, typename T:
 {
   if (!container.erase(key))
     global_sanity.generic_failure("safe_erase", origin::internal,
-                                  F("erasing nonexistent key from %s")
+                                  F("erasing nonexistent key from '%s'")
                                   % container_name,
                                   file, line);
 }
@@ -39,7 +39,7 @@ do_safe_insert(T & container, typename T
   std::pair<typename T::iterator, bool> r = container.insert(val);
   if (!r.second)
     global_sanity.generic_failure("safe_insert", origin::internal,
-                                  F("inserting duplicate entry into %s")
+                                  F("inserting duplicate entry into '%s'")
                                   % container_name,
                                   file, line);
   return r.first;
@@ -59,7 +59,7 @@ do_safe_insert(T & container, typename T
   typename T::iterator r = container.insert(where, val);
   if (pre_size == container.size())
     global_sanity.generic_failure("safe_insert", origin::internal,
-                                  F("inserting duplicate entry into %s")
+                                  F("inserting duplicate entry into '%s'")
                                   % container_name,
                                   file, line);
   return r;
@@ -77,7 +77,7 @@ do_safe_get(T & container, typename T::k
   typename T::const_iterator i = container.find(key);
   if (i == container.end())
     global_sanity.generic_failure("safe_get", origin::internal,
-                                  F("fetching nonexistent entry from %s")
+                                  F("fetching nonexistent entry from '%s'")
                                   % container_name,
                                   file, line);
   return i->second;
============================================================
--- src/cmd_merging.cc	98408f1a05aecbe60293c0607be8f6ae65f83940
+++ src/cmd_merging.cc	31b3be7d444652ec5ac6d937a987c6a65157c313
@@ -1178,9 +1178,9 @@ CMD(store, "store", "", CMD_REF(conflict
 CMD(store, "store", "", CMD_REF(conflicts),
     "[LEFT_REVID RIGHT_REVID]",
     N_("Store the conflicts from merging two revisions"),
-    N_("If no arguments are given, LEFT_REVID and RIGHT_REVID default to the "
+    (F("If no arguments are given, LEFT_REVID and RIGHT_REVID default to the "
        "first two heads that would be chosen by the 'merge' command. If "
-       "--conflicts-file is not given, '_MTN/conflicts' is used."),
+       "--conflicts-file is not given, '%s' is used.") % bookkeeping_conflicts_file).str(),
     options::opts::branch | options::opts::conflicts_opts)
 {
   database    db(app);
============================================================
--- test/func/addition_of_files_and_directories/__driver__.lua	e29d6372557546b569771cc3191cbbc4a78400a1
+++ test/func/addition_of_files_and_directories/__driver__.lua	0051df02ca4bdb2c88e5931b087aaa6b3b0e67e5
@@ -13,15 +13,15 @@ check(mtn("add", "file0"), 0, false, tru
 -- newly added files should appear as such
 
 check(mtn("add", "file0"), 0, false, true)
-check(qgrep("adding file0", "stderr"))
+check(qgrep("adding 'file0'", "stderr"))
 
 check(mtn("add", "dir"), 0, false, true)
-check(not qgrep("adding dir/file1", "stderr"))
-check(not qgrep("adding dir/file2", "stderr"))
+check(not qgrep("adding 'dir/file1'", "stderr"))
+check(not qgrep("adding 'dir/file2'", "stderr"))
 
 check(mtn("add", "-R", "dir"), 0, false, true)
-check(qgrep("adding dir/file1", "stderr"))
-check(qgrep("adding dir/file2", "stderr"))
+check(qgrep("adding 'dir/file1'", "stderr"))
+check(qgrep("adding 'dir/file2'", "stderr"))
 
 check(mtn("status"), 0, true)
 check(qgrep("file0", "stdout"))
@@ -34,11 +34,11 @@ check(mtn("add", "file0"), 0, false, tru
 -- (i.e. they should be ignored)
 
 check(mtn("add", "file0"), 0, false, true)
-check(qgrep("skipping file0", "stderr"))
+check(qgrep("skipping 'file0'", "stderr"))
 
 check(mtn("add", "-R", "dir"), 0, false, true)
-check(qgrep("skipping dir/file1", "stderr"))
-check(qgrep("skipping dir/file2", "stderr"))
+check(qgrep("skipping 'dir/file1'", "stderr"))
+check(qgrep("skipping 'dir/file2'", "stderr"))
 
 check(mtn("status"), 0, true)
 check(not qgrep("file0", "stdout"))
@@ -58,14 +58,14 @@ check(mtn("add", "--unknown"), 0, false,
 --check(raw_mtn("ls", "unkown"), 0, true, false)
 
 check(mtn("add", "--unknown"), 0, false, true)
-check(qgrep('adding file3', "stderr"))
---check(not qgrep('adding file4.ignore', "stderr"))
-check(qgrep('adding dir/file5', "stderr"))
---check(not qgrep('adding dir/file6.ignore', "stderr"))
-check(qgrep('adding dir2', "stderr"))
-check(not qgrep('adding dir2/file7', "stderr"))
-check(not qgrep('skipping dir2/file7', "stderr"))
-check(not qgrep('adding test_hooks.lua', "stderr"))
+check(qgrep("adding 'file3'", "stderr"))
+--check(not qgrep("adding 'file4\.ignore'", "stderr"))
+check(qgrep("adding 'dir/file5'", "stderr"))
+--check(not qgrep("adding 'dir/file6\.ignore'", "stderr"))
+check(qgrep("adding 'dir2'", "stderr"))
+check(not qgrep("adding 'dir2/file7'", "stderr"))
+check(not qgrep("skipping 'dir2/file7'", "stderr"))
+check(not qgrep("adding 'test_hooks.lua'", "stderr"))
 
 check(mtn("status"), 0, true)
 check(not qgrep("file0", "stdout"))
============================================================
--- test/func/drop_missing_and_unknown_files/__driver__.lua	6e145212c1085b0f6dc31673386efffdb680d23c
+++ test/func/drop_missing_and_unknown_files/__driver__.lua	dd3bde9ef250bed48c56e5f0ae0cb454ddcf9228
@@ -10,7 +10,7 @@ check(mtn("drop", "--bookkeep-only", "ma
 remove("maude")
 
 check(mtn("drop", "--bookkeep-only", "maude"), 0, false, true)
-check(qgrep('dropping maude from workspace manifest', "stderr"))
+check(qgrep("dropping 'maude' from workspace manifest", "stderr"))
 
 check(mtn("status"), 0, true)
 check(qgrep("maude", "stdout"))
@@ -18,14 +18,14 @@ check(mtn("drop", "foobar"), 0, false, t
 check(not qgrep("places/cemetery", "stdout"))
 
 check(mtn("drop", "foobar"), 0, false, true)
-check(qgrep("skipping foobar", "stderr"))
+check(qgrep("skipping 'foobar'", "stderr"))
 
 remove("harold")
 remove("places/cemetery")
 
 check(mtn("drop", "--bookkeep-only", "--missing"), 0, false, true)
-check(qgrep('dropping harold from workspace manifest', "stderr"))
-check(qgrep('dropping places/cemetery from workspace manifest', "stderr"))
+check(qgrep("dropping 'harold' from workspace manifest", "stderr"))
+check(qgrep("dropping 'places/cemetery' from workspace manifest", "stderr"))
 
 check(mtn("status"), 0, true)
 check(qgrep("maude", "stdout"))
============================================================
--- test/func/renaming_a_file/__driver__.lua	e50cef4ee28f3032e8468da7d41548b0bd1a78e9
+++ test/func/renaming_a_file/__driver__.lua	95457ea61436f717087e3334871aa2edeca75f29
@@ -49,7 +49,7 @@ check(mtn("rename", "--bookkeep-only", "
 check(mtn("status"), 0, false, false)
 rename("bar", "barfoo")
 check(mtn("rename", "--bookkeep-only", "bar", "barfoo"), 0, false, true)
-check(qgrep('renaming bar to barfoo in workspace manifest', "stderr"))
+check(qgrep("renaming 'bar' to 'barfoo' in workspace manifest", "stderr"))
 check(mtn("status"), 0, false, false)
 
 -- move file to wrong place before renaming it
@@ -58,5 +58,5 @@ check(mtn("rename", "--bookkeep-only", "
 check(mtn("status"), 0, false, false)
 rename("bar", "barfoofoo")
 check(mtn("rename", "--bookkeep-only", "bar", "barfoo"), 0, false, true)
-check(qgrep('renaming bar to barfoo in workspace manifest', "stderr"))
+check(qgrep("renaming 'bar' to 'barfoo' in workspace manifest", "stderr"))
 check(mtn("status"), 1, false, false)
============================================================
--- test/func/drop_with_actual_removal/__driver__.lua	c9d54979b1aa971f6d81159e82160a771c381dec
+++ test/func/drop_with_actual_removal/__driver__.lua	898696063d078885fbc42143a8813ec6a7c37bde
@@ -77,7 +77,7 @@ check(mtn("drop", "file2"), 0, false, tr
 -- drop of changed committed file
 -- it remains and a warning is printed
 check(mtn("drop", "file2"), 0, false, true)
-check(qgrep("file file2 changed", "stderr"))
+check(qgrep("file 'file2' changed", "stderr"))
 check(exists("file2"))
 
 -- drop of changed committed file --bookkeep-
--- test/func/ws_ops_with_wrong_node_type/__driver__.lua	67e446cbe85a052f948cb65a30114d21359c50b9
+++ test/func/ws_ops_with_wrong_node_type/__driver__.lua	5134453dc18f0f438b44c7ae71ef3a8c6548b592
@@ -17,6 +17,6 @@ check(mtn("add", "dir2"), 0, false, true
 -- should now be already added (this is a soft error now.  see stderr to
 -- ensure proper response
 check(mtn("add", "dir2"), 0, false, true)
-check(qgrep("skipping dir2, already accounted for in workspace", "stderr"))
+check(qgrep("skipping 'dir2', already accounted for in workspace", "stderr"))
 -- should have happened already in the --bookkeep-only version above.
 check(mtn("rename", "file", "dir2"), 1, false, false)
============================================================
--- src/options_list.hh	4af93dbcc45e76d732d073ced7ce21d20472cd00
+++ src/options_list.hh	3b758ede56a0a9fbd435f5eb5d5c1d9dd4418ca0
@@ -79,7 +79,7 @@
  *   DEPRECATE(option, reason, deprecated_in, will_remove_in)
  *     Do not show the named option in help output (even with --hidden), and
  *     give a warning if it is used. The reason should be
- *     gettext_noopt("some text here") as it is translatable.
+ *     gettext_noop("some text here") as it is translatable.
  *
  *
  *   Option Strings
@@ -381,8 +381,9 @@ SIMPLE_OPTION(move_conflicting_paths,
 SIMPLE_OPTION(move_conflicting_paths,
               "move-conflicting-paths/no-move-conflicting-paths",
               bool,
-              gettext_noop("move conflicting, unversioned paths into _MTN/resolutions "
-                           "before proceeding with any workspace change"))
+              (F("move conflicting, unversioned paths into '%s' "
+                 "before proceeding with any workspace change") %
+                 bookkeeping_resolutions_dir).str().c_str())
 
 OPTSET_REL(globals, ssh_sign)
 SIMPLE_INITIALIZED_OPTION(ssh_sign, "ssh-sign", enum_string, "yes,no,only,check",
@@ -589,7 +590,7 @@ OPTVAR(resolve_conflicts_opts, bookkeepi
 
 OPTSET(resolve_conflicts_opts)
 OPTVAR(resolve_conflicts_opts, bookkeeping_path,
-       resolve_conflicts_file, "_MTN/conflicts")
+       resolve_conflicts_file, bookkeeping_conflicts_file)
 
 OPTION(resolve_conflicts_opts, resolve_conflicts_file, true, "resolve-conflicts-file",
        gettext_noop("use file to resolve conflicts"))
@@ -610,7 +611,7 @@ OPTSET(conflicts_opts)
        gettext_noop("specify conflict resolutions in a file, instead of interactively"))
 
 OPTSET(conflicts_opts)
-OPTVAR(conflicts_opts, bookkeeping_path, conflicts_file, bookkeeping_path("_MTN/conflicts"))
+OPTVAR(conflicts_opts, bookkeeping_path, conflicts_file, bookkeeping_conflicts_file)
 
 OPTION(conflicts_opts, conflicts_file, true, "conflicts-file",
        gettext_noop("file in which to store conflicts"))
============================================================
--- src/pcrewrap.cc	53efd25cf048ddb118c0c21d413be62b57fe68e0
+++ src/pcrewrap.cc	7f5bd4e7d8d9e02f2b0271fa9a0f64dcbce2467d
@@ -216,7 +216,7 @@ pcre_compile_error(int errcode, char con
     case 52: // internal error: overran compiling workspace
     case 53: // internal error: previously-checked referenced subpattern
              // not found
-      throw oops((F("while compiling regex \"%s\": %s") % pattern % err)
+      throw oops((F("while compiling regex '%s': %s") % pattern % err)
                  .str().c_str());
 
     default:
@@ -225,9 +225,9 @@ pcre_compile_error(int errcode, char con
       // position-ful variant for all errors, but I'm leaving the == -1 check
       // here in case PCRE gets fixed.
       E(false, caused_by, (erroff == -1
-                           ? (F("error in regex \"%s\": %s")
+                           ? (F("error in regex '%s': %s")
                               % pattern % err)
-                           : (F("error near char %d of regex \"%s\": %s")
+                           : (F("error near char %d of regex '%s': %s")
                               % (erroff + 1) % pattern % err)
                            ));
     }
@@ -242,7 +242,7 @@ pcre_study_error(char const * err, char 
   if (!std::strcmp(err, "failed to get memory"))
     throw std::bad_alloc();
   else
-    throw oops((F("while studying regex \"%s\": %s") % pattern % err)
+    throw oops((F("while studying regex '%s': %s") % pattern % err)
                .str().c_str());
 }
 
============================================================
--- test/func/two_parent_workspace_add_drop_rename/__driver__.lua	f2edf6e4783c744259d791f4452fdabd211451e6
+++ test/func/two_parent_workspace_add_drop_rename/__driver__.lua	3caddee7e154c62729fa5f31820a5ac1915c525a
@@ -17,12 +17,12 @@ check(mtn("add", "fudgie"), 0, false, tr
 writefile("fudgie", "fudgie content")
 
 check(mtn("add", "fudgie"), 0, false, true)
-check(qgrep("adding fudgie", "stderr"))
+check(qgrep("adding 'fudgie'", "stderr"))
 
 check(mtn("drop", "--bookkeep-only", "fudgie"), 0, false, true)
-check(qgrep("dropping fudgie", "stderr"))
+check(qgrep("dropping 'fudgie'", "stderr"))
 
 check(mtn("rename", "foo", "bar"), 0, false, true)
-check(qgrep("renaming foo to bar", "stderr"))
+check(qgrep("renaming 'foo' to 'bar'", "stderr"))
 
 commit()
============================================================
--- test/func/resolve_conflicts_content/update-1	d46c485d67feccba79540fc6cd164d963ffdf500
+++ test/func/resolve_conflicts_content/update-1	e7dc8443c2b1baa9f8a7eda7cbcbbd2685cbbab3
@@ -2,8 +2,8 @@ mtn: [right] b221355d26015cd04668aa4587c
 mtn: selected update target b221355d26015cd04668aa4587cff8068e4646a2
 mtn: [left]  a047a11aae2e43f05171d8ffd8bbfde054f8a4f5
 mtn: [right] b221355d26015cd04668aa4587cff8068e4646a2
-mtn: updating files/bar
-mtn: updating files/baz
-mtn: updating files/inter1
-mtn: updating files/inter2
+mtn: updating 'files/bar'
+mtn: updating 'files/baz'
+mtn: updating 'files/inter1'
+mtn: updating 'files/inter2'
 mtn: updated to base revision b221355d26015cd04668aa4587cff8068e4646a2
============================================================
--- src/cmd_conflicts.cc	fe887aac3fd84b1048360551dd8c8638d059ea7d
+++ src/cmd_conflicts.cc	ad9dc74c27d4d8c50a74e5825ec9e0a12cdc1b09
@@ -374,7 +374,7 @@ set_first_conflict(database & db,
                       {
                         file_path left_path;
                         conflicts.left_roster->get_name(conflict.nid, left_path);
-                        result_path = bookkeeping_path("_MTN/resolutions", origin::internal) / left_path;
+                        result_path = bookkeeping_resolutions_dir / left_path;
                       }
                       break;
 
@@ -519,14 +519,11 @@ CMD(clean, "clean", "", CMD_REF(conflict
     "",
     options::opts::none)
 {
-  bookkeeping_path conflicts_file("_MTN/conflicts");
-  bookkeeping_path resolutions_dir("_MTN/resolutions");
+  if (path_exists(bookkeeping_conflicts_file))
+    delete_file(bookkeeping_conflicts_file);
 
-  if (path_exists(conflicts_file))
-    delete_file(conflicts_file);
-
-  if (path_exists(resolutions_dir))
-    delete_dir_recursive(resolutions_dir);
+  if (path_exists(bookkeeping_resolutions_dir))
+    delete_dir_recursive(bookkeeping_resolutions_dir);
 }
 
 // Local Variables:
============================================================
--- test/func/resolve_conflicts_none/update-1	9e140e34be5c104056d061ed65dd2efdf9a56c44
+++ test/func/resolve_conflicts_none/update-1	3d979297e64c6051a701f706b977eeda1d3d92d0
@@ -2,7 +2,7 @@ mtn: [right] 7f38dd536c4f995a4a7ccd788d2
 mtn: selected update target 7f38dd536c4f995a4a7ccd788d2b5be2755805cf
 mtn: [left]  9c1b716d160fd291ed7c68f5b6387cfbf8afb6fb
 mtn: [right] 7f38dd536c4f995a4a7ccd788d2b5be2755805cf
-mtn: updating bar
-mtn: updating baz
-mtn: updating foo
+mtn: updating 'bar'
+mtn: updating 'baz'
+mtn: updating 'foo'
 mtn: updated to base revision 7f38dd536c4f995a4a7ccd788d2b5be2755805cf
============================================================
--- test/func/skip_invalid_paths/__driver__.lua	18ccf9b67e40c8297e2638e49bbf300770145efc
+++ test/func/skip_invalid_paths/__driver__.lua	1b91c6e9ed33dd1066d9920a9c2a578767f13af9
@@ -9,7 +9,7 @@ check(qgrep("skipping file 'foo/\\\\' wi
 
 check(mtn("add", "--recursive", "foo"), 0, false, true)
 check(qgrep("skipping file 'foo/\\\\' with unsupported name", "stderr"))
-check(qgrep("adding foo to workspace manifest", "stderr"))
+check(qgrep("adding 'foo' to workspace manifest", "stderr"))
 
 check(mtn("add", "foo/\\"), 1, false, true)
 check(qgrep("misuse: path 'foo/\\\\' is invalid", "stderr"))
============================================================
--- test/func/rename_warns_on_subdir_move/__driver__.lua	54f3a635476643fd907b987f474d9e70fa93719f
+++ test/func/rename_warns_on_subdir_move/__driver__.lua	e8b6134155c2b559c97d153b42280f15c9db4006
@@ -6,10 +6,10 @@ check(mtn("mv", "file", "file"), 1, fals
 commit()
 
 check(mtn("mv", "file", "file"), 1, false, true)
-check(qgrep("destination file/ is not a directory", "stderr"))
+check(qgrep("destination 'file/' is not a directory", "stderr"))
 
 check(mtn("mv", "dir", "dir"), 0, false, true)
-check(qgrep("cannot move `dir' to a subdirectory of itself, `dir/dir'", "stderr"))
+check(qgrep("cannot move 'dir' to a subdirectory of itself, 'dir/dir'", "stderr"))
 
 check(mtn("mv", "dir", "dir/subdir"), 0, false, true)
-check(qgrep("cannot move `dir' to a subdirectory of itself, `dir/subdir/dir'", "stderr"))
+check(qgrep("cannot move 'dir' to a subdirectory of itself, 'dir/subdir/dir'", "stderr"))
============================================================
--- test/func/undrop/__driver__.lua	05b13418ee4c4522954a3f2534d548c8019c30fe
+++ test/func/undrop/__driver__.lua	a732b394c0292d56e3b62b151d6085bbdd3e0944
@@ -17,7 +17,7 @@ check(mtn("drop", "changed"), 0, false, 
 -- With changes before 'drop', 'undrop' is like 'revert --bookkeeponly'
 writefile("changed", "modified")
 check(mtn("drop", "changed"), 0, false, true)
-check(qgrep("file changed changed - it will be dropped but not deleted", "stderr"))
+check(qgrep("file 'changed' changed - it will be dropped but not deleted", "stderr"))
 check(mtn("undrop", "changed"), 0, false, false)
 check(readfile("changed")=="modified")
 check(mtn("status"), 0, true, false)
@@ -30,8 +30,8 @@ check(mtn("drop", "changed", "unchanged"
 
 writefile("changed", "modified")
 check(mtn("drop", "changed", "unchanged"), 0, false, true)
-check(qgrep("file changed changed - it will be dropped but not deleted", "stderr"))
-check(qgrep("dropping unchanged from workspace", "stderr"))
+check(qgrep("file 'changed' changed - it will be dropped but not deleted", "stderr"))
+check(qgrep("dropping 'unchanged' from workspace", "stderr"))
 check(mtn("undrop", "changed", "unchanged"), 0, false, false)
 check(readfile("changed")=="modified")
 check(readfile("unchanged")=="base")
@@ -48,10 +48,10 @@ check(mtn("drop", "--recursive", "dir1")
 
 writefile("dir1/file1", "file1-changed")
 check(mtn("drop", "--recursive", "dir1"), 0, false, true)
-check(qgrep("file dir1/file1 changed - it will be dropped but not deleted", "stderr"))
-check(qgrep("directory dir1 not empty - it will be dropped but not deleted", "stderr"))
-check(qgrep("dropping dir1/file2 from workspace", "stderr"))
-check(qgrep("dropping dir1 from workspace", "stderr"))
+check(qgrep("file 'dir1/file1' changed - it will be dropped but not deleted", "stderr"))
+check(qgrep("directory 'dir1' not empty - it will be dropped but not deleted", "stderr"))
+check(qgrep("dropping 'dir1/file2' from workspace", "stderr"))
+check(qgrep("dropping 'dir1' from workspace", "stderr"))
 
 check(mtn("undrop", "dir1"), 0, false, true)
 check(readfile("dir1/file1")=="file1-changed")
@@ -63,9 +63,9 @@ check(mtn("drop", "--recursive", "dir1")
 writefile("dir1/file1", "file1")
 
 check(mtn("drop", "--recursive", "dir1"), 0, false, true)
-check(qgrep("dropping dir1/file1 from workspace", "stderr"))
-check(qgrep("dropping dir1/file2 from workspace", "stderr"))
-check(qgrep("dropping dir1 from workspace", "stderr"))
+check(qgrep("dropping 'dir1/file1' from workspace", "stderr"))
+check(qgrep("dropping 'dir1/file2' from workspace", "stderr"))
+check(qgrep("dropping 'dir1' from workspace", "stderr"))
 
 check(mtn("undrop", "dir1"), 0, false, true)
 check(readfile("dir1/file1")=="file1")
@@ -79,7 +79,7 @@ check(mtn("drop", "dir1"), 1, false, tru
 
 -- drop a directory without --recursive gives an error, so 'undrop --recursive' is redundant
 check(mtn("drop", "dir1"), 1, false, true)
-check(qgrep("cannot remove dir1/, it is not empty", "stderr"))
+check(qgrep("cannot remove 'dir1/', it is not empty", "stderr"))
 
 -- File that was dropped and committed cannot be undropped
 check(mtn("drop", "changed"), 0, false, false)
============================================================
--- test/func/automate_log/__driver__.lua	58b89a2de80cf56d9bdc3983123aecf71b9475d0
+++ test/func/automate_log/__driver__.lua	686e1a3bb354f293e228e11dd630ed88fb6358ba
@@ -3,7 +3,7 @@ mtn_setup()
 mtn_setup()
 
 -- empty branch
-check(mtn("automate", "log"), 1, false, true)
+check(mtn("automate", "log"), 0, false, true)
 check(qgrep("warning: workspace has no parent revision,", "stderr"))
 
 includecommon("automate_ancestry.lua")
============================================================
--- src/migrate_schema.cc	a613cb26783685a29dd39905551b3ddc7fdaf891
+++ src/migrate_schema.cc	778717ded91866aaabea3752ebd5d44400ce8595
@@ -381,7 +381,7 @@ char const migrate_merge_url_and_group[]
   // migrate the posting_queue table
   "ALTER TABLE posting_queue RENAME TO tmp;"
   "CREATE TABLE posting_queue"
-  "  ( url not null,   -- URI we are going to send this to\n"
+  "  ( url not null,   -- URL we are going to send this to\n"
   "    content not null -- the packets we're going to send\n"
   "  );"
   "INSERT INTO posting_queue"
@@ -391,7 +391,7 @@ char const migrate_merge_url_and_group[]
   // migrate the incoming_queue table
   "ALTER TABLE incoming_queue RENAME TO tmp;"
   "CREATE TABLE incoming_queue "
-  "  ( url not null,    -- URI we got this bundle from\n"
+  "  ( url not null,    -- URL we got this bundle from\n"
   "    content not null -- the packets we're going to read\n"
   "  );"
   "INSERT INTO incoming_queue"
@@ -401,7 +401,7 @@ char const migrate_merge_url_and_group[]
   // migrate the sequence_numbers table
   "ALTER TABLE sequence_numbers RENAME TO tmp;"
   "CREATE TABLE sequence_numbers "
-  "  ( url primary key, -- URI to read from\n"
+  "  ( url primary key, -- URL to read from\n"
   "    major not null,  -- 0 in news servers, may be higher in depots\n"
   "    minor not null   -- last article / packet sequence number we got\n"
   "  );"

reply via email to

[Prev in Thread] Current Thread [Next in Thread]