Author: svn-role
Date: Sat Nov  3 04:00:14 2018
New Revision: 1845637

URL: http://svn.apache.org/viewvc?rev=1845637&view=rev
Log:
Merge the r1842814 group from trunk:

 * r1842814, r1842827, r1842829, r1842877
   Improve 'svn help' for the 'add' command and for the '-N' option.
   Justification:
     Proper documentation was lacking. Another newbie was recently surprised
     by how 'svn add' behaves relative to 'git add'.
   Notes:
     r1842814: Document the obsolete '-N' option properly.
     r1842827: Fully document 'svn add', and include some newbie tips.
     r1842829, r1842877: tweaks to formatting.
   Votes:
     +1: julianfoad, stsp, brane

Modified:
    subversion/branches/1.11.x/   (props changed)
    subversion/branches/1.11.x/STATUS
    subversion/branches/1.11.x/subversion/svn/svn.c
    
subversion/branches/1.11.x/subversion/tests/cmdline/getopt_tests_data/svn_help_log_switch_stdout

Propchange: subversion/branches/1.11.x/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Sat Nov  3 04:00:14 2018
@@ -100,4 +100,4 @@
 /subversion/branches/verify-at-commit:1462039-1462408
 /subversion/branches/verify-keep-going:1439280-1546110
 /subversion/branches/wc-collate-path:1402685-1480384
-/subversion/trunk:1840990-1840991,1840995,1840997,1841059,1841079,1841091,1841098,1841136,1841180,1841272,1841481,1841524-1841525,1841567,1841600-1841602,1841606,1841719,1841725,1841731,1841736,1841742-1841743,1841753-1841754,1841822,1841850,1841867,1842090,1842222-1842223
+/subversion/trunk:1840990-1840991,1840995,1840997,1841059,1841079,1841091,1841098,1841136,1841180,1841272,1841481,1841524-1841525,1841567,1841600-1841602,1841606,1841719,1841725,1841731,1841736,1841742-1841743,1841753-1841754,1841822,1841850,1841867,1842090,1842222-1842223,1842814,1842827,1842829,1842877

Modified: subversion/branches/1.11.x/STATUS
URL: 
http://svn.apache.org/viewvc/subversion/branches/1.11.x/STATUS?rev=1845637&r1=1845636&r2=1845637&view=diff
==============================================================================
--- subversion/branches/1.11.x/STATUS (original)
+++ subversion/branches/1.11.x/STATUS Sat Nov  3 04:00:14 2018
@@ -43,18 +43,6 @@ Veto-blocked changes:
 Approved changes:
 =================
 
- * r1842814, r1842827, r1842829, r1842877
-   Improve 'svn help' for the 'add' command and for the '-N' option.
-   Justification:
-     Proper documentation was lacking. Another newbie was recently surprised
-     by how 'svn add' behaves relative to 'git add'.
-   Notes:
-     r1842814: Document the obsolete '-N' option properly.
-     r1842827: Fully document 'svn add', and include some newbie tips.
-     r1842829, r1842877: tweaks to formatting.
-   Votes:
-     +1: julianfoad, stsp, brane
-
  * r1842334
    Update how we display the OS name on Mac in 'svn --version --verbose'.
    Justification:

