From: Jim Meyering <meyering@redhat.com>
Date: Sun, 10 Apr 2011 09:26:14 +0000 (+0200)
Subject: maint: install: remove support for --preserve_context ("_", not "-")
X-Git-Tag: v8.11~8
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b80e2c7c258bf8c747ceae4c681ee224a31e9509;p=platform%2Fupstream%2Fcoreutils.git

maint: install: remove support for --preserve_context ("_", not "-")

* src/install.c: Its use has elicited a warning for two years.
Use --preserve-context instead.
* NEWS (changes in behavior): Mention this.
---

diff --git a/NEWS b/NEWS
index 9a9b97a..ceccaca 100644
--- a/NEWS
+++ b/NEWS
@@ -47,6 +47,9 @@ GNU coreutils NEWS                                    -*- outline -*-
   df now aligns columns consistently, and no longer wraps entries
   with longer device identifiers, over two lines.
 
+  install now rejects its long-deprecated --preserve_context option.
+  Use --preserve-context instead.
+
   test now accepts "==" as a synonym for "="
 
 
diff --git a/src/install.c b/src/install.c
index 369547b..dbff9c9 100644
--- a/src/install.c
+++ b/src/install.c
@@ -114,7 +114,6 @@ static char const *strip_program = "strip";
 enum
 {
   PRESERVE_CONTEXT_OPTION = CHAR_MAX + 1,
-  PRESERVE_CONTEXT_OPTION_DEPRECATED,
   STRIP_PROGRAM_OPTION
 };
 
@@ -130,9 +129,6 @@ static struct option const long_options[] =
   {"owner", required_argument, NULL, 'o'},
   {"preserve-timestamps", no_argument, NULL, 'p'},
   {"preserve-context", no_argument, NULL, PRESERVE_CONTEXT_OPTION},
-  /* --preserve_context was silently supported until Apr 2009.
-     FIXME: disable altogether in a year or so.  */
-  {"preserve_context", no_argument, NULL, PRESERVE_CONTEXT_OPTION_DEPRECATED},
   {"strip", no_argument, NULL, 's'},
   {"strip-program", required_argument, NULL, STRIP_PROGRAM_OPTION},
   {"suffix", required_argument, NULL, 'S'},
@@ -839,10 +835,6 @@ main (int argc, char **argv)
           no_target_directory = true;
           break;
 
-        case PRESERVE_CONTEXT_OPTION_DEPRECATED:
-          error (0, 0, _("WARNING: --preserve_context is deprecated; "
-                         "use --preserve-context instead"));
-          /* fall through */
         case PRESERVE_CONTEXT_OPTION:
           if ( ! selinux_enabled)
             {