[dpkg] 61/187: dpkg-deb: Remove obsolete --old and --new options

[Reiner Herrmann]

This is an automated email from the git hooks/post-receive script.

deki-guest pushed a commit to branch master
in repository dpkg.

commit b42e8e0a76e914cd88de8cbd41150064504a597b
Author: Guillem Jover <guillem at debian.org>
Date:   Tue Aug 9 02:44:59 2016 +0200

    dpkg-deb: Remove obsolete --old and --new options
---
 debian/changelog                    |  1 +
 doc/README.feature-removal-schedule | 18 +++++++++---------
 dpkg-deb/main.c                     | 22 ----------------------
 man/dpkg-deb.man                    |  6 ------
 4 files changed, 10 insertions(+), 37 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index cd2ca78..b2d4217 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -7,6 +7,7 @@ dpkg (1.18.11) UNRELEASED; urgency=medium
   * Add support to dpkg-scanpackages for scanning a single binary file.
     Thanks to Javier Serrano Polo <javier at jasp.net>. Closes: #833964
   * Obsolete dpkg-deb bzip2 and lzma compression methods by emitting errors.
+  * Remove obsolete dpkg-deb --old and --new options.
   * Perl modules:
     - Obsolete Source-Version substvar in Dpkg::Substvars by emitting errors.
   * Packaging:
diff --git a/doc/README.feature-removal-schedule b/doc/README.feature-removal-schedule
index 2471eac..9b00250 100644
--- a/doc/README.feature-removal-schedule
+++ b/doc/README.feature-removal-schedule
@@ -37,15 +37,6 @@ Warning: man page
  required to avoid hardcoding the database layout, and it will be kept as
  long as there's legitimate uses for it and no viable replacements.
 
-What: --new, --old (dpkg-deb options)
-Status: deprecated
-When: 1.18.x
-Warning: program
-Why:
- These options are not future-proof, and do not give the caller any
- guarantee of what exact format version will be used to produce the
- output file. They have been replaced with a new --deb-format option.
-
 What: -L (dpkg-parsechangelog option)
 Status: obsolete
 Since: 1.18.8
@@ -172,3 +163,12 @@ Why:
  that compression, as such there's no point in continuing to support
  compressing new .deb files with that format, although unpacking
  will be kept being supported to handle existing compressed files.
+
+What: --new, --old (dpkg-deb options)
+Status: removed
+When: 1.18.11
+Warning: program
+Why:
+ These options are not future-proof, and do not give the caller any
+ guarantee of what exact format version will be used to produce the
+ output file. They were replaced with a new --deb-format option.
diff --git a/dpkg-deb/main.c b/dpkg-deb/main.c
index fcab668..7892fa9 100644
--- a/dpkg-deb/main.c
+++ b/dpkg-deb/main.c
@@ -102,8 +102,6 @@ usage(const struct cmdinfo *cip, const char *value)
 "      --showformat=<format>        Use alternative format for --show.\n"
 "      --deb-format=<format>        Select archive format.\n"
 "                                     Allowed values: 0.939000, 2.0 (default).\n"
-"      --old                        Legacy alias for '--deb-format=0.939000'.\n"
-"      --new                        Legacy alias for '--deb-format=2.0'.\n"
 "      --nocheck                    Suppress control file check (build bad\n"
 "                                     packages).\n"
 "      --uniform-compression        Use the compression params on all members.\n"
@@ -162,24 +160,6 @@ set_deb_format(const struct cmdinfo *cip, const char *value)
     badusage(_("unknown deb format version: %s"), value);
 }
 
-static void
-set_deb_old(const struct cmdinfo *cip, const char *value)
-{
-  deb_format = DEB_VERSION_OBJECT(0, 939000);
-
-  warning(_("obsolete option '--%s'; please use '--%s' instead"),
-          cip->olong, "deb-format=0.939000");
-}
-
-static void
-set_deb_new(const struct cmdinfo *cip, const char *value)
-{
-  deb_format = DEB_VERSION_OBJECT(2, 0);
-
-  warning(_("obsolete option '--%s'; please use '--%s' instead"),
-          cip->olong, "deb-format=2.0");
-}
-
 struct compress_params compress_params = {
   .type = DPKG_DEB_DEFAULT_COMPRESSOR,
   .strategy = COMPRESSOR_STRATEGY_NONE,
@@ -232,8 +212,6 @@ static const struct cmdinfo cmdinfos[]= {
   ACTION("show",          'W', 0, do_showinfo),
 
   { "deb-format",    0,   1, NULL,           NULL,         set_deb_format   },
-  { "new",           0,   0, NULL,           NULL,         set_deb_new      },
-  { "old",           0,   0, NULL,           NULL,         set_deb_old      },
   { "debug",         'D', 0, &debugflag,     NULL,         NULL,          1 },
   { "verbose",       'v', 0, &opt_verbose,   NULL,         NULL,          1 },
   { "nocheck",       0,   0, &nocheckflag,   NULL,         NULL,          1 },
diff --git a/man/dpkg-deb.man b/man/dpkg-deb.man
index dbdb2c9..fc8b03a 100644
--- a/man/dpkg-deb.man
+++ b/man/dpkg-deb.man
@@ -264,12 +264,6 @@ now obsolete; its only use is when building packages to be parsed by
 versions of dpkg older than 0.93.76 (September 1995), which was released
 as i386 a.out only.
 .TP
-.BR \-\-new
-This is a legacy alias for \fB\-\-deb\-format=2.0\fP.
-.TP
-.BR \-\-old
-This is a legacy alias for \fB\-\-deb\-format=0.939000\fP.
-.TP
 .BR \-\-nocheck
 Inhibits
 .BR "dpkg\-deb \-\-build" 's

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/reproducible/dpkg.git