Modified: subversion/branches/1.11.x/subversion/svn/svn.c
URL: 
http://svn.apache.org/viewvc/subversion/branches/1.11.x/subversion/svn/svn.c?rev=1845637&r1=1845636&r2=1845637&view=diff
==============================================================================
--- subversion/branches/1.11.x/subversion/svn/svn.c (original)
+++ subversion/branches/1.11.x/subversion/svn/svn.c Sat Nov  3 04:00:14 2018
@@ -165,7 +165,7 @@ const apr_getopt_option_t svn_cl__option
   {"message",       'm', 1, N_("specify log message ARG")},
   {"quiet",         'q', 0, N_("print nothing, or only summary information")},
   {"recursive",     'R', 0, N_("descend recursively, same as 
--depth=infinity")},
-  {"non-recursive", 'N', 0, N_("obsolete; try --depth=files or 
--depth=immediates")},
+  {"non-recursive", 'N', 0, N_("obsolete")},
   {"change",        'c', 1,
                     N_("the change made by revision ARG (like -r ARG-1:ARG)\n"
                        "                             "
@@ -533,13 +533,34 @@ const int svn_cl__global_options[] =
 const svn_opt_subcommand_desc3_t svn_cl__cmd_table[] =
 {
   { "add", svn_cl__add, {0}, {N_(
-     "Put files and directories under version control, scheduling\n"
-     "them for addition to repository.  They will be added in next commit.\n"
+     "Put new files and directories under version control.\n"
      "usage: add PATH...\n"
+     "\n"), N_(
+     "  Schedule unversioned PATHs for addition, so they will become versioned 
and\n"
+     "  be added to the repository in the next commit. Recurse into 
directories by\n"
+     "  default (see the --depth option).\n"
+     "\n"), N_(
+     "  The 'svn add' command is only necessary for files and directories that 
are\n"
+     "  not yet under version control. Unversioned files and directories can 
be\n"
+     "  identified with 'svn status' (see 'svn help status').\n"
+     "\n"), N_(
+     "  The effects of 'svn add' can be undone with 'svn revert' before the 
addition\n"
+     "  has been committed. Once committed, a path can be removed from 
version\n"
+     "  control with 'svn delete', and in some circumstances by running a 
reverse-\n"
+     "  merge (see 'svn help merge' for details).\n"
+     "\n"), N_(
+     "  With --force, add all the unversioned paths found in PATHs and ignore 
the\n"
+     "  rest; otherwise, error out if any specified paths are already 
versioned.\n"
+     "\n"), N_(
+     "  The selection of items to add may be influenced by the 'ignores' 
feature.\n"
+     "  Properties may be attached to the items as configured by the 
'auto-props'\n"
+     "  feature.\n"
     )},
     {opt_targets, 'N', opt_depth, 'q', opt_force, opt_no_ignore, opt_autoprops,
      opt_no_autoprops, opt_parents },
-     {{opt_parents, N_("add intermediate parents")}} },
+    {{opt_parents, N_("add intermediate parents")},
+     {'N', N_("obsolete; same as --depth=empty")},
+     {opt_force, N_("ignore already versioned paths")}} },
 
   { "auth", svn_cl__auth, {0}, {N_(
      "Manage cached authentication credentials.\n"
@@ -635,7 +656,8 @@ const svn_opt_subcommand_desc3_t svn_cl_
      "  See also 'svn help update' for a list of possible characters\n"
      "  reporting the action taken.\n"
     )},
-    {'r', 'q', 'N', opt_depth, opt_force, opt_ignore_externals} },
+    {'r', 'q', 'N', opt_depth, opt_force, opt_ignore_externals},
+    {{'N', N_("obsolete; same as --depth=files")}} },
 
   { "cleanup", svn_cl__cleanup, {0}, {N_(
      "Either recover from an interrupted operation that left the working copy 
locked,\n"
@@ -684,7 +706,8 @@ const svn_opt_subcommand_desc3_t svn_cl_
      "  fixed revision.\n"
     )},
     {'q', 'N', opt_depth, opt_targets, opt_no_unlock, SVN_CL__LOG_MSG_OPTIONS,
-     opt_changelist, opt_keep_changelists, opt_include_externals} },
+     opt_changelist, opt_keep_changelists, opt_include_externals},
+    {{'N', N_("obsolete; same as --depth=empty")}} },
 
   { "copy", svn_cl__copy, {"cp"}, {N_(
      "Copy files and directories in a working copy or repository.\n"
@@ -766,7 +789,9 @@ const svn_opt_subcommand_desc3_t svn_cl_
      opt_internal_diff, 'x', opt_no_diff_added, opt_no_diff_deleted,
      opt_ignore_properties, opt_properties_only,
      opt_show_copies_as_adds, opt_notice_ancestry, opt_summarize, 
opt_changelist,
-     opt_force, opt_xml, opt_use_git_diff_format, opt_patch_compatible} },
+     opt_force, opt_xml, opt_use_git_diff_format, opt_patch_compatible},
+    {{'N', N_("obsolete; same as --depth=files")}} },
+
   { "export", svn_cl__export, {0}, {N_(
      "Create an unversioned copy of a tree.\n"
      "usage: 1. export [-r REV] URL[@PEGREV] [PATH]\n"
@@ -788,7 +813,8 @@ const svn_opt_subcommand_desc3_t svn_cl_
      "  looked up.\n"
     )},
     {'r', 'q', 'N', opt_depth, opt_force, opt_native_eol, opt_ignore_externals,
-     opt_ignore_keywords} },
+     opt_ignore_keywords},
+    {{'N', N_("obsolete; same as --depth=files")}} },
 
   { "help", svn_cl__help, {"?", "h"}, {N_(
      "Describe the usage of this program or its subcommands.\n"
@@ -810,7 +836,8 @@ const svn_opt_subcommand_desc3_t svn_cl_
      "  if --force is specified.\n"
     )},
     {'q', 'N', opt_depth, opt_autoprops, opt_force, opt_no_autoprops,
-     SVN_CL__LOG_MSG_OPTIONS, opt_no_ignore} },
+     SVN_CL__LOG_MSG_OPTIONS, opt_no_ignore},
+    {{'N', N_("obsolete; same as --depth=files")}} },
 
   { "info", svn_cl__info, {0}, {N_(
      "Display information about a local or remote item.\n"
@@ -1341,7 +1368,8 @@ const svn_opt_subcommand_desc3_t svn_cl_
     {'r', 'c', 'N', opt_depth, 'q', opt_force, opt_dry_run, opt_merge_cmd,
      opt_record_only, 'x', opt_ignore_ancestry, opt_accept, opt_reintegrate,
      opt_allow_mixed_revisions, 'v'},
-    { { opt_force, N_("force deletions even if deleted contents don't match") 
} }
+    { { opt_force, N_("force deletions even if deleted contents don't match") 
},
+      {'N', N_("obsolete; same as --depth=files")} }
   },
 
   { "mergeinfo", svn_cl__mergeinfo, {0}, {N_(
@@ -1832,7 +1860,8 @@ const svn_opt_subcommand_desc3_t svn_cl_
     )},
     { 'u', 'v', 'N', opt_depth, 'r', 'q', opt_no_ignore, opt_incremental,
       opt_xml, opt_ignore_externals, opt_changelist},
-    {{'q', N_("don't print unversioned items")}} },
+    {{'q', N_("don't print unversioned items")},
+     {'N', N_("obsolete; same as --depth=immediates")}} },
 
   { "switch", svn_cl__switch, {"sw"}, {N_(
      "Update the working copy to a different URL within the same repository.\n"
@@ -1882,7 +1911,8 @@ const svn_opt_subcommand_desc3_t svn_cl_
      N_("allow switching to a node with no common ancestor")},
      {opt_force,
       N_("handle unversioned obstructions as changes")},
-     {opt_relocate,N_("deprecated; use 'svn relocate'")}}
+     {opt_relocate, N_("deprecated; use 'svn relocate'")},
+     {'N', N_("obsolete; same as --depth=files")}}
   },
 
   { "unlock", svn_cl__unlock, {0}, {N_(
@@ -1945,7 +1975,8 @@ const svn_opt_subcommand_desc3_t svn_cl_
      opt_ignore_externals, opt_changelist, opt_editor_cmd, opt_accept,
      opt_parents, opt_adds_as_modification},
     { {opt_force,
-       N_("handle unversioned obstructions as changes")} } },
+       N_("handle unversioned obstructions as changes")},
+      {'N', N_("obsolete; same as --depth=files")} } },
 
   { "upgrade", svn_cl__upgrade, {0}, {N_(
      "Upgrade the metadata storage format for a working copy.\n"

Modified: 
subversion/branches/1.11.x/subversion/tests/cmdline/getopt_tests_data/svn_help_log_switch_stdout
URL: 
http://svn.apache.org/viewvc/subversion/branches/1.11.x/subversion/tests/cmdline/getopt_tests_data/svn_help_log_switch_stdout?rev=1845637&r1=1845636&r2=1845637&view=diff
==============================================================================
--- 
subversion/branches/1.11.x/subversion/tests/cmdline/getopt_tests_data/svn_help_log_switch_stdout
 (original)
+++ 
subversion/branches/1.11.x/subversion/tests/cmdline/getopt_tests_data/svn_help_log_switch_stdout
 Sat Nov  3 04:00:14 2018
@@ -209,7 +209,7 @@ Valid options:
                                 'BASE'       base rev of item's working copy
                                 'COMMITTED'  last commit at or before BASE
                                 'PREV'       revision just before COMMITTED
-  -N [--non-recursive]     : obsolete; try --depth=files or --depth=immediates
+  -N [--non-recursive]     : obsolete; same as --depth=files
   --depth ARG              : limit operation by depth ARG ('empty', 'files',
                              'immediates', or 'infinity')
   --set-depth ARG          : set new working copy depth to ARG ('exclude',


Reply via email to